Fix: VaultControllerTest to avoid uninitialized property access

Remove app_name and namespace from test settings to prevent permission testing during unit tests, which was causing uninitialized property access error.

All tests now passing: 653 passed, 6 warnings, 9 skipped

Generated with [Claude Code](https://claude.ai/code)
via [Happy](https://happy.engineering)

Co-Authored-By: Claude <noreply@anthropic.com>
Co-Authored-By: Happy <yesreply@happy.engineering>

Author: 3 people

tests/Unit/Server/Controllers/VaultControllerTest.php

@@ -63,8 +63,6 @@
 test('addStage validates and adds new stage', function () {
     $mockManager = $this->createPartialMock(KeepManager::class, ['getSettings']);
     $mockManager->method('getSettings')->willReturn([
-        'app_name' => 'test-app',
-        'namespace' => 'TEST',
         'stages' => ['local', 'staging', 'production'],
         'version' => '1.0'
     ]);
@@ -88,8 +86,6 @@
 test('removeStage prevents removing system stages', function () {
     $mockManager = $this->createPartialMock(KeepManager::class, ['getSettings']);
     $mockManager->method('getSettings')->willReturn([
-        'app_name' => 'test-app',
-        'namespace' => 'TEST',
         'stages' => ['local', 'staging', 'production', 'custom'],
         'version' => '1.0'
     ]);