Add Phase 3 tests for Async Operations

Implement comprehensive test coverage for async coroutine-based
worker jobs that handle image loading and cropping. This phase
ensures async operations handle errors gracefully, manage memory
correctly, and respect lifecycle properly.

**New Test Files (33 tests total):**

TestCoroutineExtensions.kt (Test Helper)
- CoroutineTestRule for managing test dispatchers
- Automatic setup/teardown of Main dispatcher
- TestScope and UnconfinedTestDispatcher for immediate execution

BitmapLoadingWorkerJobTest.kt (19 tests)
- Successful loading flow with EXIF orientation
- Error handling (decode failures, EXIF read failures)
- Density calculation (high DPI adjustment, low DPI passthrough)
- Coroutine lifecycle (start, cancel, isActive checks)
- WeakReference memory management
- EXIF orientation support (0°, 90°, 180°, 270°)
- EXIF flip support (horizontal, vertical, combined)
- Sample size propagation

BitmapCroppingWorkerJobTest.kt (14 tests)
- URI-based cropping with all parameters
- Bitmap-based cropping with OOM handling
- Error handling (cropping failures, write failures)
- Null input handling (empty results)
- Coroutine lifecycle (cancellation prevents callback)
- Memory management (bitmap recycling when callback not invoked)
- Resize operations with RequestSizeOptions
- Compression format support (JPEG, PNG)
- Custom output URI handling
- Sample size propagation

**CI Timeout Configuration:**
Add sensible timeouts to prevent jobs from hanging indefinitely:
- gradle-wrapper-validation: 2 minutes
- Build jobs: 10 minutes each
- License/Lint checks: 5 minutes
- Android Lint: 10 minutes
- Unit Tests: 15 minutes (for all 167 tests)

Timeouts are 2-3x typical execution time to account for runner
variability and cache misses while ensuring faster failure detection.

**PLAN.md Updates:**
- Marked Phase 3 ✅ Complete
- Fixed Phase 2 test count (65 tests, not 45)
- Updated total: 167 tests across 9 test files
- Added completed test files list with Phase 3 entries

**Key Testing Patterns:**
- MockK for mocking BitmapUtils static methods
- Robolectric for Android context
- Coroutine test library for deterministic async testing
- Slot capture for verifying callback parameters
- UnconfinedTestDispatcher for immediate execution (no advanceUntilIdle needed)

This is Phase 3 of the 8-phase plan, focusing on bulletproof
async operation testing with coroutines.

Author: Canato

.github/workflows/ci-build.yml

@@ -11,6 +11,7 @@ jobs:
   build-library:
     name: Build Library (JDK ${{ matrix.java_version }})
     runs-on: macOS-latest
+    timeout-minutes: 10
 
     strategy:
       matrix:
@@ -43,6 +44,7 @@ jobs:
   build-sample:
     name: Build Sample App (JDK ${{ matrix.java_version }})
     runs-on: macOS-latest
+    timeout-minutes: 10
 
     strategy:
       matrix:

.github/workflows/ci-tests.yml

@@ -11,6 +11,7 @@ jobs:
   license-check:
     name: License Check
     runs-on: macOS-latest
+    timeout-minutes: 5
 
     steps:
       - name: Checkout code
@@ -32,6 +33,7 @@ jobs:
   ktlint:
     name: Kotlin Lint (ktlint)
     runs-on: macOS-latest
+    timeout-minutes: 5
 
     steps:
       - name: Checkout code
@@ -53,6 +55,7 @@ jobs:
   lint:
     name: Android Lint
     runs-on: macOS-latest
+    timeout-minutes: 10
 
     steps:
       - name: Checkout code
@@ -84,6 +87,7 @@ jobs:
   unit-tests:
     name: Unit Tests (JDK ${{ matrix.java_version }})
     runs-on: macOS-latest
+    timeout-minutes: 15
 
     strategy:
       matrix:

.github/workflows/gradle-wrapper-validation.yml

@@ -11,6 +11,7 @@ jobs:
   validation:
     name: Validate Gradle Wrapper
     runs-on: ubuntu-latest
+    timeout-minutes: 2
 
     steps:
       - name: Checkout code

PLAN.md

@@ -2,7 +2,7 @@
 
 **Created:** 2026-04-23  
 **Last Updated:** 2026-04-24  
