krikkert/habitica
1
0
Fork 0
mirror of https://github.com/HabitRPG/habitica.git synced 2025-12-18 07:07:35 +01:00
Code Issues Packages Projects Releases Wiki Activity

chore(test): increase test coverage around logger lib

Browse Source
This commit is contained in:
Blade Barringer
2016-08-27 07:49:26 -05:00
parent 6480602ee6
commit 45e4c6867e
1 changed files with 145 additions and 17 deletions
Download Patch File Download Diff File Expand all files Collapse all files

130
test/api/v3/unit/libs/logger.js
View File

@@ -1,5 +1,8 @@
import winston from 'winston';
import logger from '../../../../../website/server/libs/logger';
import {
NotFound,
} from '../../../../../website/server/libs//errors';
describe('logger', () => {
let logSpy;
@@ -21,13 +24,16 @@ describe('logger', () => {
});
describe('error', () => {
context('non-error object', () => {
it('passes through arguments if the first arg is not an error object', () => {
logger.error(1, 2, 3, 4);
expect(logSpy).to.be.calledOnce;
expect(logSpy).to.be.calledWith('error', 1, 2, 3, 4);
});
});
it('parses the error and passes it to the logger when the first arg is an error object', () => {
context('error object', () => {
it('logs the stack and the err data', () => {
let errInstance = new Error('An error.');
logger.error(errInstance, {
data: 1,
@@ -42,5 +48,127 @@ describe('logger', () => {
3
);
});
it('logs the stack and the err data with it\'s own fullError property', () => {
let errInstance = new Error('An error.');
let anotherError = new Error('another error');
logger.error(errInstance, {
data: 1,
fullError: anotherError,
}, 2, 3);
expect(logSpy).to.be.calledOnce;
expect(logSpy).to.be.calledWith(
'error',
errInstance.stack,
{ data: 1, fullError: anotherError },
2,
3
);
});
it('logs the error when errorData is null', () => {
let errInstance = new Error('An error.');
logger.error(errInstance, null, 2, 3);
expect(logSpy).to.be.calledOnce;
expect(logSpy).to.be.calledWith(
'error',
errInstance.stack,
null,
2,
3
);
});
it('logs the error when errorData is not an object', () => {
let errInstance = new Error('An error.');
logger.error(errInstance, true, 2, 3);
expect(logSpy).to.be.calledOnce;
expect(logSpy).to.be.calledWith(
'error',
errInstance.stack,
true,
2,
3
);
});
it('logs the error when errorData does not include isHandledError property', () => {
let errInstance = new Error('An error.');
logger.error(errInstance, { httpCode: 400 }, 2, 3);
expect(logSpy).to.be.calledOnce;
expect(logSpy).to.be.calledWith(
'error',
errInstance.stack,
{ httpCode: 400, fullError: errInstance },
2,
3
);
});
it('logs the error when errorData includes isHandledError property but is a 500 error', () => {
let errInstance = new Error('An error.');
logger.error(errInstance, {
isHandledError: true,
httpCode: 502,
}, 2, 3);
expect(logSpy).to.be.calledOnce;
expect(logSpy).to.be.calledWith(
'error',
errInstance.stack,
{ httpCode: 502, isHandledError: true, fullError: errInstance },
2,
3
);
});
it('logs a warning when errorData includes isHandledError property and is not a 500 error', () => {
let errInstance = new Error('An error.');
logger.error(errInstance, {
isHandledError: true,
httpCode: 403,
}, 2, 3);
expect(logSpy).to.be.calledOnce;
expect(logSpy).to.be.calledWith(
'warn',
errInstance.stack,
{ httpCode: 403, isHandledError: true, fullError: errInstance },
2,
3
);
});
it('logs additional data from a CustomError', () => {
let errInstance = new NotFound('An error.');
errInstance.customField = 'Some interesting data';
logger.error(errInstance, {}, 2, 3);
expect(logSpy).to.be.calledOnce;
expect(logSpy).to.be.calledWith(
'error',
errInstance.stack,
{
fullError: {
customField: 'Some interesting data',
},
},
2,
3
);
});
});
});
});