From c44712e8130d602f6696336b67a5d69e3b2fcf9f Mon Sep 17 00:00:00 2001 From: LordMathis Date: Tue, 21 Oct 2025 22:15:12 +0200 Subject: [PATCH] Remove redundant instance manager tests --- pkg/manager/manager_test.go | 15 ------- pkg/manager/operations_test.go | 80 ---------------------------------- 2 files changed, 95 deletions(-) diff --git a/pkg/manager/manager_test.go b/pkg/manager/manager_test.go index ed0ed25..34f63cf 100644 --- a/pkg/manager/manager_test.go +++ b/pkg/manager/manager_test.go @@ -12,21 +12,6 @@ import ( "testing" ) -func TestNewInstanceManager(t *testing.T) { - mgr := createTestManager() - if mgr == nil { - t.Fatal("NewInstanceManager returned nil") - } - - instances, err := mgr.ListInstances() - if err != nil { - t.Fatalf("ListInstances failed: %v", err) - } - if len(instances) != 0 { - t.Errorf("Expected empty instance list, got %d instances", len(instances)) - } -} - func TestManager_PersistsAndLoadsInstances(t *testing.T) { tempDir := t.TempDir() cfg := createPersistenceConfig(tempDir) diff --git a/pkg/manager/operations_test.go b/pkg/manager/operations_test.go index dc3ae1e..976fb39 100644 --- a/pkg/manager/operations_test.go +++ b/pkg/manager/operations_test.go @@ -192,32 +192,6 @@ func TestDeleteInstance_ReleasesPort(t *testing.T) { } } -func TestGetInstance(t *testing.T) { - manager := createTestManager() - - options := &instance.Options{ - BackendOptions: backends.Options{ - BackendType: backends.BackendTypeLlamaCpp, - LlamaServerOptions: &backends.LlamaServerOptions{ - Model: "/path/to/model.gguf", - }, - }, - } - - created, err := manager.CreateInstance("test-instance", options) - if err != nil { - t.Fatalf("CreateInstance failed: %v", err) - } - - retrieved, err := manager.GetInstance("test-instance") - if err != nil { - t.Fatalf("GetInstance failed: %v", err) - } - if retrieved.Name != created.Name { - t.Errorf("Expected name %q, got %q", created.Name, retrieved.Name) - } -} - func TestUpdateInstance(t *testing.T) { manager := createTestManager() @@ -255,60 +229,6 @@ func TestUpdateInstance(t *testing.T) { } } -func TestListInstances(t *testing.T) { - manager := createTestManager() - - options := &instance.Options{ - BackendOptions: backends.Options{ - BackendType: backends.BackendTypeLlamaCpp, - LlamaServerOptions: &backends.LlamaServerOptions{ - Model: "/path/to/model.gguf", - }, - }, - } - - _, err := manager.CreateInstance("test-instance", options) - if err != nil { - t.Fatalf("CreateInstance failed: %v", err) - } - - instances, err := manager.ListInstances() - if err != nil { - t.Fatalf("ListInstances failed: %v", err) - } - if len(instances) != 1 { - t.Errorf("Expected 1 instance, got %d", len(instances)) - } -} - -func TestDeleteInstance(t *testing.T) { - manager := createTestManager() - - options := &instance.Options{ - BackendOptions: backends.Options{ - BackendType: backends.BackendTypeLlamaCpp, - LlamaServerOptions: &backends.LlamaServerOptions{ - Model: "/path/to/model.gguf", - }, - }, - } - - _, err := manager.CreateInstance("test-instance", options) - if err != nil { - t.Fatalf("CreateInstance failed: %v", err) - } - - err = manager.DeleteInstance("test-instance") - if err != nil { - t.Fatalf("DeleteInstance failed: %v", err) - } - - _, err = manager.GetInstance("test-instance") - if err == nil { - t.Error("Instance should not exist after deletion") - } -} - func TestInstanceOperations_NonExistentInstance(t *testing.T) { manager := createTestManager()