codelab icon indicating copy to clipboard operation
codelab copied to clipboard

Massive Refactor for jest testing

Open O-Mutt opened this issue 5 years ago • 8 comments

This modifies all the tests to run under jest. There are a lot of broken tests still.

Ref: #1136 #1137

O-Mutt avatar Oct 22 '19 14:10 O-Mutt

wow, this looks awesome!

Curious if you were in touch with @hansal7014 (from #1136) to avoid duplicate work?

kirjs avatar Oct 22 '19 14:10 kirjs

wow, this looks awesome!

Curious if you were in touch with @hansal7014 (from #1136) to avoid duplicate work?

Did not, i must have started before those comments came in. It took me a few days of messing around to get everything working well enough

O-Mutt avatar Oct 22 '19 16:10 O-Mutt

Ok, cool, good to know, I pinged the other thread to avoid doing the same stuff twice in the future, for now let's merge this in

kirjs avatar Oct 22 '19 16:10 kirjs

Ok, cool, good to know, I pinged the other thread to avoid doing the same stuff twice in the future, for now let's merge this in

I'm pulled the travis changes because there are still a ton of broken tests.

O-Mutt avatar Oct 22 '19 16:10 O-Mutt

Just wanted to send a friendly ping to understand the state of this and make sure it's not blocked from our side.

NothingEverHappens avatar Oct 29 '19 22:10 NothingEverHappens

Hey @NothingEverHappens this was not blocked just haven't had a time to get back to the enormity of this. This in and of itself should be ok if we wanted to take a small chunk out of the project

O-Mutt avatar Nov 04 '19 19:11 O-Mutt

Cool, would you have some time to address the feedback? (I think it's mostly dropping unused code bytes, and other small things)?

I think this is a big step forward compered to what we have now and would be awesome to get merged.

NothingEverHappens avatar Nov 07 '19 23:11 NothingEverHappens

Hey @Mutmatt, are you planning to look into merging this in? Or alternatively I can work on doing this.

NothingEverHappens avatar Nov 27 '19 23:11 NothingEverHappens