From 0db20f859291dcfb530abff95525c30b42c22059 Mon Sep 17 00:00:00 2001 From: Julian Godesa Date: Sun, 25 Feb 2018 19:36:56 +0100 Subject: [PATCH] #94 - Rename benchmark_test_* files 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. --- benchmark_test_gae.go => benchmark_gae_test.go | 0 benchmark_test_standalone.go => benchmark_standalone_test.go | 0 2 files changed, 0 insertions(+), 0 deletions(-) rename benchmark_test_gae.go => benchmark_gae_test.go (100%) rename benchmark_test_standalone.go => benchmark_standalone_test.go (100%) diff --git a/benchmark_test_gae.go b/benchmark_gae_test.go similarity index 100% rename from benchmark_test_gae.go rename to benchmark_gae_test.go diff --git a/benchmark_test_standalone.go b/benchmark_standalone_test.go similarity index 100% rename from benchmark_test_standalone.go rename to benchmark_standalone_test.go