fix(sanity): revert OAuth approach and expose bulk job auth errors

OAuth Changes:
- Revert to separate login for OAuth tests (creates authtoken #2)
- Maintains proper business logic separation between basic auth and OAuth flow
- Total: 2 authtokens (both reused, no token limit issue)

Bulk Operation Changes:
- Expose 401/403 authentication errors immediately (don't retry)
- Previously: silently caught ALL errors, hiding auth failures
- Now: fail fast on auth errors, retry only on network/5xx errors
- Increased polling: 5→10 attempts (50s max), 60s timeout
- Result: tests will show real error instead of generic timeout

This fixes the 401 errors visible in pipeline console that were being masked.

Author: AniketDev7

.talismanrc

@@ -74,7 +74,7 @@ fileignoreconfig:
   - filename: test/sanity-check/api/team-test.js
     checksum: e4b7a6824b89e634981651ad29161901377f23bb37d3653a389ac3dc4e7653c7
   - filename: test/sanity-check/api/oauth-test.js
-    checksum: fd8a4fe7a644955ea6609813c655d8fca6bb3c7eeea4ae2c5ba99d30b1950172
+    checksum: 7e2d6314f120c20a491d2f8403c9aa4e0af6dfc6fab436f0acc77953cef866e2
   - filename: test/sanity-check/api/branchAlias-test.js
     checksum: 0b6cacee74d7636e84ce095198f0234d491b79ea20d3978a742a5495692bd61d
   - filename: test/sanity-check/utility/testSetup.js

test/sanity-check/api/bulkOperation-test.js

@@ -40,7 +40,7 @@ function assetsWithValidUids () {
   return [assetUid1, assetUid2].filter(uid => uid && String(uid).trim()).map(uid => ({ uid }))
 }
 
-async function waitForJobReady (jobId, maxAttempts = 5) {
+async function waitForJobReady (jobId, maxAttempts = 10) {
   for (let attempt = 1; attempt <= maxAttempts; attempt++) {
     try {
       const response = await doBulkOperation()
@@ -49,7 +49,11 @@ async function waitForJobReady (jobId, maxAttempts = 5) {
         return response
       }
     } catch (error) {
-      // retry
+      // Don't retry on auth errors (401/403) - fail immediately
+      if (error.status === 401 || error.status === 403) {
+        throw new Error(`Job status check failed with ${error.status}: ${error.errorMessage || error.message}`)
+      }
+      // For other errors (network, 5xx), retry
     }
     await delay(5000)
   }
@@ -360,7 +364,7 @@ describe('BulkOperation api test', () => {
   })
 
   it('should wait for jobs to be ready and get job status for the first publish job', async function () {
-    this.timeout(30000)
+    this.timeout(60000)
 
     const response = await waitForJobReady(jobId1)
     expect(response).to.not.equal(undefined)
@@ -384,7 +388,7 @@ describe('BulkOperation api test', () => {
   })
 
   it('should get job status for the second publish job', async function () {
-    this.timeout(30000)
+    this.timeout(60000)
 
     const response = await waitForJobReady(jobId2)
     expect(response).to.not.equal(undefined)
@@ -396,7 +400,7 @@ describe('BulkOperation api test', () => {
   })
 
   it('should get job status for the third publish job', async function () {
-    this.timeout(30000)
+    this.timeout(60000)
 
     const response = await waitForJobReady(jobId3)
     expect(response).to.not.equal(undefined)
@@ -408,7 +412,7 @@ describe('BulkOperation api test', () => {
   })
 
   it('should get job status for publishAllLocalized=true job', async function () {
-    this.timeout(30000)
+    this.timeout(60000)
 
     const response = await waitForJobReady(jobId4)
     expect(response).to.not.equal(undefined)
@@ -420,7 +424,7 @@ describe('BulkOperation api test', () => {
   })
 
   it('should get job status for publishAllLocalized=false job', async function () {
-    this.timeout(30000)
+    this.timeout(60000)
 
     const response = await waitForJobReady(jobId5)
     expect(response).to.not.equal(undefined)
@@ -432,7 +436,7 @@ describe('BulkOperation api test', () => {
   })
 
   it('should get job status for asset publishAllLocalized job', async function () {
-    this.timeout(30000)
+    this.timeout(60000)
 
     const response = await waitForJobReady(jobId6)
     expect(response).to.not.equal(undefined)
@@ -444,7 +448,7 @@ describe('BulkOperation api test', () => {
   })
 
   it('should get job status for unpublishAllLocalized=true job', async function () {
-    this.timeout(30000)
+    this.timeout(60000)
 
     const response = await waitForJobReady(jobId7)
     expect(response).to.not.equal(undefined)
@@ -456,7 +460,7 @@ describe('BulkOperation api test', () => {
   })
 
   it('should get job status for unpublishAllLocalized=false job', async function () {
-    this.timeout(30000)
+    this.timeout(60000)
 
     const response = await waitForJobReady(jobId8)
     expect(response).to.not.equal(undefined)
@@ -468,7 +472,7 @@ describe('BulkOperation api test', () => {
   })
 
   it('should get job status for asset unpublishAllLocalized job', async function () {
-    this.timeout(30000)
+    this.timeout(60000)
 
     const response = await waitForJobReady(jobId9)
     expect(response).to.not.equal(undefined)
@@ -480,7 +484,7 @@ describe('BulkOperation api test', () => {
   })
 
   it('should get job status for multiple parameters job', async function () {
-    this.timeout(30000)
+    this.timeout(60000)
 
     const response = await waitForJobReady(jobId10)
     expect(response).to.not.equal(undefined)
@@ -492,7 +496,7 @@ describe('BulkOperation api test', () => {
   })
 
   it('should get job status with bulk_version parameter', async function () {
-    this.timeout(30000)
+    this.timeout(60000)
 
     await waitForJobReady(jobId1)
     const response = await doBulkOperationWithManagementToken(tokenUidDev)
@@ -506,7 +510,7 @@ describe('BulkOperation api test', () => {
   })
 
   it('should get job items for a completed job', async function () {
-    this.timeout(30000)
+    this.timeout(60000)
 
     await waitForJobReady(jobId1)
     const response = await doBulkOperationWithManagementToken(tokenUidDev)
@@ -516,7 +520,7 @@ describe('BulkOperation api test', () => {
   })
 
   it('should get job items with explicit api_version', async function () {
-    this.timeout(30000)
+    this.timeout(60000)
 
     await waitForJobReady(jobId2)
     const response = await doBulkOperationWithManagementToken(tokenUidDev)