Fixed broken tests

kristus123 · Mar 7, 2019 · 7020bf8 · 7020bf8
1 parent 233f04a
commit 7020bf8
Show file tree

Hide file tree

Showing 4 changed files with 46 additions and 5 deletions.
diff --git a/src/common/logger/log-message-type.ts b/src/common/logger/log-message-type.ts
@@ -0,0 +1,4 @@
+export enum LogMessageType {
+    Error,
+    Debug,
+}
diff --git a/src/common/logger/log-message.ts b/src/common/logger/log-message.ts
@@ -0,0 +1,6 @@
+import { LogMessageType } from "./log-message-type";
+
+export interface LogMessage {
+    message: string;
+    type: LogMessageType;
+}
diff --git a/src/tests/search-engine.test.ts b/src/tests/search-engine.test.ts
@@ -7,8 +7,10 @@ import { FakeSearchPlugin } from "./fake-search-plugin";
 import { SearchResultItem } from "../common/search-result-item";
 import { PluginType } from "../main/plugin-type";
 import { dummyIcon } from "./dummy-icon";
+import { TestLogger } from "./test-logger";
 
 describe(SearchEngine.name, () => {
+    const logger = new TestLogger();
     it("should find search results when searching for the exact name", (done) => {
         const items: SearchResultItem[] = [
             {
@@ -30,7 +32,7 @@ describe(SearchEngine.name, () => {
         } as UserConfigOptions;
         const config = Object.assign({}, defaultUserConfigOptions, userConfig);
 
-        const searchEngine = new SearchEngine(searchPlugins, [], [], config, englishTranslationSet);
+        const searchEngine = new SearchEngine(searchPlugins, [], [], config, englishTranslationSet, logger);
 
         searchEngine.getSearchResults("Google Chrome")
             .then((searchResults) => {
@@ -63,7 +65,7 @@ describe(SearchEngine.name, () => {
         } as UserConfigOptions;
         const config = Object.assign({}, defaultUserConfigOptions, userConfig);
 
-        const searchEngine = new SearchEngine(searchPlugins, [], [], config, translationSet);
+        const searchEngine = new SearchEngine(searchPlugins, [], [], config, translationSet, logger);
 
         searchEngine.getSearchResults("blabla")
             .then((searchResults) => {
@@ -96,7 +98,7 @@ describe(SearchEngine.name, () => {
         } as UserConfigOptions;
         const config = Object.assign({}, defaultUserConfigOptions, userConfig);
 
-        const searchEngine = new SearchEngine(searchPlugins, [], [], config, translationSet);
+        const searchEngine = new SearchEngine(searchPlugins, [], [], config, translationSet, logger);
 
         searchEngine.getSearchResults("gOoGlE ChRoMe")
             .then((searchResults) => {
@@ -131,7 +133,7 @@ describe(SearchEngine.name, () => {
         } as UserConfigOptions;
         const config = Object.assign({}, defaultUserConfigOptions, userConfig);
 
-        const searchEngine = new SearchEngine(searchPlugins, [], [], config, translationSet);
+        const searchEngine = new SearchEngine(searchPlugins, [], [], config, translationSet, logger);
 
         searchEngine.getSearchResults("gglchrm")
             .then((searchResults) => {
@@ -166,7 +168,7 @@ describe(SearchEngine.name, () => {
         } as UserConfigOptions;
         const config = Object.assign({}, defaultUserConfigOptions, userConfig);
 
-        const searchEngine = new SearchEngine(searchPlugins, [], [], config, translationSet);
+        const searchEngine = new SearchEngine(searchPlugins, [], [], config, translationSet, logger);
 
         searchEngine.getSearchResults("gglchrm")
             .then((searchResults) => {

diff --git a/src/tests/test-logger.ts b/src/tests/test-logger.ts
@@ -0,0 +1,29 @@
+import { Logger } from "../common/logger/logger";
+import { LogMessage } from "../common/logger/log-message";
+import { LogMessageType } from "../common/logger/log-message-type";
+
+export class TestLogger implements Logger {
+    private readonly messages: LogMessage[];
+
+    constructor() {
+        this.messages = [];
+    }
+
+    public debug(message: string) {
+        this.messages.push({
+            message,
+            type: LogMessageType.Error,
+        });
+    }
+
+    public error(message: string) {
+        this.messages.push({
+            message,
+            type: LogMessageType.Debug,
+        });
+    }
+
+    public getAllMessages(): LogMessage[] {
+        return this.messages;
+    }
+}