Merge lp:~allenap/gwacl/tests-not-testing into lp:gwacl

[Gavin Panella]

Get all tests running.

The logging tests were not running, but there was no obvious
indication that there was a problem; the module was developed in the
top-level gwacl package, moved to a sub-package when it was complete,
but it was missing the hook function from testing into gocheck.

In any case, `make check` would not have run tests in any sub-package.
When -gocheck.v is passed into `go test` the package specification
appears to be ignored, and only tests in the current package are
exercised.

Get all tests running.

The logging tests were not running, but there was no obvious
indication that there was a problem; the module was developed in the
top-level gwacl package, moved to a sub-package when it was complete,
but it was missing the hook function from testing into gocheck.

In any case, `make check` would not have run tests in any sub-package.
When -gocheck.v is passed into `go test` the package specification
appears to be ignored, and only tests in the current package are
exercised.

[Gavin Panella (allenap)]

Please take a look.

[Jeroen T. Vermeulen (jtv)]

Thanks for dealing with this. It sounds like a pretty nasty bug if a verbosity option suppresses tests...

review: Approve

[John A Meinel (jameinel)]

With the new bot you have to set the Commit Message before it will land the request. Was that not true with the bot that Julian was running? Do you know how he had it configured in that case?

[Gavin Panella (allenap)]

It was the same, and I am too: I keep forgetting :)

[Gavin Panella (allenap)]

Thanks for sorting it out John.

[Julian Edwards (julian-edwards)]

On Thursday 29 Aug 2013 11:03:37 Jeroen T. Vermeulen wrote:
> Review: Approve
>
> Thanks for dealing with this. It sounds like a pretty nasty bug if a
> verbosity option suppresses tests...

A bug should be filed on gocheck. It was probably not checking an error
return value or something.

=== modified file 'Makefile'
--- Makefile	2013-07-02 02:29:44 +0000
+++ Makefile	2013-08-29 10:57:38 +0000
@@ -1,6 +1,7 @@
-# Build, and run tests.
+# Build, and run tests. Do *not* pass -gocheck.v because it prevents
+# tests in subdirectories from being run (a really _special_ bug).
 check: examples
-	go test -gocheck.v=true ./...
+	go test ./...
 
 debug-test:
 	go test -c -gcflags "-N -l"

=== modified file 'logging/logging_test.go'
--- logging/logging_test.go	2013-04-22 16:17:23 +0000
+++ logging/logging_test.go	2013-08-29 10:57:38 +0000
@@ -5,6 +5,7 @@
 
 import (
     . "launchpad.net/gocheck"
+	"testing"
 )
 
 var originalLevel = level
@@ -37,3 +38,8 @@
     setLevel("123")
     c.Check(level, Equals, -1)
 }
+
+// Master loader for all tests.
+func Test(t *testing.T) {
+    TestingT(t)
+}