-**Status:** In Progress - Phase 2 Complete  
+**Status:** In Progress - Phase 3 Complete  
 **Goal:** Achieve comprehensive test coverage to catch bugs before CI/release
 
 ---
@@ -12,24 +12,27 @@
 | Phase | Status | Tests Added | Description |
 |-------|--------|-------------|-------------|
 | **Phase 1** | ✅ **Complete** | 69 tests | Security Foundation - URI and file handling |
-| **Phase 2** | ✅ **Complete** | 48 tests | Configuration & Options validation |
-| **Phase 3** | ⏳ Pending | - | Async Operations (coroutines) |
+| **Phase 2** | ✅ **Complete** | 65 tests | Configuration & Options validation |
+| **Phase 3** | ✅ **Complete** | 33 tests | Async Operations (coroutines, worker jobs) |
 | **Phase 4** | ⏳ Pending | - | Core Bitmap Operations |
 | **Phase 5** | ⏳ Pending | - | Crop Window Logic |
 | **Phase 6** | ⏳ Pending | - | UI Components |
 | **Phase 7** | ⏳ Pending | - | Public API |
 | **Phase 8** | ⏳ Pending | - | Supporting Features |
 
-**Total Tests:** 117 tests across 6 test files  
+**Total Tests:** 167 tests across 9 test files  
 **Coverage Target:** 70-80% overall, 90%+ for critical security files
 
 ### Completed Test Files
 - ✅ `GetFilePathFromUriTest.kt` (27 tests) - Phase 1
 - ✅ `GetUriForFileTest.kt` (25 tests) - Phase 1
 - ✅ `BitmapUtilsTest.kt` (17+ tests expanded) - Phase 1
 - ✅ `CropImageOptionsTest.kt` (33 tests) - Phase 2
-- ✅ `CropExceptionTest.kt` (11 tests) - Phase 2
+- ✅ `CropExceptionTest.kt` (8 tests) - Phase 2
 - ✅ `ParcelableUtilsTest.kt` (24 tests) - Phase 2
+- ✅ `BitmapLoadingWorkerJobTest.kt` (19 tests) - Phase 3
+- ✅ `BitmapCroppingWorkerJobTest.kt` (14 tests) - Phase 3
+- ✅ `TestCoroutineExtensions.kt` (test helper) - Phase 3
 
 ---
 
@@ -999,22 +1002,22 @@ Create `cropper/src/test/resources/`:
 **Goal:** Ensure configuration validation is bulletproof
 
 4. ✅ `CropImageOptionsTest.kt` (33 tests) - All validation rules, boundary values, Parcelable
-5. ✅ `CropExceptionTest.kt` (11 tests) - Exception hierarchy, messages, serialization
+5. ✅ `CropExceptionTest.kt` (8 tests) - Exception hierarchy, messages, serialization
 6. ✅ `ParcelableUtilsTest.kt` (24 tests) - Bundle/Intent extraction, type safety
 
-**Deliverable:** ✅ Configuration cannot be misconfigured, exceptions well-tested (48 total tests)
+**Deliverable:** ✅ Configuration cannot be misconfigured, exceptions well-tested (65 total tests)
 
 ---
 
-### Phase 3: Async Operations (Week 2)
+### Phase 3: Async Operations ✅ **COMPLETE**
 **Goal:** Async jobs handle errors gracefully
 
-7. Add `kotlinx-coroutines-test` dependency
-8. Create `TestCoroutineExtensions.kt` helper
-9. `BitmapLoadingWorkerJobTest.kt` (3-4 days)
-10. `BitmapCroppingWorkerJobTest.kt` (4 days)
+7. ✅ `kotlinx-coroutines-test` dependency (already added in Phase 1)
+8. ✅ `TestCoroutineExtensions.kt` helper - CoroutineTestRule for dispatcher setup
+9. ✅ `BitmapLoadingWorkerJobTest.kt` (19 tests) - Loading, EXIF, density, lifecycle, errors
+10. ✅ `BitmapCroppingWorkerJobTest.kt` (14 tests) - URI/Bitmap cropping, memory, lifecycle, errors
 
-**Deliverable:** All async operations tested with cancellation, errors, lifecycle
+**Deliverable:** ✅ All async operations tested with cancellation, errors, lifecycle (33 total tests)
 
 ---