Merge branch 'j94/invalid-log-levels' of...

Merge branch 'j94/invalid-log-levels' of https://github.com/jaller94/matrix-appservice-discord into develop

src/log.ts

@@ -46,6 +46,10 @@ export class Log {
     private static config: DiscordBridgeConfigLogging;
     private static logger: Logger;
 
+    private static isValidLevel(level: string) {
+        return ["silly", "verbose", "info", "http", "warn", "error", "silent"].includes(level);
+    }
+
     private static setupLogger() {
         if (Log.logger) {
             Log.logger.close();
@@ -53,6 +57,9 @@ export class Log {
         const tsports: transports.StreamTransportInstance[] = Log.config.files.map((file) =>
             Log.setupFileTransport(file),
         );
+        if (Log.config.console && !Log.isValidLevel(Log.config.console)) {
+            new Log("Log").warn("Console log level is invalid. Please pick one of the case-sensitive levels provided in the sample config.");
+        }
         tsports.push(new transports.Console({
             level: Log.config.console,
         }));
@@ -80,6 +87,10 @@ export class Log {
             return info;
         });
 
+        if (config.level && !Log.isValidLevel(config.level)) {
+            new Log("Log").warn(`Log level of ${config.file} is invalid. Please pick one of the case-sensitive levels provided in the sample config.`);
+        }
+
         const opts = {
             datePattern: config.datePattern,
             filename: config.file,

test/test_log.ts

@@ -71,6 +71,24 @@ describe("Log", () => {
             expect(Log.config.files).to.not.be.empty;
             expect(Log.config.files[0].file).to.equal("./logfile.log");
         });
+        it("should warn if log level got misspelled", () => {
+            Log.Configure({
+                console: "WARNING",
+                lineDateFormat: "HH:mm:ss",
+            });
+            expect(loggedMessages).to.contain("Console log level is invalid. Please pick one of the case-sensitive levels provided in the sample config.");
+        });
+        it("should warn if log level for a file got misspelled", () => {
+            Log.Configure({
+                files: [
+                    {
+                        file: "./logfile.log",
+                        level: "WARNING",
+                    },
+                ],
+            });
+            expect(loggedMessages).to.contain("Log level of ./logfile.log is invalid. Please pick one of the case-sensitive levels provided in the sample config.");
+        });
     });
     describe("ForceSilent", () => {
         it("should be silent", () => {