console errors when running test

[jbibla]

jest or yarn test will result in console errors which the CI ignores, but my git client doesn't like.

 PASS  test/unit/specs/store/node.spec.js
  ● Console

    console.log app/src/renderer/vuex/modules/notifications.js:34
      [ERROR] Error fetching validator set: Cannot read property 'validators'of undefined
    console.log app/src/renderer/vuex/modules/notifications.js:34
      [ERROR] Critical Error: Couldn't initialize the blockchain client. If the problem persists, please make an issue on GitHub.
 PASS  test/unit/specs/store/user.spec.js
  ● Console

    console.log app/src/renderer/vuex/modules/notifications.js:34
      [ERROR] Couldn't read keys: undefined
    console.log node_modules/raven-js/src/console.js:35
      [ERROR] Couldn't switch on error collection.: Error collection is disabled during development.
 PASS  test/unit/specs/components/wallet/PageSend.spec.js
  ● Console

    console.log app/src/renderer/vuex/modules/notifications.js:34
      [ERROR] Error Sending: An error occurred while trying to send: "Cannot read property 'amount' of undefined" 
PASS  test/unit/specs/components/common/TmSessionSignUp.spec.js
  ● Console

    console.log node_modules/raven-js/src/console.js:35
      [ERROR] Couldn't create account: test

 PASS  test/unit/specs/App.spec.js
  ● Console

    console.log app/src/renderer/vuex/modules/notifications.js:34
      [ERROR] Couldn't read keys: node.listKeys is not a function

 PASS  test/unit/specs/components/common/TmSessionSignIn.spec.js
  ● Console

    console.log app/src/renderer/vuex/modules/notifications.js:34
      [ERROR] Signing In Failed: undefined
 PASS  test/unit/specs/store/blockchain.spec.js
  ● Console

    console.log app/src/renderer/vuex/modules/notifications.js:34
      [ERROR] Couldn't query block: undefined

Fix errors in:

• test/unit/specs/store/node.spec.js
• test/unit/specs/store/user.spec.js
• test/unit/specs/components/wallet/PageSend.spec.js
• test/unit/specs/components/common/TmSessionSignUp.spec.js
• test/unit/specs/components/common/TmSessionSignIn.spec.js
• test/unit/specs/store/blockchain.spec.js
• test/unit/specs/App.spec.js

[faboweb]

Those are printed "errors" but in reality they are just console logs and shouldn't cause trouble. What does your git client say?

[faboweb]

@jbibla any feedback?

[jbibla]

i don't think they are effecting my git client. but should these be resolved?

[faboweb]

Let's fix them one at a time.