chapter 03: using the fakeManagerExtension to fix the failed test from the previous commit

Author: devcorpio

chapter_02-a-first-unit-test/LogAn/logAnalyzer.test.js

@@ -35,7 +35,6 @@ describe('isValidLogFileName', () => {
     `(
         'when called there changes wasLastFileNameValid that returns $expected',
         ({ fileName, expected }) => {
-            console.log(fileName);
             logAnalyzerInstance.isValidLogFileName(fileName);
             const result = logAnalyzerInstance.getWasLastFileNameValid();
 

chapter_03-using-stubs-to-break-dependencies/LogAn/logAnalyzer.test.js

@@ -1,8 +1,10 @@
-const logAnalyzer = require('./logAnalyzer');
+const logAnalyzerFactory = require('./logAnalyzer');
+const fakeExtensionManagerFactory = require('./fakeExtensionManager');
+
+let myFakeExtensionManager;
 
-let logAnalyzerInstance;
 beforeEach(() => {
-    logAnalyzerInstance = logAnalyzer();
+    myFakeExtensionManager = fakeExtensionManagerFactory();
 });
 
 describe.each([
@@ -11,15 +13,20 @@ describe.each([
     ['johndoe.SLF', true],
 ])('isValidLogFileName("%s"))', (fileName, expected) => {
     it(`bad extension returns ${expected}`, async () => {
-        const result = await logAnalyzerInstance.isValidLogFileName(fileName);
+        myFakeExtensionManager.willBeValid(expected);
+
+        const logAnalyzer = logAnalyzerFactory(myFakeExtensionManager);
+        const result = await logAnalyzer.isValidLogFileName(fileName);
         expect(result).toBe(expected);
     });
 });
 
 describe('isValidLogFileName', () => {
     it('empty filename throws error', async () => {
         async function emptyLogFileName() {
-            return logAnalyzerInstance.isValidLogFileName('');
+            const logAnalyzer = logAnalyzerFactory(myFakeExtensionManager);
+
+            return logAnalyzer.isValidLogFileName('');
         }
 
         await expect(emptyLogFileName()).rejects.toThrow(
@@ -37,8 +44,11 @@ describe('isValidLogFileName', () => {
     `(
         'when called there changes wasLastFileNameValid that returns $expected',
         async ({ fileName, expected }) => {
-            await logAnalyzerInstance.isValidLogFileName(fileName);
-            const result = logAnalyzerInstance.getWasLastFileNameValid();
+            myFakeExtensionManager.willBeValid(expected);
+
+            const logAnalyzer = logAnalyzerFactory(myFakeExtensionManager);
+            await logAnalyzer.isValidLogFileName(fileName);
+            const result = logAnalyzer.getWasLastFileNameValid();
 
             expect(result).toBe(expected);
         }