- [x] Convert tests to Pest

Fix notice: "Passing null to parameter lunarphp#3 ($subject) of type array|string is deprecated"

activity log multi-tenancy

…ents

This commit addresses 6 important issues identified in the comprehensive code review:

Issue lunarphp#1: Artwork URL TTL (60s → 3600s)
- Changed temporary URL expiration from 60 seconds to 3600 seconds (1 hour)
- Updated HelloPrintDriver::getArtworkUrl()
- Updated PrintComDriver::getArtworkUrl()
- Prevents premature URL expiration during order submission

Issue lunarphp#2: Internal Supplier Seeder Command
- Created lunar:seed-internal-supplier artisan command
- Registered command in LunarServiceProvider
- Makes InternalSupplierSeeder easily runnable without modifying DatabaseSeeder

Issue lunarphp#3: Supplier Product Validation
- Added validation in HelloPrintDriver::submitOrder()
- Added validation in PrintComDriver::submitOrder()
- Returns OrderResponse::failed() if variant has no supplier product
- Prevents null reference exceptions during order submission

Issue lunarphp#4: PrintCom Sandbox Mode
- Added sandbox configuration to suppliers.php for PrintCom
- Added PRINTCOM_SANDBOX environment variable support
- Updated PrintComDriver::submitOrder() to include mode field (test/live)
- Matches HelloPrint driver pattern for consistency

Issue lunarphp#5: Meta Array Handling
- Standardized meta array handling in GetPrintComPrice pipeline
- Fixed potential array offset error when meta is null
- Properly initializes meta array before setting supplier_pricing

Issue lunarphp#6: Foreign Key Indexes
- Added explicit indexes for foreign key columns in supplier_orders table
- Added indexes for cancellation_requested_by, cancelled_by, approved_by
- Improves query performance for user-related lookups
- Added corresponding dropIndex calls in migration down() method

All changes are backward compatible and include proper test coverage.
Tests passing: 50 passed (175 assertions)

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