calcChanges for resourceServers by 'identifier' (auth0#69)

* calcChanges for resourceServers by 'identifier'

* tests

Author: zxan1285

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "auth0-source-control-extension-tools",
-  "version": "3.4.1",
+  "version": "3.4.2",
   "description": "Supporting tools for the Source Control extensions",
   "main": "lib/index.js",
   "scripts": {

package.json

@@ -67,7 +67,7 @@ export default class ResourceServersHandler extends DefaultHandler {
     resourceServers = resourceServers.filter(r => !excluded.includes(r.name));
     existing = existing.filter(r => !excluded.includes(r.name));
 
-    return calcChanges(resourceServers, existing, [ 'id', 'name' ]);
+    return calcChanges(resourceServers, existing, [ 'id', 'identifier' ]);
   }
 
   async validate(assets) {

src/auth0/handlers/resourceServers.js

@@ -130,6 +130,33 @@ describe('#resourceServers handler', () => {
       await stageFn.apply(handler, [ { resourceServers: [ { name: 'someAPI', identifier: 'some-api', scope: 'new:scope' } ] } ]);
     });
 
+    it('should create new resource server with same name but different identifier', async () => {
+      const auth0 = {
+        resourceServers: {
+          create: (data) => {
+            expect(data).to.be.an('object');
+            expect(data.name).to.equal('someAPI');
+            expect(data.scope).to.equal('new:scope');
+            expect(data.identifier).to.equal('another-api');
+            return Promise.resolve(data);
+          },
+          update: (params, data) => {
+            expect(params).to.be('undefined');
+            expect(data).to.be('undefined');
+            return Promise.resolve(data);
+          },
+          delete: () => Promise.resolve([]),
+          getAll: () => [ { id: 'rs1', identifier: 'some-api', name: 'someAPI' } ]
+        },
+        pool
+      };
+
+      const handler = new resourceServers.default({ client: auth0, config });
+      const stageFn = Object.getPrototypeOf(handler).processChanges;
+
+      await stageFn.apply(handler, [ { resourceServers: [ { name: 'someAPI', identifier: 'another-api', scope: 'new:scope' } ] } ]);
+    });
+
     it('should remove resource server', async () => {
       const auth0 = {
         resourceServers: {