(#1513) Update to test descriptions

storybookjs · Nov 25, 2017 · fa16aa8 · fa16aa8
1 parent 283f6dd
commit fa16aa8
Show file tree

Hide file tree

Showing 2 changed files with 30 additions and 34 deletions.
diff --git a/lib/client-logger/src/index.test.js b/lib/client-logger/src/index.test.js
@@ -1,6 +1,6 @@
 import { logger } from './.';
 
-describe('loggers', () => {
+describe('client-logger', () => {
   const initialConsole = { ...global.console };
   beforeEach(() => {
     global.console.log = jest.fn();
@@ -10,21 +10,19 @@ describe('loggers', () => {
   afterAll(() => {
     global.console = initialConsole;
   });
-  describe('browser-logger', () => {
-    it('should have an info method with blue `Info`', () => {
-      const message = 'information';
-      logger.info(message);
-      expect(global.console.log).toHaveBeenCalledWith(message);
-    });
-    it('should have a warning method with yellow `Warn`', () => {
-      const message = 'warning message';
-      logger.warn(message);
-      expect(global.console.warn).toHaveBeenCalledWith(message);
-    });
-    it('should have an error method with red `Error`', () => {
-      const message = 'error message';
-      logger.error(message);
-      expect(global.console.error).toHaveBeenCalledWith(message);
-    });
+  it('should have an info method that displays the message', () => {
+    const message = 'information';
+    logger.info(message);
+    expect(global.console.log).toHaveBeenCalledWith(message);
+  });
+  it('should have a warning method that displays the message in yellow, with a trace', () => {
+    const message = 'warning message';
+    logger.warn(message);
+    expect(global.console.warn).toHaveBeenCalledWith(message);
+  });
+  it('should have an error method that displays the message in red, with a trace', () => {
+    const message = 'error message';
+    logger.error(message);
+    expect(global.console.error).toHaveBeenCalledWith(message);
   });
 });
diff --git a/lib/node-logger/src/index.test.js b/lib/node-logger/src/index.test.js
@@ -7,27 +7,25 @@ jest.mock('npmlog', () => ({
   error: jest.fn(),
 }));
 
-describe('loggers', () => {
+describe('node-logger', () => {
   beforeEach(() => {
     npmLog.info.mockReset();
     npmLog.warn.mockReset();
     npmLog.error.mockReset();
   });
-  describe('node-logger', () => {
-    it('should have an info method', () => {
-      const message = 'information';
-      logger.info(message);
-      expect(npmLog.info).toHaveBeenCalledWith('', message);
-    });
-    it('should have a warn method', () => {
-      const message = 'warning message';
-      logger.warn(message);
-      expect(npmLog.warn).toHaveBeenCalledWith('', message);
-    });
-    it('should have an error method', () => {
-      const message = 'error message';
-      logger.error(message);
-      expect(npmLog.error).toHaveBeenCalledWith('', message);
-    });
+  it('should have an info method', () => {
+    const message = 'information';
+    logger.info(message);
+    expect(npmLog.info).toHaveBeenCalledWith('', message);
+  });
+  it('should have a warn method', () => {
+    const message = 'warning message';
+    logger.warn(message);
+    expect(npmLog.warn).toHaveBeenCalledWith('', message);
+  });
+  it('should have an error method', () => {
+    const message = 'error message';
+    logger.error(message);
+    expect(npmLog.error).toHaveBeenCalledWith('', message);
   });
 });