chapter 07: refactored solution based in abstract test infrastructure…

… class pattern
devcorpio · Jan 13, 2019 · df26d6d · df26d6d
1 parent 2fc73fd
commit df26d6d
Show file tree

Hide file tree

Showing 3 changed files with 22 additions and 8 deletions.
diff --git a/...hies-and-organization/abstractTestInfrastructureClassPattern/configurationManager.test.js b/...hies-and-organization/abstractTestInfrastructureClassPattern/configurationManager.test.js
@@ -1,13 +1,12 @@
 const configurationManagerFactory = require('./configurationManager');
-const loggingFacility = require('./loggingFacility');
+const { fakeTheLogger, tearDown } = require('./testsUtils/baseTests');
 
 describe('isConfigured', () => {
     it('logging file check', () => {
+        fakeTheLogger();
         const configurationManager = configurationManagerFactory();
         configurationManager.isConfigured('');
     });
 
-    afterEach(() => {
-        loggingFacility.setLogger(null);
-    });
+    afterEach(tearDown);
 });
diff --git a/...t-hierarchies-and-organization/abstractTestInfrastructureClassPattern/logAnalyzer.test.js b/...t-hierarchies-and-organization/abstractTestInfrastructureClassPattern/logAnalyzer.test.js
@@ -1,13 +1,12 @@
 const logAnalyzerFactory = require('./logAnalyzer');
-const loggingFacility = require('./loggingFacility');
+const { fakeTheLogger, tearDown } = require('./testsUtils/baseTests');
 
 describe('analyze', () => {
     it('empty file throws exception', () => {
+        fakeTheLogger();
         const logAnalyzer = logAnalyzerFactory();
         logAnalyzer.analyze('');
     });
 
-    afterEach(() => {
-        loggingFacility.setLogger(null);
-    });
+    afterEach(tearDown);
 });
diff --git a/...erarchies-and-organization/abstractTestInfrastructureClassPattern/testsUtils/baseTests.js b/...erarchies-and-organization/abstractTestInfrastructureClassPattern/testsUtils/baseTests.js
@@ -0,0 +1,16 @@
+let loggingFacility = require('../loggingFacility');
+
+function fakeTheLogger() {
+    loggingFacility.setLogger(function fakeLogger(value) {
+        return value;
+    });
+}
+
+function tearDown() {
+    loggingFacility.setLogger(null);
+}
+
+module.exports = {
+    fakeTheLogger,
+    tearDown,
+};