Skip to content

Commit

Permalink
jmcvetta#94 - Rename benchmark_test_* files
Browse files Browse the repository at this point in the history
Importing the testing package in a file, which is used by the main binary, will pollute the flags package with testing help usage messages. This hinders the usage of the neoism package in a cli application. The file benchmark_test*.go are loading the testing package and are therefore responsible for the flag pollution. To exclude the benchmark_connect helpers one can suffix the files with _test. This will ensure that the benchmark_connect helpers are only loaded during testing and do not pollute the main binary. Another solution would be to introduce a neoism_test package, which would result a massive code reorganization.
  • Loading branch information
co0p committed Feb 25, 2018
1 parent 2ffb5c6 commit 0db20f8
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 0 deletions.
File renamed without changes.
File renamed without changes.

0 comments on commit 0db20f8

Please sign in to comment.