Skip to content

v2.0.0-rc2

Latest
Compare
Choose a tag to compare
@ansel1 ansel1 released this 27 Jun 03:39

This release changes the v2/flumetest package. It removes the XXXSetDefault() functions and support for configuring log settings from a special, test-specific environment variable (the "config string").

These features presumed that the flumetest package should be used both in your TestMain() as well as at the beginning of each test. But this was too opinionated. The stuff it flumetest was doing in Start() (in test cases) was not related or dependent on the stuff it was doing in SetDefaults() (which was mainly concerned with offering sensible default log settings for tests).

It is simpler and less opinionated for consumers to just use the same flume.ConfigFromEnv() function in TestMain() as they use in Main(). If they want to use different defaults, they can do so trivially in their own code, or write their own setup function for tests. Trying to standardize this in flumetest created complicated problems with ordering of configuration and bootstrap stuff.

  • BREAKING CHANGE Removed SetDefault(), MustSetDefault, and "-log-config" support are removed
  • Improved inline docs