Closed Bug 1517881 Opened 6 years ago Closed 6 years ago

Move us onto more modern testing tool

Categories

(Taskcluster :: Services, enhancement)

enhancement
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: bstack, Assigned: bstack)

References

Details

jest seems like a good option. It does some nice things with output during testing, etc. Also has ways of integrating with monorepo nicely.
Depends on: tc-monorepo
Assignee: nobody → bstack
Component: General → Platform Libraries

Do you have an opinion on this, John? Owlish has had good experience with jest but otherwise I don't have a lot of direct experience.

Mostly this is motivated by better integration with monorepo stuff, speed/consistency of tests, and reporting concerns.

Flags: needinfo?(jhford)

It looks like a nice framework, though I haven't really gotten deep into it. My only concern here is that we have working tests with Mocha right now, and depending on how much work it is to port them to jest, it might not be worthwhile for existing tests. Basically, it feels like adding another variable when we've already got a decent amount of churn happening.

That said, I have no strong attachment to Mocha and if Jest is better, then let's do it! Before we decide to stop using Sinon though, we should play around with the Jest mocking/spying/stubbing stuff.

What about converting a smaller library or service to Jest and see how it goes before we make a call?

Flags: needinfo?(jhford)

Basically, it feels like adding another variable when we've already got a decent amount of churn happening.

Yeah, I definitely feel this. The only reason I would want to do it sooner than later is that I want code coverage and the way we would do that with mocha is different than with jest so I'd like to avoid doing it twice.

What about converting a smaller library or service to Jest and see how it goes before we make a call?

https://github.com/taskcluster/taskcluster/compare/jester is a WIP that ports a handful of our smaller libraries. They actually have an automated tool for converting 90% of cases.

Component: Platform Libraries → Services

Meh, I added coverage separately and mocha is more than good enough.

Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.