chore: delete bad tests

spinifex/admin/telemetry_test.go

@@ -99,9 +99,3 @@ func TestSendTelemetry_HandlesUnreachableServer(t *testing.T) {
 	// URL pointing to a port that's not listening
 	SendTelemetry(ctx, TelemetryPayload{MachineID: "unreachable-test", URL: "http://127.0.0.1:1"})
 }
-
-func TestFirstMAC_ReturnsNonEmpty(t *testing.T) {
-	t.Parallel()
-	mac := firstMAC()
-	assert.NotEmpty(t, mac, "expected at least one MAC address")
-}

spinifex/gateway/ec2/instance/RunInstances_test.go

@@ -205,31 +205,10 @@ func TestParseRunInstances(t *testing.T) {
 			},
 			want: errors.New(awserrors.ErrorMissingParameter),
 		},
-
-		// Successful test
-		{
-			name: "ValidTest",
-			input: &ec2.RunInstancesInput{
-				ImageId:          defaults.ImageId,
-				InstanceType:     defaults.InstanceType,
-				MinCount:         aws.Int64(1),
-				MaxCount:         aws.Int64(1),
-				KeyName:          defaults.KeyName,
-				SecurityGroupIds: defaults.SecurityGroupIds,
-				SubnetId:         defaults.SubnetId,
-			},
-			want: nil,
-		},
 	}
 
 	for _, test := range tests {
 		t.Run(test.name, func(t *testing.T) {
-			// Skip the valid test as it requires full daemon infrastructure
-			// These tests are covered by the integration tests in service_nats_test.go
-			if test.want == nil {
-				t.Skip("Skipping valid test - requires full daemon infrastructure")
-			}
-
 			// For validation tests, we can pass nil conn since validation happens before NATS call
 			response, err := RunInstances(test.input, nil, nil, "123456789012", nil, 1)
 

spinifex/objectstore/ensurebucket_test.go

@@ -102,7 +102,3 @@ func TestS3ObjectStore_EnsureBucket_BackendErrorPropagates(t *testing.T) {
 
 	require.Error(t, newS3Store(t, srv.URL).EnsureBucket("ecr-000000000001"))
 }
-
-func TestMemoryObjectStore_EnsureBucketNoop(t *testing.T) {
-	require.NoError(t, NewMemoryObjectStore().EnsureBucket("any"))
-}