Review: Unit testing Fail Report

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?

Imgur
Imgur
Imgur
Imgur

2 Likes

@SysAdmin @tmcnulty?

1 Like

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.

1 Like

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

1 Like

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

2 Likes

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

1 Like