Should have test targets

[jart]

I've noticed this repository doesn't have any test targets. Consider adding some.

[ekuefler]

Yeah, I should really do that.

Is there a recommended structure for tests in rules repositories? Scanning over a few of the other ones they look inconsistent in where they put their tests.

[jart]

I'm not sure if a best practice has been established. But I can share with you what I did with the Closure Rules repository. I put my tests in the following directories:

• //closure/testing/test
• //closure/compiler/test
• //closure/library/test
• //closure/templates/test
• //closure/stylesheets/test
• //closure/protobuf/test

All tests are implemented as rules. This way Jenkins (ci.bazel.io) will know how to run them. Closure Rules has to use a bunch of private testing rules like file_test to aid with this.