Fixed tests.

jitesoft · Jun 5, 2020 · 7bf0dd8 · 7bf0dd8
1 parent bea1bbe
commit 7bf0dd8
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 10 deletions.
diff --git a/src/node/index.js b/src/node/index.js
@@ -42,7 +42,7 @@ export default class JsonPlugin extends YologPlugin {
     };
 
     if (error && !['warning', 'debug', 'info'].includes(tag)) {
-      result[error] = {
+      result.error = {
         message: error.message,
         stack: error.stack
       };

diff --git a/src/web/index.js b/src/web/index.js
@@ -29,7 +29,7 @@ export default class JsonPlugin extends YologPlugin {
     };
 
     if (error && !['warning', 'debug', 'info'].includes(tag)) {
-      result[error] = {
+      result.error = {
         message: error.message,
         stack: error.stack
       };

diff --git a/tests/Node.test.js b/tests/Node.test.js
@@ -29,17 +29,17 @@ describe('Test node plugin.', () => {
 
     test('Test that Debug is printed to standard output.', async () => {
       await plugin.log('debug', 123, 'message');
-      expect(stdout).toHaveBeenCalledWith(sprintf('%j', { tag: 'debug', timestamp: 123, message: 'message' }), 'UTF-8', expect.any(Function));
+      expect(stdout).toHaveBeenCalledWith(sprintf('%j\n', { tag: 'debug', timestamp: 123, message: 'message' }), 'UTF-8', expect.any(Function));
     });
 
     test('Test that Info is printed to standard output.', async () => {
       await plugin.log('info', 123, 'message');
-      expect(stdout).toHaveBeenCalledWith(sprintf('%j', { tag: 'info', timestamp: 123, message: 'message' }), 'UTF-8', expect.any(Function));
+      expect(stdout).toHaveBeenCalledWith(sprintf('%j\n', { tag: 'info', timestamp: 123, message: 'message' }), 'UTF-8', expect.any(Function));
     });
 
     test('Test that Warning is printed standard output.', async () => {
       await plugin.log('warning', 123, 'message');
-      expect(stdout).toHaveBeenCalledWith(sprintf('%j', { tag: 'warning', timestamp: 123, message: 'message' }), 'UTF-8', expect.any(Function));
+      expect(stdout).toHaveBeenCalledWith(sprintf('%j\n', { tag: 'warning', timestamp: 123, message: 'message' }), 'UTF-8', expect.any(Function));
     });
 
     test('Test that error types are printed to error output.', async () => {
@@ -50,10 +50,10 @@ describe('Test node plugin.', () => {
       await plugin.log('emergency', 123456, 'message4', err);
 
       expect(stderr).toHaveBeenCalledTimes(4);
-      expect(stderr).toHaveBeenNthCalledWith(1, sprintf('%j', { tag: 'error', timestamp: 123, message: 'message1' }), 'UTF-8', expect.any(Function));
-      expect(stderr).toHaveBeenNthCalledWith(2, sprintf('%j', { tag: 'critical', timestamp: 1234, message: 'message2' }), 'UTF-8', expect.any(Function));
-      expect(stderr).toHaveBeenNthCalledWith(3, sprintf('%j', { tag: 'alert', timestamp: 12345, message: 'message3' }), 'UTF-8', expect.any(Function));
-      expect(stderr).toHaveBeenNthCalledWith(4, sprintf('%j', { tag: 'emergency', timestamp: 123456, message: 'message4', error: { message: err.message, stack: err.stack } }), 'UTF-8', expect.any(Function));
+      expect(stderr).toHaveBeenNthCalledWith(1, sprintf('%j\n', { tag: 'error', timestamp: 123, message: 'message1' }), 'UTF-8', expect.any(Function));
+      expect(stderr).toHaveBeenNthCalledWith(2, sprintf('%j\n', { tag: 'critical', timestamp: 1234, message: 'message2' }), 'UTF-8', expect.any(Function));
+      expect(stderr).toHaveBeenNthCalledWith(3, sprintf('%j\n', { tag: 'alert', timestamp: 12345, message: 'message3' }), 'UTF-8', expect.any(Function));
+      expect(stderr).toHaveBeenNthCalledWith(4, sprintf('%j\n', { tag: 'emergency', timestamp: 123456, message: 'message4', error: { message: err.message, stack: err.stack } }), 'UTF-8', expect.any(Function));
     });
   });
 
@@ -88,7 +88,7 @@ describe('Test node plugin.', () => {
       expect(fs.appendFile).toHaveBeenCalledTimes(7);
       for (let i = 0; i < tags.length; i++) {
         const tag = tags[i];
-        expect(fs.appendFile).toHaveBeenNthCalledWith(i + 1, 'file/path/test.txt', sprintf('%j\n', { tag: tag, timestamp: i + 10, message: `This is a ${tag} message!` }), expect.any(Function));
+        expect(fs.appendFile).toHaveBeenNthCalledWith(i + 1, expect.stringContaining('file/path/test.txt'), sprintf('%j\n', { tag: tag, timestamp: i + 10, message: `This is a ${tag} message!` }), expect.any(Function));
       }
     });
   });