all tests pass locally now!

lperson · Apr 28, 2024 · 255fb9c · 255fb9c
1 parent ab862e5
commit 255fb9c
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 4 deletions.
diff --git a/__test__/server/api/createOptOut.test.js b/__test__/server/api/createOptOut.test.js
@@ -1,5 +1,5 @@
 /* eslint-disable no-unused-expressions, consistent-return */
-import { r } from "../../../src/server/models/";
+import { r } from "../../../src/server/models";
 
 import {
   setupTest,

diff --git a/src/server/api/errors.js b/src/server/api/errors.js
@@ -21,7 +21,11 @@ export async function accessRequired(
     return;
   }
   // require a permission at-or-higher than the permission requested
-  const hasRole = await cacheableData.user.userHasRole(user, orgId, role);
+  const hasRole = await cacheableData.user.userHasRole(
+    user,
+    orgId.toString(),
+    role
+  );
   if (!hasRole) {
     const error = new GraphQLError(
       "You are not authorized to access that resource."
@@ -65,7 +69,7 @@ export async function assignmentRequiredOrAdminRole(
   const roleRequired = userHasAssignment ? "TEXTER" : "SUPERVOLUNTEER";
   const hasPermission = await cacheableData.user.userHasRole(
     user,
-    orgId,
+    orgId.toString(),
     roleRequired
   );
   if (!hasPermission) {

diff --git a/src/server/api/schema.js b/src/server/api/schema.js
@@ -313,7 +313,7 @@ async function editCampaign(id, campaign, loaders, user, origCampaignRecord) {
     changed = true;
     await accessRequired(
       user,
-      organizationId.toString(),
+      organizationId,
       "SUPERVOLUNTEER",
       /* superadmin*/ true
     );