fix: deep merge config (#274)

* fix: deep merge config

* chore: format code

Author: JerrysShan

src/application.ts

@@ -4,6 +4,7 @@ import { ArtusInjectEnum } from './constant';
 import { ArtusStdError } from './exception';
 import { HookFunction, LifecycleManager } from './lifecycle';
 import { LoaderFactory, Manifest } from './loader';
+import { mergeConfig } from './loader/utils/merge';
 import { Application, ApplicationInitOptions } from './types';
 import ConfigurationHandler from './configuration';
 import { Logger, LoggerType } from './logger';
@@ -114,7 +115,7 @@ export class ArtusApplication implements Application {
     const newConfig = this.configurationHandler.getMergedConfig() ?? {};
     this.container.set({
       id: ArtusInjectEnum.Config,
-      value: Object.assign(oldConfig, newConfig),
+      value: mergeConfig(oldConfig, newConfig),
     });
   }
 }

test/config.test.ts

@@ -16,4 +16,13 @@ describe("test/config.test.ts", () => {
       process.env[ARTUS_SERVER_ENV] = undefined;
     });
   });
+
+
+  describe("app with manifest should load config ok", () => {
+    it("should load config ok", async () => {
+      const { main } = require("./fixtures/app_with_manifest/bootstrap");
+      const app = await main();
+      expect(app.config).toEqual({ httpConfig: { key1: 'value1', port: 3000 }, plugin: {} });
+    });
+  });
 });

test/fixtures/app_with_manifest/bootstrap.ts

@@ -0,0 +1,29 @@
+import path from "path";
+// import fs from 'fs';
+import 'reflect-metadata';
+import { ArtusApplication } from "../../../src";
+
+const rootDir = path.resolve(__dirname, "./");
+
+async function main() {
+  const app = new ArtusApplication();
+  const metaFilePath = path.resolve(rootDir, 'manifest.json');
+  //   let manifest;
+  // if (fs.existsSync(metaFilePath)) {
+  const manifest = require((metaFilePath));
+  // } else {
+  // const scanner = new ArtusScanner({
+  //     configDir: 'config',
+  //     needWriteFile: true,
+  //     useRelativePath: true,
+  //     extensions: ['.ts', '.js', '.json'],
+  //     app,
+  // });
+  // manifest = await scanner.scan(rootDir);
+  // }
+  await app.load(manifest, rootDir);
+  await app.run();
+  return app;
+
+}
+export { main };

test/fixtures/app_with_manifest/config/config.default.ts

@@ -0,0 +1,6 @@
+
+export default {
+  httpConfig: {
+    port: 3000,
+  },
+};

test/fixtures/app_with_manifest/controller/home.ts

@@ -0,0 +1,8 @@
+import { Injectable } from '../../../../src/';
+
+@Injectable()
+export default class DemoController {
+  public async index() {
+    return { code: 0, message: 'ok', data: {} };
+  }
+}

test/fixtures/app_with_manifest/lifecycle.ts

@@ -0,0 +1,13 @@
+import { LifecycleHookUnit, ApplicationLifecycle, LifecycleHook, Inject, ArtusApplication, ArtusInjectEnum } from '../../../src';
+
+@LifecycleHookUnit()
+export default class MyLifecycle implements ApplicationLifecycle {
+  @Inject(ArtusInjectEnum.Application)
+  private app: ArtusApplication;
+
+  @LifecycleHook()
+  public async configWillLoad() {
+    this.app.config.httpConfig = this.app.config.httpConfig ?? {};
+    this.app.config.httpConfig.key1 = 'value1';
+  }
+}

test/fixtures/app_with_manifest/manifest.json

@@ -0,0 +1,49 @@
+{
+  "version": "2",
+  "refMap": {
+    "_app": {
+      "relativedPath": "",
+      "pluginConfig": {},
+      "items": [
+        {
+          "path": "config/config.default",
+          "extname": ".ts",
+          "filename": "config.default.ts",
+          "loader": "config",
+          "source": "app",
+          "unitName": "_app",
+          "loaderState": {
+            "exportNames": []
+          }
+        },
+        {
+          "path": "controller/home",
+          "extname": ".ts",
+          "filename": "home.ts",
+          "loader": "module",
+          "source": "app",
+          "unitName": "_app",
+          "loaderState": {
+            "exportNames": [
+              "default"
+            ]
+          }
+        },
+        {
+          "path": "lifecycle",
+          "extname": ".ts",
+          "filename": "lifecycle.ts",
+          "loader": "lifecycle-hook-unit",
+          "source": "app",
+          "unitName": "_app",
+          "loaderState": {
+            "exportNames": [
+              "default"
+            ]
+          }
+        }
+      ]
+    }
+  },
+  "extraPluginConfig": {}
+}