fixes tests

Author: ljonesfl

.gitignore

@@ -10,6 +10,7 @@ build/
 .phpunit.result.cache
 /examples/test.log
 /examples/bad/test.log
+test.log
 CLAUDE.md
 /cache/
 /.claude/

tests/Mvc/ApplicationTest.php

@@ -25,10 +25,35 @@ protected function setUp() : void
 	{
 		parent::setUp();
 
+		// Clear any previous output buffers to ensure clean state
+		while( ob_get_level() > 0 )
+		{
+			ob_end_clean();
+		}
+
+		// Reset Registry singleton state to avoid pollution between tests
+		$registry = Registry::getInstance();
+		$registry->reset();
+
 		$Ini = new Yaml( './examples/config/neuron.yaml' );
 		$this->App = new Application( "1.0.0", $Ini );
 	}
 
+	protected function tearDown() : void
+	{
+		// Clean up $_SERVER state
+		unset( $_SERVER['HTTP_CONTENT_TYPE'] );
+		unset( $_SERVER['HTTP_AUTHORIZATION'] );
+
+		// Clean up any leftover output buffers that tests may have opened
+		while( ob_get_level() > 0 )
+		{
+			ob_end_clean();
+		}
+
+		parent::tearDown();
+	}
+
 	/**
 	 * @throws \Exception
 	 */
@@ -375,6 +400,10 @@ public function testRoutesPath()
 	public function testBadRoutes()
 	{
 		$this->expectException( \Neuron\Core\Exceptions\Validation::class );
+
+		// Clear Registry to ensure clean state for this test
+		Registry::getInstance()->reset();
+
 		$Ini = new Yaml( './examples/bad/neuron.yaml' );
 		$App = new Application( "1.0.0", $Ini );
 	}

tests/Mvc/Cache/Storage/RedisCacheStorageUnitTest.php

@@ -211,10 +211,16 @@ public function testCustomPrefixIsUsed()
 	public function testIsExpiredReturnsTrueWhenTtlIsNegative()
 	{
 		$MockRedis = $this->createMock( Redis::class );
+
+		// First exists() is called, return false to indicate key doesn't exist
 		$MockRedis->expects( $this->once() )
-			->method( 'ttl' )
+			->method( 'exists' )
 			->with( 'neuron_cache_test_key' )
-			->willReturn( -2 ); // Key doesn't exist
+			->willReturn( 0 ); // Redis exists returns 0 for non-existent keys
+
+		// ttl() should not be called since exists() returns false
+		$MockRedis->expects( $this->never() )
+			->method( 'ttl' );
 
 		$Storage = $this->createStorageWithMock( $MockRedis );
 
@@ -225,6 +231,14 @@ public function testIsExpiredReturnsTrueWhenTtlIsNegative()
 	public function testIsExpiredReturnsFalseWhenTtlIsPositive()
 	{
 		$MockRedis = $this->createMock( Redis::class );
+
+		// First exists() is called, return true to indicate key exists
+		$MockRedis->expects( $this->once() )
+			->method( 'exists' )
+			->with( 'neuron_cache_test_key' )
+			->willReturn( 1 ); // Redis exists returns 1 for existing keys
+
+		// Then ttl() is called and returns positive value
 		$MockRedis->expects( $this->once() )
 			->method( 'ttl' )
 			->with( 'neuron_cache_test_key' )

tests/Mvc/Controllers/CacheInitializationTest.php

@@ -101,24 +101,27 @@ public function testInitializeViewCacheReturnsExistingInstanceFromRegistry()
 	public function testInitializeViewCacheReturnsNullWhenCacheDisabled()
 	{
 		// Setup settings with cache disabled
-		$Settings = $this->createMock( \Neuron\Data\Setting\Source\ISettingSource::class );
-		$Settings->method( 'get' )
+		$SettingSource = $this->createMock( \Neuron\Data\Setting\Source\ISettingSource::class );
+		$SettingSource->method( 'get' )
 			->willReturnMap( [
 				['cache', 'enabled', 'false'],
 				['cache', 'storage', 'file'],
 				['cache', 'path', 'cache/views'],
 				['cache', 'ttl', '3600']
 			] );
-		
+
+		// Wrap the mock source in a proper SettingManager
+		$Settings = new \Neuron\Data\Setting\SettingManager( $SettingSource );
+
 		$Registry = Registry::getInstance();
 		$Registry->set( 'Settings', $Settings );
-		
+
 		// Create test controller
 		$Controller = new CacheInitTestController( new Application() );
-		
+
 		// Test initialization returns null
 		$ViewCache = $Controller->testInitializeViewCache();
-		
+
 		$this->assertNull( $ViewCache );
 		$this->assertNull( $Registry->get( 'ViewCache' ) );
 	}

tests/Mvc/Requests/RequestTest.php

@@ -57,6 +57,13 @@ public function testRequest()
 
 	public function testProcessPayloadSuccess()
 	{
+		// Set required headers for the request
+		setHeaders(
+			[
+				'Content-Type' => 'application/json'
+			]
+		);
+
 		$Request = new Request();
 		$Request->loadFile( 'examples/config/requests/login.yaml' );
 
@@ -144,6 +151,13 @@ public function testProcessPayloadFail()
 
 	public function testRequestPayload()
 	{
+		// Set required headers for the request
+		setHeaders(
+			[
+				'Content-Type' => 'application/json'
+			]
+		);
+
 		$Request = new Request();
 		$Request->loadFile( 'examples/config/requests/login.yaml' );
 
@@ -179,6 +193,13 @@ public function testRequestPayload()
 
 	public function testNoProperties()
 	{
+		// Set required headers for the request
+		setHeaders(
+			[
+				'Content-Type' => 'application/json'
+			]
+		);
+
 		$Request = new Request();
 		$Request->loadFile( 'examples/config/requests/logout.yaml' );