search

codefordenver / Circular

Denver Re:Imagine recycling app
https://re-imagine-dev.firebaseapp.com/
15 stars 19 forks source link

Jest tests #453

Closed dwhite96 closed 5 years ago

dwhite96 commented 5 years ago

Need Jest installed to continue with Firestore security rules unit testing I'm working on in another branch. Request to merge this branch into master so it can be merged in that branch. The only Jest test written is a simple Jest test for the HeroCTA component.

wdoug commented 5 years ago

Have you heard of react-testing-library?

hoop71 commented 5 years ago

Looks like we have Jest installed at the Client root and the Src root? Do we need it in both places? If not, can we delete it one place or the other?

I'm also getting an error: Test suite failed to run.

@wdoug were you able to get these to run?

dwhite96 commented 5 years ago

I'm wondering if we should think about getting rid of the "client" directory since we no longer have a "backend" directory with the node or express code. Then, maybe merge all packages into one package file and one node_modules directory.