fix(apisix): upstream nodes should be allowed to be null (#222)

Author: bzp2010

libs/backend-apisix/e2e/resources/consumer.e2e-spec.ts

@@ -1,5 +1,5 @@
 import * as ADCSDK from '@api7/adc-sdk';
-import { gte, lt } from 'semver';
+import { gte } from 'semver';
 
 import { BackendAPISIX } from '../../src';
 import { server, token } from '../support/constants';
@@ -96,7 +96,6 @@ describe('Consumer E2E', () => {
           backend,
         )) as ADCSDK.Configuration;
         expect(result.consumers).toHaveLength(1);
-        console.log(result.consumers[0]);
         expect(result.consumers[0].credentials).toBeUndefined();
       });
 

libs/backend-apisix/e2e/resources/upstream.e2e-spec.ts

@@ -0,0 +1,56 @@
+import * as ADCSDK from '@api7/adc-sdk';
+
+import { BackendAPISIX } from '../../src';
+import { server, token } from '../support/constants';
+import {
+  createEvent,
+  deleteEvent,
+  dumpConfiguration,
+  syncEvents,
+} from '../support/utils';
+
+describe('Upstream E2E', () => {
+  let backend: BackendAPISIX;
+
+  beforeAll(() => {
+    backend = new BackendAPISIX({
+      server,
+      token,
+      tlsSkipVerify: true,
+    });
+  });
+
+  describe('Sync and dump upstream (nodes = null)', () => {
+    const upstream = {
+      scheme: 'https',
+      discovery_type: 'kubernetes',
+      service_name: 'test',
+    } as ADCSDK.Upstream;
+    const serviceName = 'service1';
+    const service = {
+      name: serviceName,
+      upstream: structuredClone(upstream),
+    } as ADCSDK.Service;
+
+    it('Create services', async () =>
+      syncEvents(backend, [
+        createEvent(ADCSDK.ResourceType.SERVICE, serviceName, service),
+      ]));
+
+    it('Dump', async () => {
+      const result = (await dumpConfiguration(backend)) as ADCSDK.Configuration;
+      expect(result.services).toHaveLength(1);
+      expect(result.services[0]).toMatchObject(service);
+    });
+
+    it('Delete service', async () =>
+      syncEvents(backend, [
+        deleteEvent(ADCSDK.ResourceType.SERVICE, serviceName),
+      ]));
+
+    it('Dump again', async () => {
+      const result = (await dumpConfiguration(backend)) as ADCSDK.Configuration;
+      expect(result.services).toHaveLength(0);
+    });
+  });
+});

libs/backend-apisix/src/transformer.ts

@@ -179,21 +179,23 @@ export class ToADC {
       grpc: 80,
       grpcs: 443,
     };
-    const nodes = Array.isArray(upstream.nodes)
-      ? upstream.nodes
-      : Object.keys(upstream.nodes).map<ADCSDK.UpstreamNode>((node) => {
-          const hostport = node.split(':');
-          return {
-            host: hostport[0],
-            port:
-              hostport.length === 2
-                ? parseInt(hostport[1])
-                : defaultPortMap[upstream.scheme]
-                  ? defaultPortMap[upstream.scheme]
-                  : 80,
-            weight: upstream.nodes[node],
-          };
-        });
+    const nodes = upstream.nodes
+      ? Array.isArray(upstream.nodes)
+        ? upstream.nodes
+        : Object.keys(upstream.nodes).map<ADCSDK.UpstreamNode>((node) => {
+            const hostport = node.split(':');
+            return {
+              host: hostport[0],
+              port:
+                hostport.length === 2
+                  ? parseInt(hostport[1])
+                  : defaultPortMap[upstream.scheme]
+                    ? defaultPortMap[upstream.scheme]
+                    : 80,
+              weight: upstream.nodes[node],
+            };
+          })
+      : undefined;
     return ADCSDK.utils.recursiveOmitUndefined({
       name: upstream.name ?? upstream.id,
       description: upstream.desc,