LiskArchive / lisk-sdk

🔩 Lisk software development kit
https://lisk.com
Apache License 2.0
2.72k stars 456 forks source link

Add jest tests to CI pipeline #2946

Closed yatki closed 5 years ago

yatki commented 5 years ago

Expected behavior

Actual behavior

Steps to reproduce

Which version(s) does this affect? (Environment, OS, etc...)

1.6.0

michielmulders commented 5 years ago

Added --passWithNotests flag for jest:integration and jest:functional as the absence of tests makes Jenkins fail.

michielmulders commented 5 years ago

Jest added to Jenkins script and create a command that executes all tests for a certain type e.g. unit. However, using this command, the success/failure of the separate commands are lost between all the logs as they execute in parallel.

Todo @yatki

We can safely remove the --passWithNotests flag in package.json script for running Jest tests whenever we have tests for each type (functional and integration).