* The tests are much better, but hiding a login() in a helper is a bit obscure. Replaced with a removeSecurityProxy.
* In test_binary_only_false, instead of just testing that builds.count() == 3 for a binary-only query, better to test separately that (1) all its elements are binary builds, (2) the result is nonempty so the test isn't trivial, and (3) the result is smaller than with a non-binary-only query so the test isn't meaningless.
A few more changes discussed on IRC:
* The tests are much better, but hiding a login() in a helper is a bit obscure. Replaced with a removeSecurityP roxy. only_false, instead of just testing that builds.count() == 3 for a binary-only query, better to test separately that (1) all its elements are binary builds, (2) the result is nonempty so the test isn't trivial, and (3) the result is smaller than with a non-binary-only query so the test isn't meaningless.
* In test_binary_
Great shape now. Land that baby!
Quod subigo farinam,
Jeroen