test(jest): Fix app import pattern - destructure from index.js - Issue #618

### Error Fixed (18 occurrences across 7 files)

**Error:** TypeError: app.address is not a function

**Root Cause:**
- src/index.js exports: `module.exports = { app, server }`
- Tests imported: `const app = require('../../src/index')` ❌
- This assigns the entire `{ app, server }` object to variable `app`
- When supertest calls `app.address()`, fails because object doesn't have that method

**Fix:**
- Changed to: `const { app } = require('../../src/index')` ✅
- Destructures `app` from the exported object
- Now supertest can call `app.address()` correctly

### Files Modified (7 tests)

1. tests/integration/styleProfileWorkflow.test.js
2. tests/integration/autoApprovalSecurityV2.test.js
3. tests/integration/multiTenantWorkflowComplete.test.js
4. tests/qa/oauth-integration-tests.js
5. tests/qa/token-management-tests.js
6. tests/qa/payload-processing-tests.js
7. tests/qa/webhook-tests.js

### Pattern

This is the **same import/export mismatch pattern as logger** (Session #2):
- Module exports object with named properties
- Tests must destructure to get the specific property
- Without destructuring, variable holds the container object, not the property

### Test Results

**Before:**
- Error: "app.address is not a function" in 18 test cases
- autoApprovalSecurityV2.test.js: 0/9 passing

**After:**
- Error: ✅ RESOLVED
- autoApprovalSecurityV2.test.js: Tests run (fail for different reasons - 404s)
- **Progress:** Tests now fail on business logic, not Jest/import errors

### Prevention

- ✅ Check how module exports before importing
- ✅ If exports object, always destructure: `const { item } = require(...)`
- ✅ Pattern applies to: app, server, logger, flags, etc.

Related: Pattern #10 (logger import), Session #2 (Issue #618)

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>

Author: Eibon7

tests/integration/autoApprovalSecurityV2.test.js

@@ -11,7 +11,7 @@
  */
 
 const request = require('supertest');
-const app = require('../../src/index');
+const { app } = require('../../src/index');
 const { supabaseServiceClient } = require('../../src/config/supabase');
 
 // Mock Supabase

tests/integration/multiTenantWorkflowComplete.test.js

@@ -11,7 +11,7 @@
  */
 
 const request = require('supertest');
-const app = require('../../src/index');
+const { app } = require('../../src/index');
 const { createClient } = require('@supabase/supabase-js');
 const BaseWorker = require('../../src/workers/BaseWorker');
 const AnalyzeToxicityWorker = require('../../src/workers/AnalyzeToxicityWorker');

tests/integration/styleProfileWorkflow.test.js

@@ -1,5 +1,5 @@
 const request = require('supertest');
-const app = require('../../src/index');
+const { app } = require('../../src/index');
 const styleProfileService = require('../../src/services/styleProfileService');
 const queueService = require('../../src/services/queueService');
 const { supabaseServiceClient } = require('../../src/config/supabase');

tests/qa/oauth-integration-tests.js

@@ -7,7 +7,7 @@
  */
 
 const request = require('supertest');
-const app = require('../../src/index');
+const { app } = require('../../src/index');
 const { flags } = require('../../src/config/flags');
 const { logger } = require('../../src/utils/logger');
 

tests/qa/payload-processing-tests.js

@@ -7,7 +7,7 @@
 
 const request = require('supertest');
 const crypto = require('crypto');
-const app = require('../../src/index');
+const { app } = require('../../src/index');
 const { logger } = require('../../src/utils/logger');
 const { flags } = require('../../src/config/flags');
 

tests/qa/token-management-tests.js

@@ -6,7 +6,7 @@
  */
 
 const request = require('supertest');
-const app = require('../../src/index');
+const { app } = require('../../src/index');
 const { OAuthProviderFactory } = require('../../src/services/oauthProvider');
 const { flags } = require('../../src/config/flags');
 const { logger } = require('../../src/utils/logger');

tests/qa/webhook-tests.js

@@ -10,7 +10,7 @@
 
 const request = require('supertest');
 const crypto = require('crypto');
-const app = require('../../src/index');
+const { app } = require('../../src/index');
 const { logger } = require('../../src/utils/logger');
 const { flags } = require('../../src/config/flags');