From 989416105b2c7f7044fa53f9d734eaf46a547d44 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Wed, 28 Jan 2026 04:16:31 +0000 Subject: [PATCH] Update unit tests to reflect cache-based minification behavior Co-authored-by: jmiller-moko <230051081+jmiller-moko@users.noreply.github.com> --- tests/unit/AssetMinifierTest.php | 25 ++++++++++++++++++++++--- 1 file changed, 22 insertions(+), 3 deletions(-) diff --git a/tests/unit/AssetMinifierTest.php b/tests/unit/AssetMinifierTest.php index 2974a93..dfdcb32 100644 --- a/tests/unit/AssetMinifierTest.php +++ b/tests/unit/AssetMinifierTest.php @@ -192,22 +192,41 @@ class AssetMinifierTest extends Unit } /** - * Test process assets in development mode + * Test process assets with cache disabled (use non-minified) */ - public function testProcessAssetsDevelopmentMode() + public function testProcessAssetsCacheDisabled() { // Create some minified files file_put_contents($this->testDir . '/test.min.css', 'body{}'); file_put_contents($this->testDir . '/test.min.js', 'var x=1;'); + // When cache is disabled, useNonMinified = true $result = \AssetMinifier::processAssets($this->testDir, true); - $this->assertEquals('development', $result['mode']); + $this->assertEquals('cache-disabled', $result['mode']); $this->assertGreaterThanOrEqual(2, $result['deleted'], 'Should delete minified files'); $this->assertFileDoesNotExist($this->testDir . '/test.min.css'); $this->assertFileDoesNotExist($this->testDir . '/test.min.js'); } + /** + * Test process assets with cache enabled (use minified) + */ + public function testProcessAssetsCacheEnabled() + { + // Create source files + file_put_contents($this->testDir . '/test.css', 'body { margin: 0; }'); + file_put_contents($this->testDir . '/test.js', 'function test() { return true; }'); + + // When cache is enabled, useNonMinified = false + // This would try to minify files in the hardcoded list, which won't match our test files + // So we just verify the mode is set correctly + $result = \AssetMinifier::processAssets($this->testDir, false); + + $this->assertEquals('cache-enabled', $result['mode']); + $this->assertEquals(0, $result['minified'], 'Should not minify test files (not in hardcoded list)'); + } + /** * Test process assets returns error for non-existent directory */