929590 - Allow running just one test from a suite in Zippy

Status: RESOLVED FIXED

(Marketplace Graveyard :: Payments/Refunds, defect)

Comment 1

[Kumar McMillan [:kumar]]

I think if we re-write sellers.test.js to use supertest then we could get rid of runtests.js (no need for a test server) and we could put the nodeunit task back in the grunt file. That might give us some more features like running one file but I'm not sure.

Comment 2

[Kumar McMillan [:kumar]]

Ah, looks like David added a way to send test file args.

https://github.com/mozilla/zippy/commit/f9bd2e51f98c2e112a0fc5b5bf252f5fde5b7a20

Comment 3

[[e-mail]]

This isn't a dupe - David added running a single test suite. This adds running a single test within a suite.

Comment 4

[[e-mail]]

https://github.com/mozilla/zippy/commit/02b027e784e05862df13cec724f370f2ee5d86c3

Comment 5

[[e-mail]]

(In reply to Kumar McMillan [:kumar] from comment #1)
> I think if we re-write sellers.test.js to use supertest then we could get
> rid of runtests.js (no need for a test server) and we could put the nodeunit
> task back in the grunt file. That might give us some more features like
> running one file but I'm not sure.

Yes, if we can do that then that's cool - we should be able to maintain the same grunt args anyway. The main thing to note is if we use the grunt reporter it has very limited support for all the nodeunit reporter features e.g. testspec which is what allows running a single test within a suite. So if this gets refactored we need to avoid using the grunt reporter.

Comment 6

[Victor Carciu]

Can you please add some STRs to this bug or mark it as [qa-] ?