fix(core): fix loadApp lost appInfo information after the second merge (#575)

* fix lost appInfo information after the second merge in loadApp

* tests(core): update tests for loadApp

Author: danpeen

packages/core/__tests__/loadApp.spec.ts

@@ -174,11 +174,8 @@ describe('Core: load process', () => {
   });
 
   it('config provided by options in loadApp will have the highest priority after config merged', async () => {
-    const mockBeforeLoad = jest.fn(() => {
-      GarfishInstance.appInfos['vue-app'] = {
-        name: 'vue-app',
-        entry: vue3SubAppEntry,
-      };
+    const mockBeforeLoad = jest.fn((appInfo) => {
+      appInfo.entry = vue3SubAppEntry;
     });
 
     GarfishInstance.run({
@@ -195,10 +192,16 @@ describe('Core: load process', () => {
     await expect(
       GarfishInstance.loadApp('vue-app', {
         domGetter: '#container',
+        props: {
+          aaa: '123'
+        }
       }),
     ).resolves.toMatchObject({
       appInfo: {
         entry: vue3SubAppEntry,
+        props: {
+          aaa: '123'
+        }
       },
     });
 

packages/core/src/garfish.ts

@@ -173,7 +173,7 @@ export class Garfish extends EventEmitter2 {
       }
 
       //merge configs again after beforeLoad for the reason of app may be re-registered during beforeLoad resulting in an incorrect information
-      appInfo = generateAppOptions(appName, this, options);
+      appInfo = generateAppOptions(appName, this, appInfo);
 
       assert(
         appInfo.entry,