chore(lint): fix linting issues

addon/authenticators/oidc.js

@@ -31,14 +31,14 @@ export default class OidcAuthenticator extends BaseAuthenticator {
   async authenticate({ code, redirectUri, codeVerifier, isRefresh }) {
     if (!this.config.tokenEndpoint || !this.config.userinfoEndpoint) {
       throw new Error(
-        "Please define all OIDC endpoints (auth, token, userinfo)"
+        "Please define all OIDC endpoints (auth, token, userinfo)",
       );
     }
 
     if (isRefresh) {
       return await this._refresh(
         this.session.data.authenticated.refresh_token,
-        redirectUri
+        redirectUri,
       );
     }
 
@@ -66,7 +66,7 @@ export default class OidcAuthenticator extends BaseAuthenticator {
           "Content-Type": "application/x-www-form-urlencoded",
         },
         body,
-      }
+      },
     );
 
     const isServerError = isServerErrorResponse(response);
@@ -110,7 +110,9 @@ export default class OidcAuthenticator extends BaseAuthenticator {
 
     if (this.config.afterLogoutUri) {
       params.push(
-        `post_logout_redirect_uri=${getAbsoluteUrl(this.config.afterLogoutUri)}`
+        `post_logout_redirect_uri=${getAbsoluteUrl(
+          this.config.afterLogoutUri,
+        )}`,
       );
     }
 
@@ -121,8 +123,8 @@ export default class OidcAuthenticator extends BaseAuthenticator {
     this._redirectToUrl(
       `${getAbsoluteUrl(
         this.config.endSessionEndpoint,
-        this.config.host
-      )}?${params.join("&")}`
+        this.config.host,
+      )}?${params.join("&")}`,
     );
   }
 
@@ -183,7 +185,7 @@ export default class OidcAuthenticator extends BaseAuthenticator {
             "Content-Type": "application/x-www-form-urlencoded",
           },
           body,
-        }
+        },
       );
       isServerError = isServerErrorResponse(response);
       if (isServerError) throw new Error(response.message);
@@ -208,9 +210,9 @@ export default class OidcAuthenticator extends BaseAuthenticator {
             this,
             () =>
               resolve(
-                this._refresh(refresh_token, redirectUri, retryCount + 1)
+                this._refresh(refresh_token, redirectUri, retryCount + 1),
               ),
-            this.config.retryTimeout
+            this.config.retryTimeout,
           );
         });
       }
@@ -232,7 +234,7 @@ export default class OidcAuthenticator extends BaseAuthenticator {
           Authorization: `${this.config.authPrefix} ${accessToken}`,
           Accept: "application/json",
         },
-      }
+      },
     );
 
     const userinfo = await response.json();

addon/routes/oidc-authentication.js

@@ -68,7 +68,7 @@ export default class OIDCAuthenticationRoute extends Route {
   async afterModel(_, transition) {
     if (!this.config.authEndpoint) {
       throw new Error(
-        "Please define all OIDC endpoints (auth, token, logout, userinfo)"
+        "Please define all OIDC endpoints (auth, token, logout, userinfo)",
       );
     }
 
@@ -80,7 +80,7 @@ export default class OIDCAuthenticationRoute extends Route {
       return await this._handleCallbackRequest(
         queryParams.code,
         queryParams.state,
-        transition
+        transition,
       );
     }
 
@@ -159,7 +159,7 @@ export default class OIDCAuthenticationRoute extends Route {
 
     if (this.config.enablePkce) {
       const pkceChallenge = generatePkceChallenge(
-        this.session.data.pkceCodeVerifier
+        this.session.data.pkceCodeVerifier,
       );
       search.push(`code_challenge=${pkceChallenge}`);
       search.push("code_challenge_method=S256");
@@ -168,7 +168,9 @@ export default class OIDCAuthenticationRoute extends Route {
     search = search.filter(Boolean).join("&");
 
     this._redirectToUrl(
-      `${getAbsoluteUrl(this.config.host)}${this.config.authEndpoint}?${search}`
+      `${getAbsoluteUrl(this.config.host)}${
+        this.config.authEndpoint
+      }?${search}`,
     );
   }
 }

addon/unauthorized.js

@@ -7,7 +7,7 @@ import getAbsoluteUrl from "ember-simple-auth-oidc/utils/absolute-url";
 
 const replaceUri = (session) => {
   location.replace(
-    getAbsoluteUrl(getConfig(getOwner(session)).afterLogoutUri || "")
+    getAbsoluteUrl(getConfig(getOwner(session)).afterLogoutUri || ""),
   );
 };
 
@@ -28,7 +28,7 @@ export default function handleUnauthorized(session) {
       this,
       replaceUri,
       session,
-      getConfig(getOwner(session)).unauthorizedRequestRedirectTimeout
+      getConfig(getOwner(session)).unauthorizedRequestRedirectTimeout,
     );
   }
 }

addon/utils/pkce.js

@@ -22,7 +22,7 @@ export function generateCodeVerifier(len = 43) {
   const chars = new Array(len);
   for (let i = 0; i < len; i++) {
     chars[i] = RANDOM_DATA_ALPHABET.charCodeAt(
-      randomData[i] % RANDOM_DATA_ALPHABET.length
+      randomData[i] % RANDOM_DATA_ALPHABET.length,
     );
   }
 

tests/dummy/mirage/config.js

@@ -33,7 +33,7 @@ export default function makeServer(config) {
           {},
           {
             data: { items: [{ id: 1, name: "Test" }] },
-          }
+          },
         );
       });
     },

tests/unit/adapters/oidc-json-api-adapter-test.js

@@ -8,8 +8,6 @@ module("Unit | Adapter | oidc json api adapter", function (hooks) {
   setupMirage(hooks);
 
   test("it sets the correct headers", async function (assert) {
-    assert.expect(1);
-
     const adapter = this.owner.lookup("adapter:oidc-json-api-adapter");
     const session = this.owner.lookup("service:session");
     set(session, "session.isAuthenticated", true);
@@ -19,8 +17,6 @@ module("Unit | Adapter | oidc json api adapter", function (hooks) {
   });
 
   test("it refreshes the access token before ember-data requests", async function (assert) {
-    assert.expect(21);
-
     const adapter = this.owner.lookup("adapter:oidc-json-api-adapter");
 
     adapter.session.refreshAuthentication.perform = () => {
@@ -46,8 +42,6 @@ module("Unit | Adapter | oidc json api adapter", function (hooks) {
   });
 
   test("it invalidates the session correctly on a 401 response", function (assert) {
-    assert.expect(3);
-
     const adapter = this.owner.lookup("adapter:oidc-json-api-adapter");
     const session = adapter.session;
     session.session.content = {};
@@ -60,7 +54,7 @@ module("Unit | Adapter | oidc json api adapter", function (hooks) {
 
     assert.strictEqual(
       adapter.session.data.nextURL,
-      location.href.replace(location.origin, "")
+      location.href.replace(location.origin, ""),
     );
 
     assert.verifySteps(["invalidate"]);

tests/unit/adapters/oidc-rest-adapter-test.js

@@ -8,8 +8,6 @@ module("Unit | Adapter | oidc rest adapter", function (hooks) {
   setupMirage(hooks);
 
   test("it sets the correct headers", async function (assert) {
-    assert.expect(1);
-
     const adapter = this.owner.lookup("adapter:oidc-rest-adapter");
     const session = this.owner.lookup("service:session");
     set(session, "session.isAuthenticated", true);
@@ -19,8 +17,6 @@ module("Unit | Adapter | oidc rest adapter", function (hooks) {
   });
 
   test("it refreshes the access token before ember-data requests", async function (assert) {
-    assert.expect(21);
-
     const adapter = this.owner.lookup("adapter:oidc-rest-adapter");
 
     adapter.session.refreshAuthentication.perform = () => {
@@ -46,8 +42,6 @@ module("Unit | Adapter | oidc rest adapter", function (hooks) {
   });
 
   test("it invalidates the session correctly on a 401 response", function (assert) {
-    assert.expect(3);
-
     const adapter = this.owner.lookup("adapter:oidc-rest-adapter");
     const session = adapter.session;
     session.session.content = {};
@@ -60,7 +54,7 @@ module("Unit | Adapter | oidc rest adapter", function (hooks) {
 
     assert.strictEqual(
       adapter.session.data.nextURL,
-      location.href.replace(location.origin, "")
+      location.href.replace(location.origin, ""),
     );
 
     assert.verifySteps(["invalidate"]);

tests/unit/authenticators/oidc-test.js

@@ -10,7 +10,7 @@ const getTokenBody = (expired) => {
   return btoa(
     JSON.stringify({
       exp: Date.now() + time,
-    })
+    }),
   );
 };
 
@@ -19,8 +19,6 @@ module("Unit | Authenticator | OIDC", function (hooks) {
   setupMirage(hooks);
 
   test("it can authenticate", async function (assert) {
-    assert.expect(4);
-
     const subject = this.owner.lookup("authenticator:oidc");
 
     set(subject, "redirectUri", "test");
@@ -34,8 +32,6 @@ module("Unit | Authenticator | OIDC", function (hooks) {
   });
 
   test("it can restore a session", async function (assert) {
-    assert.expect(4);
-
     const subject = this.owner.lookup("authenticator:oidc");
 
     const data = await subject.restore({
@@ -51,16 +47,12 @@ module("Unit | Authenticator | OIDC", function (hooks) {
   });
 
   test("it can invalidate a session", async function (assert) {
-    assert.expect(1);
-
     const subject = this.owner.lookup("authenticator:oidc");
 
     assert.ok(await subject.invalidate());
   });
 
   test("it can refresh a session", async function (assert) {
-    assert.expect(4);
-
     const subject = this.owner.lookup("authenticator:oidc");
 
     const data = await subject._refresh("x.y.z");
@@ -72,15 +64,13 @@ module("Unit | Authenticator | OIDC", function (hooks) {
   });
 
   test("it can make a single logout", async function (assert) {
-    assert.expect(3);
-
     const { endSessionEndpoint, afterLogoutUri } = getConfig(this.owner);
     const subject = this.owner.lookup("authenticator:oidc");
 
     subject._redirectToUrl = (url) => {
       assert.ok(new RegExp(endSessionEndpoint).test(url));
       assert.ok(
-        new RegExp(`post_logout_redirect_uri=${afterLogoutUri}`).test(url)
+        new RegExp(`post_logout_redirect_uri=${afterLogoutUri}`).test(url),
       );
       assert.ok(new RegExp("id_token_hint=myIdToken").test(url));
     };