TykTechnologies / tyk / 2532

push

travis-ci

jsvm: don't modify the global logger in the tests

There's no need to, and doing so is racy since plenty of goroutines use
it.

Race report:

	WARNING: DATA RACE
	Write at 0x0000018a7f38 by goroutine 609:
	  github.com/TykTechnologies/tyk.TestJSVMLogs.func1()
	      /home/mvdan/go/src/github.com/TykTechnologies/tyk/plugins_test.go:20 +0x3d
	  github.com/TykTechnologies/tyk.TestJSVMLogs()
	      /home/mvdan/go/src/github.com/TykTechnologies/tyk/plugins_test.go:66 +0x9b9
	  testing.tRunner()
	      /home/mvdan/tip/src/testing/testing.go:754 +0x16c

	Previous read at 0x0000018a7f38 by goroutine 398:
	  github.com/TykTechnologies/tyk.startRateLimitNotifications.func1()
	      /home/mvdan/go/src/github.com/TykTechnologies/tyk/distributed_rate_limiter.go:34 +0x12d

Updates #363.

4 of 4 new or added lines in 1 file covered. (100.0%)

5744 of 13588 relevant lines covered (42.27%)

0.46 hits per line