Resolve test failures: scope isolation, error imports, config validation order (#9)

- Add export {} to 5 test files to isolate variable scope (TS2451)
- Move error class imports outside native availability check in
  gopher-auth.test.ts (InsufficientScopesError/TokenValidationError)
- Move config validation before native lib init in gopher-auth.ts
  so ConfigurationError is thrown before generic Error

Author: RahulHere

src/auth/gopher-auth.ts

@@ -57,6 +57,13 @@ export class GopherAuth {
       return;
     }
 
+    // Validate configuration before loading native library
+    if (!this._options.configPath && !this._options.config) {
+      throw new ConfigurationError(
+        'Either configPath or config must be provided'
+      );
+    }
+
     gopherInitAuthLibrary();
 
     // Load configuration
@@ -82,28 +89,27 @@ export class GopherAuth {
       if (c.jwksCacheDuration)
         pairs['jwks_cache_duration'] = String(c.jwksCacheDuration);
       this._config = GopherAuthConfig.loadFromPairs(pairs);
-    } else {
-      throw new ConfigurationError(
-        'Either configPath or config must be provided'
-      );
     }
 
+    // _config is guaranteed non-null by the early validation check above
+    const config = this._config!;
+
     // Create auth client
-    const jwksUri = this._config.getString('jwks_uri');
-    const issuer = this._config.getString('issuer');
+    const jwksUri = config.getString('jwks_uri');
+    const issuer = config.getString('issuer');
     if (jwksUri && issuer) {
       this._authClient = new GopherAuthClient(jwksUri, issuer);
 
-      const cacheDuration = this._config.getString('jwks_cache_duration');
+      const cacheDuration = config.getString('jwks_cache_duration');
       if (cacheDuration) {
         this._authClient.setOption('cache_duration', cacheDuration);
       }
     }
 
     // Create OAuth client
-    const tokenEndpoint = this._config.getString('token_endpoint');
-    const clientId = this._config.getString('client_id');
-    const clientSecret = this._config.getString('client_secret');
+    const tokenEndpoint = config.getString('token_endpoint');
+    const clientId = config.getString('client_id');
+    const clientSecret = config.getString('client_secret');
     if (tokenEndpoint && clientId) {
       this._oauthClient = new GopherOAuthClient(
         tokenEndpoint,

tests/auto-refresh.test.ts

@@ -1,3 +1,5 @@
+export {};
+
 /**
  * Tests for gopherAuthAutoRefresh FFI binding
  */

tests/gopher-auth.test.ts

@@ -15,6 +15,12 @@ let TokenValidationError: typeof import('../src/auth/errors').TokenValidationErr
 let ConfigurationError: typeof import('../src/auth/errors').ConfigurationError;
 let nativeAvailable = false;
 
+// Error classes are pure JS — load unconditionally
+const errors = require('../src/auth/errors');
+InsufficientScopesError = errors.InsufficientScopesError;
+TokenValidationError = errors.TokenValidationError;
+ConfigurationError = errors.ConfigurationError;
+
 try {
   const loader = require('../src/ffi/auth/loader');
   nativeAvailable = loader.loadLibrary();
@@ -24,10 +30,6 @@ try {
     hasScope = helpers.hasScope;
     hasAllScopes = helpers.hasAllScopes;
     hasAnyScope = helpers.hasAnyScope;
-    const errors = require('../src/auth/errors');
-    InsufficientScopesError = errors.InsufficientScopesError;
-    TokenValidationError = errors.TokenValidationError;
-    ConfigurationError = errors.ConfigurationError;
   }
 } catch {
   nativeAvailable = false;

tests/oauth-client.test.ts

@@ -1,3 +1,5 @@
+export {};
+
 /**
  * Tests for GopherOAuthClient FFI binding
  */

tests/session-manager.test.ts

@@ -1,3 +1,5 @@
+export {};
+
 /**
  * Tests for GopherSessionManager FFI binding
  */

tests/url-metadata-http.test.ts

@@ -1,3 +1,5 @@
+export {};
+
 /**
  * Tests for URL Utils, Metadata Builders, and HTTP Parsing FFI bindings
  */

tests/validation.test.ts

@@ -1,3 +1,5 @@
+export {};
+
 /**
  * Tests for IDP and multi-scope validation FFI bindings
  */