Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

How to make ng test to fail on warnings

I've just finished fixing several warnings on my unit/component tests of my Angular app, that run using ng test (with Karma/Jasmine).

This process can be quite time-consuming as it is not always obvious to spot which test case(s) is actually causing the warning(s).

So now that I don't have any more warnings, I'm wondering if there is a way to make ng test to automatically fail whenever there are any warnings.

like image 901
Francesco Borzi Avatar asked Mar 06 '20 15:03

Francesco Borzi


3 Answers

Define an extra file in your workspace root - called karma.globals.js for example which looks like:

// add globals for karma test runs
console.warn = (message) => { throw new Error(message); };

Now include it in your root karma.conf.js like:

const { join } = require('path');

module.exports = () => {
  return {
    
    // ...

    files: [
      join(__dirname, 'karma.globals.js')
    ]

    // ...
  };
};
like image 88
bohoffi Avatar answered Oct 06 '22 03:10

bohoffi


Maybe something like this in test.ts or someplace where the tests get initialized.

console.warn = (message) => throw new Error(message);

I don't know if I would recommend this because warnings are just that warnings, but errors are errors and should be dealt with immediately. Where you would place this for just the tests can be tricky too.

like image 41
AliF50 Avatar answered Oct 06 '22 03:10

AliF50


A simple and easy fix is a beforeAll block in your test.ts file. It replaces all console methods with an error function.

That way the tests will fail if you call console.log and you will know which ones.

If you still wanna use console.log within a unit being tested, then you could spy on it. But that's also bad due to shared global state.

Here's an example:

const errorFn = () => { throw new Error('No console logs!'); };
beforeAll(() => Object.keys(console).forEach((fn: string) => (console as any)[fn] = errorFn));

like image 30
Darlan Alves Avatar answered Oct 06 '22 03:10

Darlan Alves