Failed to generate code suggestions for PR

Greptile Overview

Greptile Summary

This PR improves SDR (Sensitive Data Redaction) E2E test reliability by adding test isolation and retry logic to handle parallel execution and backend timing issues.

• Added unique testRunId (based on timestamp) to pattern and stream names across all SDR tests, preventing conflicts when tests run in parallel
• Introduced retry logic with exponential backoff in ingestMultipleFields and ingestSingleLog functions to handle "stream being deleted" errors
• Added waitForStreamAvailable API check before UI stream selection, with fallback retries
• Enhanced selectStream with search box filtering and scroll-based navigation for reliable stream selection in long lists
• Added retry logic to checkPatternExists (3 attempts with 2s wait) and Stream Detail button detection
• Changed navigation methods to use direct URL navigation with dynamic org_identifier instead of clicking through UI elements
• Fixed a template literal bug in iamPage.js where ${emailName} wasn't being interpolated
• Consolidated duplicate ingestMultipleFields helper function into logsPage.js page object
• Updated API URL patterns across multiple spec files to use dynamic ORGNAME instead of hardcoded 'default'
• Enhanced apiCleanup.js with prefix-based pattern matching and new SDR cleanup methods

Confidence Score: 5/5

• This PR is safe to merge - it only modifies E2E test infrastructure with no changes to production code.
• The changes are limited to test files and test page objects. They add retry logic and test isolation patterns that improve test reliability without affecting production code. The code changes are well-structured and follow existing patterns in the codebase.
• No files require special attention - all changes are in test infrastructure.

Important Files Changed

File Analysis

Sequence Diagram

sequenceDiagram
    participant Test as SDR Test
    participant LogsPage as logsPage.js
    participant API as Backend API
    participant UI as UI Components
    
    Test->>LogsPage: ingestMultipleFields(streamName, data)
    loop Retry up to 5 times
        LogsPage->>API: POST /api/{org}/{stream}/_json
        alt Success (200)
            API-->>LogsPage: OK
            LogsPage->>LogsPage: Wait 5s for indexing
        else "Stream being deleted" error
            API-->>LogsPage: Error
            LogsPage->>LogsPage: Exponential backoff wait
        end
    end
    
    Test->>LogsPage: selectStream(streamName)
    LogsPage->>API: GET /api/{org}/streams (waitForStreamAvailable)
    loop Poll until found or timeout
        API-->>LogsPage: Stream list
        LogsPage->>LogsPage: Check if stream exists
    end
    
    LogsPage->>UI: Navigate to logs URL
    loop Retry up to 3 times
        LogsPage->>UI: Click dropdown, search, scroll
        alt Stream found
            UI-->>LogsPage: Stream selected
        else Not found
            LogsPage->>UI: Escape, wait 5s, retry
        end
    end

@greptileai