Skip to content

Conversation

ahmer-yasin
Copy link
Contributor

Ready for test

Issue BD-1116

Description

  • add test in push hook
  • skip failed test cases

Testing Instructions

  • syscoin qt must be run before run test
  • setup syscoin-api
  • checkout BD-1116-unit-test-git-hook
  • cd server/nodejs
  • then run npm run test
  • then add any js file or change any file
  • commit file git-precommit hook will work and it will check lint errors
  • push file into BD-1116-unit-test-git-hook git-prepush hook will work and it will check lint errors and then run test

@ahmer-yasin ahmer-yasin requested a review from jnessbcf July 6, 2018 00:10
@oshchepkov
Copy link
Contributor

@ahmer-yasin , there is a configuration file for unit tests in 'server\nodejs\controllers\test\config.js' For unit test to properly execute the values in that file should reflect the current state of the node you're running the tests on. Setting those values will prevent from skipping too many unit tests.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants