Review: Unit testing Fail Report

vutran · 16 January 2020 00:16

I run test on master branch: master_withoutDCS. I found out some fail test.

Can reproduce npm run test.

Can someone confirm this and what type of action should be take to fix these fix?

AndyatFocallocal · 16 January 2020 00:15

vutran · 13 February 2020 02:43

I can try to get these test to pass if I can. If I can fix some of the error. I might create branch to work from master branch.

AndyatFocallocal · 13 February 2020 03:04

Sorry @vutran I forgot to follow this up. I’ll DM Tom and see if he can head over to answer.

tmcnulty · 13 February 2020 17:00

Hey @vutran

Our tests have been neglected for a long time, if you can get them fixed that’d be a great help, as it’s preventing me from adding testing to the build process.

It’s same to assume that any failed test should be fixed, and if you find while fixing it that it’s no longer an applicable test, either removing it or adjusting it would be great too.

Thanks

vutran · 20 February 2020 03:19

I was able to fix some test that was fail to pass. I am not sure error abour Meteor is not defined almost all the test suites.I try to reinstall npm if will help install Meteor. No go. I will continue to research on this issue.

For now I create a branch call: unit_testing_branch. I submit PR request. @AndyatFocallocal