fix(tests): Round 17 - Fix workflow template and compilation errors

- Fixed workflow timeout-minutes template (removed fromJson)
- Fixed fixtures.mMTCVNFDeployment → MMTCVNFDeployment capitalization
- Fixed O2DMSClient struct field names (BaseURL→Endpoint, AuthToken→Token, HTTPClient→httpClient, Timeout/RetryCount→maxRetries/retryDelay)
- Removed unused metav1 import from deployment_controller_test.go
- Added missing methods to MockK8sClient (SubResource, GroupVersionKindFor, IsObjectNamespaced)
- Added MockSubResourceClient implementation

Errors fixed: 8 total
- 1 workflow template error
- 2 fixture capitalization errors
- 5 O2DMSClient field errors

Author: thc1006

.github/workflows/test.yml

@@ -222,7 +222,7 @@ jobs:
 
     - name: Run unit tests with coverage
       working-directory: ${{ matrix.component.path }}
-      timeout-minutes: ${{ fromJson(matrix.component.timeout) }}
+      timeout-minutes: ${{ matrix.component.timeout }}
       run: |
         echo "Running unit tests for ${{ matrix.component.name }}"
 

adapters/vnf-operator/pkg/controller/deployment_controller_test.go

@@ -7,7 +7,6 @@ import (
 
 	"github.com/stretchr/testify/assert"
 	"github.com/stretchr/testify/mock"
-	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
 	"k8s.io/apimachinery/pkg/runtime"
 	"k8s.io/apimachinery/pkg/types"
 	ctrl "sigs.k8s.io/controller-runtime"
@@ -140,7 +139,7 @@ func TestVNFDeploymentReconciler_Reconcile(t *testing.T) {
 		},
 		{
 			name:           "mmtc_slice_deployment",
-			vnfDeployment:  fixtures.mMTCVNFDeployment(),
+			vnfDeployment:  fixtures.MMTCVNFDeployment(),
 			expectedResult: reconcile.Result{RequeueAfter: time.Minute * 5},
 			expectedError:  false,
 			validateCalls: func(t *testing.T, mockK8s *mocks.MockK8sClient, mockDMS *MockDMSClient, mockGitOps *MockGitOpsClient) {

adapters/vnf-operator/pkg/dms/o2_client_test.go

@@ -118,11 +118,11 @@ func TestO2DMSClient_GetInventory(t *testing.T) {
 
 			// Create client with mock
 			client := &O2DMSClient{
-				BaseURL:    "http://test-o2dms:8080",
-				AuthToken:  "test-token",
-				HTTPClient: mockHTTP,
-				Timeout:    time.Second * 30,
-				RetryCount: 3,
+				Endpoint:   "http://test-o2dms:8080",
+				Token:      "test-token",
+				httpClient: mockHTTP,
+				maxRetries: 3,
+				retryDelay: time.Second,
 			}
 
 			// Execute test

tests/mocks/k8s_client_mock.go

@@ -105,6 +105,38 @@ func (m *MockK8sClient) RESTMapper() meta.RESTMapper {
 	return nil
 }
 
+func (m *MockK8sClient) SubResource(subResource string) client.SubResourceClient {
+	return &MockSubResourceClient{}
+}
+
+// GroupVersionKindFor returns GVK for the given object
+func (m *MockK8sClient) GroupVersionKindFor(obj runtime.Object) (schema.GroupVersionKind, error) {
+	return schema.GroupVersionKind{}, nil
+}
+
+// IsObjectNamespaced returns true if the object is namespaced
+func (m *MockK8sClient) IsObjectNamespaced(obj runtime.Object) (bool, error) {
+	return true, nil
+}
+
+type MockSubResourceClient struct{}
+
+func (m *MockSubResourceClient) Get(ctx context.Context, obj client.Object, subResource client.Object, opts ...client.SubResourceGetOption) error {
+	return nil
+}
+
+func (m *MockSubResourceClient) Create(ctx context.Context, obj client.Object, subResource client.Object, opts ...client.SubResourceCreateOption) error {
+	return nil
+}
+
+func (m *MockSubResourceClient) Update(ctx context.Context, obj client.Object, opts ...client.SubResourceUpdateOption) error {
+	return nil
+}
+
+func (m *MockSubResourceClient) Patch(ctx context.Context, obj client.Object, patch client.Patch, opts ...client.SubResourcePatchOption) error {
+	return nil
+}
+
 type MockStatusWriter struct{}
 
 func (m *MockStatusWriter) Create(ctx context.Context, obj client.Object, subResource client.Object, opts ...client.SubResourceCreateOption) error {