diff --git a/spinifex/admin/telemetry_test.go b/spinifex/admin/telemetry_test.go index c0e045136..9a4bacb06 100644 --- a/spinifex/admin/telemetry_test.go +++ b/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") -} diff --git a/spinifex/gateway/ec2/instance/RunInstances_test.go b/spinifex/gateway/ec2/instance/RunInstances_test.go index be5d4ecc0..ee339b322 100644 --- a/spinifex/gateway/ec2/instance/RunInstances_test.go +++ b/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) diff --git a/spinifex/objectstore/ensurebucket_test.go b/spinifex/objectstore/ensurebucket_test.go index 1bd938b5a..59b954b75 100644 --- a/spinifex/objectstore/ensurebucket_test.go +++ b/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")) -}