Skip to content

Commit

Permalink
Merge pull request #12 from codevor/develop
Browse files Browse the repository at this point in the history
fix: logger when receive object params
  • Loading branch information
helderberto committed Jan 16, 2020
2 parents a768041 + b0c6af3 commit 6b208bf
Show file tree
Hide file tree
Showing 5 changed files with 17 additions and 15 deletions.
2 changes: 1 addition & 1 deletion example.js
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import Logger, { Emoji } from './dist/logger';

Logger.success('Success example log'); // => ✅ Success example log
Logger.success('Success example log', { test: 1 }); // => ✅ Success example log
Logger.error('Error example log'); // => 🚫 Error example log
Logger.info('Info example log'); // => ℹ️ Info example log
Logger.trace('Trace example log'); // => 🐾 Trace example log
Expand Down
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "@codevor/logger.js",
"version": "0.5.1",
"version": "0.5.2",
"description": "💢 Make logging friendlier, simpler and use emojis in your messages. Try logger!",
"main": "dist/logger.js",
"unpkg": "dist/logger.min.js",
Expand Down
12 changes: 6 additions & 6 deletions src/logger.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,12 +2,12 @@ import { intercept } from './utils';

const Logger = (() => {
return {
success: args => intercept('success', args),
info: args => intercept('info', args),
warn: args => intercept('warn', args),
error: args => intercept('error', args),
trace: args => intercept('trace', args),
debug: args => intercept('debug', args),
success: (...args) => intercept('success', args),
info: (...args) => intercept('info', args),
warn: (...args) => intercept('warn', args),
error: (...args) => intercept('error', args),
trace: (...args) => intercept('trace', args),
debug: (...args) => intercept('debug', args),
}
})();

Expand Down
2 changes: 1 addition & 1 deletion src/utils.js
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ const createMessage = (type, message) => {
return message;
}

export const intercept = (type, ...args) => {
export const intercept = (type, args) => {
const message = args.shift();

const parsedMessage = createMessage(type, message);
Expand Down
14 changes: 8 additions & 6 deletions tests/logger.test.js
Original file line number Diff line number Diff line change
@@ -1,38 +1,40 @@
import Logger from '../src';

const MOCK_TEST = { helloWorld: true };

describe('Logger', () => {
test('should create Logger.success', () => {
expect(Logger.success('Success message example')).toEqual(
expect(Logger.success('Success message example', MOCK_TEST)).toEqual(
'✅ Success message example'
);
});

test('should create Logger.info', () => {
expect(Logger.info('Informational message example')).toEqual(
expect(Logger.info('Informational message example', MOCK_TEST)).toEqual(
'ℹ️ Informational message example'
);
});

test('should create Logger.warn', () => {
expect(Logger.warn('Warning message example')).toEqual(
expect(Logger.warn('Warning message example', MOCK_TEST)).toEqual(
'⚠️ Warning message example'
);
});

test('should create Logger.error', () => {
expect(Logger.error('Error message example')).toEqual(
expect(Logger.error('Error message example', MOCK_TEST)).toEqual(
'🚫 Error message example'
);
});

test('should create Logger.trace', () => {
expect(Logger.trace('Trace message example')).toEqual(
expect(Logger.trace('Trace message example', MOCK_TEST)).toEqual(
'🐾 Trace message example'
);
});

test('should create Logger.debug', () => {
expect(Logger.debug('Debug message example')).toEqual(
expect(Logger.debug('Debug message example', MOCK_TEST)).toEqual(
'🐛 Debug message example'
);
});
Expand Down

0 comments on commit 6b208bf

Please sign in to comment.