fix tokenUtils tests, skip if not configured

Author: mchangrh

ci.json

@@ -72,7 +72,7 @@
         }
     },
     "patreon": {
-        "clientID": "testClientID",
+        "clientId": "testClientID",
         "clientSecret": "testClientSecret",
         "redirectUri": "http://127.0.0.1/fake/callback"
     },

test.json

@@ -59,5 +59,10 @@
           "statusCode": 200
         }
     },
+    "patreon": {
+        "clientId": "testClientID",
+        "clientSecret": "testClientSecret",
+        "redirectUri": "http://127.0.0.1/fake/callback"
+    },
     "minReputationToSubmitFiller": -1
 }

test/cases/tokenUtils.ts

@@ -1,23 +1,41 @@
 import assert from "assert";
+import { config } from "../../src/config";
 import axios from "axios";
 import * as tokenUtils from "../../src/utils/tokenUtils";
 import MockAdapter from "axios-mock-adapter";
 import { validatelicenseKeyRegex } from "../../src/routes/verifyToken";
 let mock: MockAdapter;
 
 const validateToken = validatelicenseKeyRegex;
+const fakePatreonIdentity = {
+    data: {},
+    links: {},
+    included: [
+        {
+            attributes: {
+                is_monthly: true,
+                currently_entitled_amount_cents: 100,
+                patron_status: "active_patron",
+            },
+            id: "id",
+            type: "campaign"
+        }
+    ],
+};
 
 describe("tokenUtils test", function() {
     before(function() {
         mock = new MockAdapter(axios, { onNoMatch: "throwException" });
-    });
-
-    it("Should be able to create patreon token", (done) => {
         mock.onPost("https://www.patreon.com/api/oauth2/token").reply(200, {
             access_token: "test_access_token",
             refresh_token: "test_refresh_token",
             expires_in: 3600,
         });
+        mock.onGet(/identity/).reply(200, fakePatreonIdentity);
+    });
+
+    it("Should be able to create patreon token", function (done) {
+        if (!config?.patreon) this.skip();
         tokenUtils.createAndSaveToken(tokenUtils.TokenType.patreon, "test_code").then((licenseKey) => {
             assert.ok(validateToken(licenseKey));
             done();
@@ -29,34 +47,15 @@ describe("tokenUtils test", function() {
             done();
         });
     });
-    it("Should be able to get patreon identity", (done) => {
-        const fakePatreonIdentity = {
-            data: {},
-            links: {},
-            included: [
-                {
-                    attributes: {
-                        is_monthly: true,
-                        currently_entitled_amount_cents: 100,
-                        patron_status: "active_patron",
-                    },
-                    id: "id",
-                    type: "campaign"
-                }
-            ],
-        };
-        mock.onGet(/identity/).reply(200, fakePatreonIdentity);
+    it("Should be able to get patreon identity", function (done) {
+        if (!config?.patreon) this.skip();
         tokenUtils.getPatreonIdentity("fake_access_token").then((result) => {
             assert.deepEqual(result, fakePatreonIdentity);
             done();
         });
     });
-    it("Should be able to refresh token", (done) => {
-        mock.onPost("https://www.patreon.com/api/oauth2/token").reply(200, {
-            access_token: "test_access_token",
-            refresh_token: "test_refresh_token",
-            expires_in: 3600,
-        });
+    it("Should be able to refresh token", function (done) {
+        if (!config?.patreon) this.skip();
         tokenUtils.refreshToken(tokenUtils.TokenType.patreon, "fake-licence-Key", "fake_refresh_token").then((result) => {
             assert.strictEqual(result, true);
             done();