fix(ns-asyncapi-2): fix bug regarding refracting of Operation.message (#1239)

Refs #427

Author: char0n

packages/apidom-ns-asyncapi-2/src/refractor/visitors/async-api-2/operation/MessageVisitor.ts

@@ -13,7 +13,8 @@ const MessageVisitor = stampit(SpecificationVisitor, FallbackVisitor, {
         this.element = this.toRefractedElement(['document', 'objects', 'Reference'], objectElement);
         this.element.setMetaProperty('referenced-element', 'message');
       } else if (isArrayElement(objectElement.get('oneOf'))) {
-        this.element = new OperationMessageElement();
+        this.element = new ObjectElement();
+        const operationMessageElement = new OperationMessageElement();
 
         objectElement.get('oneOf').forEach((item: ObjectElement) => {
           let element;
@@ -24,8 +25,10 @@ const MessageVisitor = stampit(SpecificationVisitor, FallbackVisitor, {
             element = this.toRefractedElement(['document', 'objects', 'Message'], item);
           }
 
-          this.element.push(element);
+          operationMessageElement.push(element);
         });
+
+        this.element.set('oneOf', operationMessageElement);
       } else {
         this.element = this.toRefractedElement(['document', 'objects', 'Message'], objectElement);
       }

packages/apidom-ns-asyncapi-2/test/refractor/elements/Operation/__snapshots__/index.ts.snap

@@ -21,12 +21,15 @@ exports[`refractor elements OperationElement given message field of \`oneOf\` sh
 (OperationElement
   (MemberElement
     (StringElement)
-    (ArrayElement
-      (MessageElement)
-      (ReferenceElement
-        (MemberElement
-          (StringElement)
-          (StringElement))))))
+    (ObjectElement
+      (MemberElement
+        (StringElement)
+        (ArrayElement
+          (MessageElement)
+          (ReferenceElement
+            (MemberElement
+              (StringElement)
+              (StringElement))))))))
 `;
 
 exports[`refractor elements OperationElement given message field of type MessageElement should refract to semantic ApiDOM tree 1`] = `

packages/apidom-ns-asyncapi-2/test/refractor/plugins/replace-empty-element/__snapshots__/sequences.ts.snap

@@ -73,9 +73,12 @@ exports[`given multiple empty value instead of MessageElement should replace emp
             (OperationElement
               (MemberElement
                 (StringElement)
-                (ArrayElement
-                  (MessageElement)
-                  (MessageElement))))))))))
+                (ObjectElement
+                  (MemberElement
+                    (StringElement)
+                    (ArrayElement
+                      (MessageElement)
+                      (MessageElement))))))))))))
 `;
 
 exports[`given multiple empty value instead of TagElement should replace empty values with semantic elements 1`] = `

packages/apidom-reference/src/dereference/strategies/asyncapi-2/visitor.ts

@@ -229,7 +229,7 @@ const AsyncApi2DereferenceVisitor = stampit({
         referencedElement.meta.clone(),
         referencedElement.attributes.clone(),
       );
-      // existing keywords from referencing ChannelItemElement overrides ones from referenced schema
+      // existing keywords from referencing ChannelItemElement overrides ones from referenced ChannelItemElement
       channelItemElement.forEach((value: Element, key: Element, item: Element) => {
         mergedResult.remove(key.toValue());
         mergedResult.content.push(item);