Skip to content

Commit b183c08

Browse files
committed
fix: logger pass args destructured
1 parent b98d109 commit b183c08

File tree

3 files changed

+15
-13
lines changed

3 files changed

+15
-13
lines changed

src/logger.js

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -2,12 +2,12 @@ import { intercept } from './utils';
22

33
const Logger = (() => {
44
return {
5-
success: args => intercept('success', args),
6-
info: args => intercept('info', args),
7-
warn: args => intercept('warn', args),
8-
error: args => intercept('error', args),
9-
trace: args => intercept('trace', args),
10-
debug: args => intercept('debug', args),
5+
success: (...args) => intercept('success', args),
6+
info: (...args) => intercept('info', args),
7+
warn: (...args) => intercept('warn', args),
8+
error: (...args) => intercept('error', args),
9+
trace: (...args) => intercept('trace', args),
10+
debug: (...args) => intercept('debug', args),
1111
}
1212
})();
1313

src/utils.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ const createMessage = (type, message) => {
1010
return message;
1111
}
1212

13-
export const intercept = (type, ...args) => {
13+
export const intercept = (type, args) => {
1414
const message = args.shift();
1515

1616
const parsedMessage = createMessage(type, message);

tests/logger.test.js

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,38 +1,40 @@
11
import Logger from '../src';
22

3+
const MOCK_TEST = { helloWorld: true };
4+
35
describe('Logger', () => {
46
test('should create Logger.success', () => {
5-
expect(Logger.success('Success message example')).toEqual(
7+
expect(Logger.success('Success message example', MOCK_TEST)).toEqual(
68
'✅ Success message example'
79
);
810
});
911

1012
test('should create Logger.info', () => {
11-
expect(Logger.info('Informational message example')).toEqual(
13+
expect(Logger.info('Informational message example', MOCK_TEST)).toEqual(
1214
'ℹ️ Informational message example'
1315
);
1416
});
1517

1618
test('should create Logger.warn', () => {
17-
expect(Logger.warn('Warning message example')).toEqual(
19+
expect(Logger.warn('Warning message example', MOCK_TEST)).toEqual(
1820
'⚠️ Warning message example'
1921
);
2022
});
2123

2224
test('should create Logger.error', () => {
23-
expect(Logger.error('Error message example')).toEqual(
25+
expect(Logger.error('Error message example', MOCK_TEST)).toEqual(
2426
'🚫 Error message example'
2527
);
2628
});
2729

2830
test('should create Logger.trace', () => {
29-
expect(Logger.trace('Trace message example')).toEqual(
31+
expect(Logger.trace('Trace message example', MOCK_TEST)).toEqual(
3032
'🐾 Trace message example'
3133
);
3234
});
3335

3436
test('should create Logger.debug', () => {
35-
expect(Logger.debug('Debug message example')).toEqual(
37+
expect(Logger.debug('Debug message example', MOCK_TEST)).toEqual(
3638
'🐛 Debug message example'
3739
);
3840
});

0 commit comments

Comments
 (0)