feat: use default logger in DenoBridge (#89)

* feat: use default logger in DenoBridge

* refactor: use `getLogger` instead of `getDefaultLogger`

src/bridge.ts

@@ -7,7 +7,7 @@ import semver from 'semver'
 
 import { download } from './downloader.js'
 import { getPathInHome } from './home_path.js'
-import { Logger } from './logger.js'
+import { getLogger, Logger } from './logger.js'
 import { getBinaryExtension } from './platform.js'
 
 const DENO_VERSION_FILE = 'version.txt'
@@ -20,7 +20,7 @@ interface DenoOptions {
   cacheDirectory?: string
   debug?: boolean
   denoDir?: string
-  logger: Logger
+  logger?: Logger
   onAfterDownload?: OnAfterDownloadHook
   onBeforeDownload?: OnBeforeDownloadHook
   useGlobal?: boolean
@@ -50,7 +50,7 @@ class DenoBridge {
     this.cacheDirectory = options.cacheDirectory ?? getPathInHome('deno-cli')
     this.debug = options.debug ?? false
     this.denoDir = options.denoDir
-    this.logger = options.logger
+    this.logger = options.logger ?? getLogger(undefined, options.debug)
     this.onAfterDownload = options.onAfterDownload
     this.onBeforeDownload = options.onBeforeDownload
     this.useGlobal = options.useGlobal ?? true

test/main.ts

@@ -10,12 +10,8 @@ import { spy } from 'sinon'
 import tmp from 'tmp-promise'
 
 import { DenoBridge } from '../src/bridge.js'
-import { getLogger } from '../src/logger.js'
 import { getPlatformTarget } from '../src/platform.js'
 
-const testLogger = getLogger(() => {
-  // no-op
-})
 const require = createRequire(import.meta.url)
 const archiver = require('archiver')
 
@@ -40,7 +36,6 @@ test('Downloads the Deno CLI on demand and caches it for subsequent calls', asyn
   const afterDownload = spy()
   const deno = new DenoBridge({
     cacheDirectory: tmpDir.path,
-    logger: testLogger,
     onBeforeDownload: beforeDownload,
     onAfterDownload: afterDownload,
     useGlobal: false,