chore: throw error if hasAuthority is called with invalid arguments

src/components/Interpretations/common/getInterpretationAccess.js

@@ -1,12 +1,15 @@
 // For backwards compatibility
 // accept both Set (from the old d2.currentUser object) and array
-const hasAuthority = (authorities = [], authority) => {
-    if (Array.isArray(authorities)) {
+const hasAuthority = (authorities, authority) => {
+    if (!authority || typeof authority !== 'string) {
+        throw new Error(`"hasAuthority" requires "authority" to be a populated string but received ${authority}`)
+    } else if (Array.isArray(authorities)) {
         return authorities.includes(authority)
+    } else if (typeof authorities.has === 'function') {
+        return authorities.has(authority)
+    } else {
+        throw new Error(`"hasAuthority" requires "authorities" to be an array or set of authorities (strings)`)
     }
-    return typeof authorities.has === 'function'
-        ? authorities.has(authority)
-        : false
 }
 
 const isSuperuser = (authorities) => hasAuthority(authorities, 'ALL')