fix: verify isEggMultiInstancePrototype before proto exists (#164)

MultiInstanceProto also has proto.

<!--
Thank you for your pull request. Please review below requirements.
Bug fixes and new features should include tests and possibly benchmarks.
Contributors guide:
https://github.com/eggjs/egg/blob/master/CONTRIBUTING.md

感谢您贡献代码。请确认下列 checklist 的完成情况。
Bug 修复和新功能必须包含测试，必要时请附上性能测试。
Contributors guide:
https://github.com/eggjs/egg/blob/master/CONTRIBUTING.md
-->

##### Checklist
<!-- Remove items that do not apply. For completed items, change [ ] to
[x]. -->

- [x] `npm test` passes
- [x] tests and/or benchmarks are included
- [ ] documentation is changed or added
- [x] commit message follows commit guidelines

##### Affected core subsystem(s)
<!-- Provide affected core subsystem(s). -->


##### Description of change
<!-- Provide a description of the change below this comment. -->

<!--
- any feature?
- close https://github.com/eggjs/egg/ISSUE_URL
-->

Author: killagu

core/runtime/src/factory/EggContainerFactory.ts

@@ -54,12 +54,12 @@ export class EggContainerFactory {
    */
   static async getOrCreateEggObjectFromClazz(clazz: EggProtoImplClass, name?: EggObjectName, qualifiers?: QualifierInfo[]): Promise<EggObject> {
     let proto = PrototypeUtil.getClazzProto(clazz as EggProtoImplClass) as EggPrototype | undefined;
-    if (proto) {
-      name = name ?? proto.name;
-    } else if (PrototypeUtil.isEggMultiInstancePrototype(clazz as EggProtoImplClass)) {
+    if (PrototypeUtil.isEggMultiInstancePrototype(clazz as EggProtoImplClass)) {
       const defaultName = NameUtil.getClassName(clazz as EggProtoImplClass);
       name = name ?? defaultName;
       proto = EggPrototypeFactory.instance.getPrototype(name, undefined, qualifiers);
+    } else if (proto) {
+      name = name ?? proto.name;
     }
     if (!proto) {
       throw new Error(`can not get proto for clazz ${clazz.name}`);

core/runtime/test/LoadUnitInstance.test.ts

@@ -87,11 +87,16 @@ describe('test/LoadUnit/LoadUnitInstance.test.ts', () => {
       assert(foo2.loadUnitPath);
       assert(foo2.foo === 'foo2');
 
-      const obj = await EggContainerFactory.getOrCreateEggObjectFromClazz(FooLogger, 'foo', [{
+      const obj1 = await EggContainerFactory.getOrCreateEggObjectFromClazz(FooLogger, 'foo', [{
+        attribute: FOO_ATTRIBUTE,
+        value: 'foo1',
+      }]);
+      const obj2 = await EggContainerFactory.getOrCreateEggObjectFromClazz(FooLogger, 'foo', [{
         attribute: FOO_ATTRIBUTE,
         value: 'foo2',
       }]);
-      assert(foo2Obj === obj);
+      assert(foo1Obj === obj1);
+      assert(foo2Obj === obj2);
 
       await TestUtil.destroyLoadUnitInstance(instance);
     });

plugin/tegg/app/extend/application.ts

@@ -88,7 +88,10 @@ export default {
     };
   },
 
-  async getEggObject(clazz: EggProtoImplClass, name?: string, qualifiers?: QualifierInfo[]) {
+  async getEggObject(clazz: EggProtoImplClass, name?: string, qualifiers?: QualifierInfo | QualifierInfo[]) {
+    if (qualifiers) {
+      qualifiers = Array.isArray(qualifiers) ? qualifiers : [ qualifiers ];
+    }
     const eggObject = await EggContainerFactory.getOrCreateEggObjectFromClazz(clazz, name, qualifiers);
     return eggObject.obj;
   },