test(core-logger-pino): increase coverage to 100% (#3580)

ArkEcosystem · Mar 5, 2020 · c1c7add · c1c7add
1 parent e428af2
commit c1c7add
Show file tree

Hide file tree

Showing 4 changed files with 45 additions and 3 deletions.
diff --git a/__tests__/unit/core-logger-pino/service-provider.test.ts b/__tests__/unit/core-logger-pino/service-provider.test.ts
@@ -0,0 +1,40 @@
+import "jest-extended";
+
+import { Providers } from "@arkecosystem/core-kernel";
+import { Application, Container, Services } from "@packages/core-kernel/src";
+import { ServiceProvider } from "@packages/core-logger-pino/src";
+import { defaults } from "@packages/core-logger-pino/src/defaults";
+import { dirSync } from "tmp";
+
+let app: Application;
+
+beforeEach(() => {
+    app = new Application(new Container.Container());
+});
+
+describe("ServiceProvider", () => {
+    let serviceProvider: ServiceProvider;
+
+    beforeEach(() => {
+        serviceProvider = app.resolve<ServiceProvider>(ServiceProvider);
+    });
+
+    it("should register", async () => {
+        app.bind<Services.Log.LogManager>(Container.Identifiers.LogManager)
+            .to(Services.Log.LogManager)
+            .inSingletonScope();
+
+        await app.get<Services.Log.LogManager>(Container.Identifiers.LogManager).boot();
+
+        serviceProvider.setConfig(app.resolve(Providers.PluginConfiguration).merge(defaults));
+
+        app.bind(Container.Identifiers.ApplicationNamespace).toConstantValue("token-network");
+        app.bind("path.log").toConstantValue(dirSync().name);
+
+        await expect(serviceProvider.register()).toResolve();
+    });
+
+    it("should be disposable", async () => {
+        await expect(serviceProvider.dispose()).toResolve();
+    });
+});
diff --git a/__tests__/unit/core-transactions/service-provider.test.ts b/__tests__/unit/core-transactions/service-provider.test.ts
@@ -1,7 +1,7 @@
 import "jest-extended";
 
-import { Application, Container } from "@arkecosystem/core-kernel";
-import { ServiceProvider } from "@arkecosystem/core-transactions/src";
+import { Application, Container } from "@packages/core-kernel/src";
+import { ServiceProvider } from "@packages/core-transactions/src/service-provider";
 
 let app: Application;
 

diff --git a/jest.config.js b/jest.config.js
@@ -14,7 +14,7 @@ module.exports = {
     collectCoverage: false,
     coverageDirectory: "<rootDir>/.coverage",
     collectCoverageFrom: [
-        "packages/**/src/**/{!(index|manager),}.ts",
+        "packages/**/src/**/{!(index|manager|defaults),}.ts",
         "!packages/**/src/**/contracts/**",
         "!packages/**/src/**/enums/**",
         "!packages/**/src/**/exceptions/**",

diff --git a/packages/core-logger-pino/src/driver.ts b/packages/core-logger-pino/src/driver.ts
@@ -242,6 +242,7 @@ export class PinoLogger implements Contracts.Kernel.Logger {
                     }
                 } catch {}
 
+                /* istanbul ignore next */
                 return cb();
             },
         });
@@ -261,6 +262,7 @@ export class PinoLogger implements Contracts.Kernel.Logger {
                 }
 
                 if (typeof time === "number") {
+                    /* istanbul ignore next */
                     time = new Date(time);
                 }