From 34ef7407655502133bcbea0731d203ae5f970a0d Mon Sep 17 00:00:00 2001
From: Florian Schopp <florian.schopp@hbkworld.com>
Date: Thu, 2 Nov 2023 14:23:14 +0100
Subject: [PATCH] Deleted filelogger

---
 src/3_jet/log.ts | 32 ----------------------
 test/log.test.ts | 70 +-----------------------------------------------
 2 files changed, 1 insertion(+), 101 deletions(-)

diff --git a/src/3_jet/log.ts b/src/3_jet/log.ts
index bee69dd..fafd41c 100644
--- a/src/3_jet/log.ts
+++ b/src/3_jet/log.ts
@@ -1,4 +1,3 @@
-import * as fs from 'fs'
 export enum LogLevel {
   'socket' = 1,
   'debug',
@@ -22,7 +21,6 @@ export class Logger {
   logName: string
   logLevel: LogLevel
   callBacks: LogFunction[] | undefined
-  stream: fs.WriteStream | undefined
   /**
    * Constructor to create a new Logger instance
    * @param settings
@@ -31,9 +29,6 @@ export class Logger {
     this.logName = settings.logName
     this.logLevel = settings.logLevel || LogLevel['none']
     this.callBacks = settings.logCallbacks
-    if (settings.logFile) {
-      this.stream = fs.createWriteStream(settings.logFile)
-    }
   }
   /**
    * Function that transforms a message into a string of the format "<Date> <Time> <LogName> <LogLevel> <Message>"
@@ -58,10 +53,6 @@ export class Logger {
       return
     }
     const logMessage = this.stringBuilder(msg, level)
-    if (this.stream) {
-      this.stream.write(logMessage)
-      this.stream.write('\n')
-    }
     if (this.callBacks) {
       this.callBacks.every((cb) => cb(logMessage))
     }
@@ -101,27 +92,4 @@ export class Logger {
   error(msg: string) {
     this.log(msg, LogLevel.error)
   }
-  /**
-   * Function that can be called to wait until the stream has completed to write everything
-   */
-  flush(): Promise<void> {
-    return new Promise((resolve) => {
-      if (this.stream) {
-        const interval = setInterval(() => {
-          if (!this.stream || !this.stream.pending) {
-            clearInterval(interval)
-            resolve()
-          }
-        })
-      }
-    })
-  }
-  /**
-   * Function to close the Logger and the fileStream in case any fileStream was used
-   */
-  close() {
-    if (this.stream) {
-      this.stream.end()
-    }
-  }
 }
diff --git a/test/log.test.ts b/test/log.test.ts
index 7155d8f..f2519de 100644
--- a/test/log.test.ts
+++ b/test/log.test.ts
@@ -1,5 +1,5 @@
 import { Logger, LogLevel } from '../src/3_jet/log'
-import { readFileSync, mkdirSync, rmSync, existsSync } from 'fs'
+import { mkdirSync, rmSync, existsSync } from 'fs'
 const tmpDir = '.cache/tmp'
 describe('Testing Logging', () => {
   beforeAll(() => {
@@ -27,7 +27,6 @@ describe('Testing Logging', () => {
     logger.info('Bar')
     logger.warn('Bar')
     logger.error('Bar')
-    return logger.flush().then(() => logger.close())
   }
   describe('Should test callbacks', () => {
     it('Should create empty logger', () => {
@@ -85,71 +84,4 @@ describe('Testing Logging', () => {
       expect(logSpy).toHaveBeenCalledTimes(2)
     })
   })
-  describe('Should test file', () => {
-    it('Should create sock logger', async () => {
-      const logFile = `${tmpDir}/sock.txt`
-      const logger = new Logger({
-        logLevel: LogLevel.socket,
-        logName: 'Foo',
-        logFile
-      })
-      await log(logger).then(() => {
-        const content = readFileSync(logFile).toString().split('\n')
-        expect(content).toHaveLength(12)
-        expect(content[0]).toContain('Foo	socket	Bar')
-      })
-    })
-    it('Should create debug logger', async () => {
-      const logFile = `${tmpDir}/debug.txt`
-      const logger = new Logger({
-        logLevel: LogLevel.debug,
-        logName: 'Foo',
-        logFile
-      })
-      await log(logger).then(() => {
-        const content = readFileSync(logFile).toString().split('\n')
-        expect(content).toHaveLength(10)
-        expect(content[0]).toContain('Foo	debug	Bar')
-      })
-    })
-    it('Should create info logger', async () => {
-      const logFile = `${tmpDir}/info.txt`
-      const logger = new Logger({
-        logLevel: LogLevel.info,
-        logName: 'Foo',
-        logFile
-      })
-      await log(logger).then(() => {
-        const content = readFileSync(logFile).toString().split('\n')
-        expect(content).toHaveLength(7)
-        expect(content[0]).toContain('Foo	info	Bar')
-      })
-    })
-    it('Should create warn logger', async () => {
-      const logFile = `${tmpDir}/warn.txt'`
-      const logger = new Logger({
-        logLevel: LogLevel.warn,
-        logName: 'Foo',
-        logFile
-      })
-      await log(logger).then(() => {
-        const content = readFileSync(logFile).toString().split('\n')
-        expect(content).toHaveLength(5)
-        expect(content[0]).toContain('Foo	warn	Bar')
-      })
-    })
-    it('Should create error logger', async () => {
-      const logFile = `${tmpDir}/error.txt'`
-      const logger = new Logger({
-        logLevel: LogLevel.error,
-        logName: 'Foo',
-        logFile
-      })
-      await log(logger).then(() => {
-        const content = readFileSync(logFile).toString().split('\n')
-        expect(content).toHaveLength(3)
-        expect(content[0]).toContain('Foo	error	Bar')
-      })
-    })
-  })
 })