[WIP] Ensure scopes defined in scopes.equals()

src/auth/scopes/index.ts

@@ -1,3 +1,5 @@
+//import {MissingRequiredArgument} from '../../error';
+
 class AuthScopes {
   public static SCOPE_DELIMITER = ',';
 
@@ -38,6 +40,10 @@ class AuthScopes {
   public equals(otherScopes: string | string[] | AuthScopes) {
     let other: AuthScopes;
 
+    // if (!otherScopes) {
+    //   throw new MissingRequiredArgument('Missing scopes');
+    // }
+
     if (otherScopes instanceof AuthScopes) {
       other = otherScopes;
     } else {

src/auth/scopes/test/scopes.test.ts

@@ -1,4 +1,5 @@
 import '../../../test/test_helper';
+import * as ShopifyErrors from '../../../error';
 
 import {AuthScopes} from '../index';
 
@@ -65,6 +66,13 @@ describe('AuthScopes.equals', () => {
     expect(scopes2.equals(scopes1)).toBeFalsy();
   });
 
+  it('throws an error if no scopes', () => {
+    const scopes1 = new AuthScopes('write_customers,read_products');
+    const scopes2: string = '';
+
+    expect(scopes1.equals(scopes2)).toThrow(ShopifyErrors.MissingRequiredArgument);
+  })
+
   it('allows comparing against strings', () => {
     const scopes1 = new AuthScopes('write_customers,read_products,write_products');