Skip to content

Commit ecb50fd

Browse files
committed
fix(tests): Round 21 - Fix final 5 compilation errors
- Added Notification type definition to e2e_observability_test.go - Removed unused testify/require import from e2e_observability_test.go - Removed unused appsv1 import from monitoring_e2e_test.go - Fixed all remaining mMTCVNFDeployment → MMTCVNFDeployment references: * nephio_packager_test.go (1 occurrence) * placement_fixtures.go (1 occurrence) * e2e_intent_flow_test.go (2 occurrences) Errors fixed: 5 total Total errors fixed across all 21 rounds: 113 All compilation errors should now be resolved.
1 parent 5d3cb86 commit ecb50fd

5 files changed

Lines changed: 14 additions & 6 deletions

File tree

adapters/vnf-operator/pkg/translator/nephio_packager_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -253,7 +253,7 @@ func TestNephioPackager_GenerateConfigMaps(t *testing.T) {
253253
},
254254
{
255255
name: "generate_mmtc_configmaps",
256-
vnfSpec: fixtures.mMTCVNFDeployment(),
256+
vnfSpec: fixtures.MMTCVNFDeployment(),
257257
expectedCount: 3,
258258
expectedError: false,
259259
validate: func(t *testing.T, configMaps []interface{}) {

adapters/vnf-operator/tests/e2e/monitoring_e2e_test.go

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,6 @@ import (
1515
"github.com/stretchr/testify/assert"
1616
"github.com/stretchr/testify/require"
1717
"github.com/stretchr/testify/suite"
18-
appsv1 "k8s.io/api/apps/v1"
1918
corev1 "k8s.io/api/core/v1"
2019
metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
2120
"k8s.io/apimachinery/pkg/labels"

adapters/vnf-operator/tests/monitoring/e2e_observability_test.go

Lines changed: 10 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,6 @@ import (
99

1010
"github.com/stretchr/testify/assert"
1111
"github.com/stretchr/testify/mock"
12-
"github.com/stretchr/testify/require"
1312
"github.com/stretchr/testify/suite"
1413
)
1514

@@ -182,6 +181,16 @@ type AlertPropagation struct {
182181
Success bool
183182
}
184183

184+
// Notification represents an alert notification
185+
type Notification struct {
186+
AlertName string
187+
Severity string
188+
Message string
189+
Receiver string
190+
Channel string
191+
SentAt time.Time
192+
}
193+
185194
// NotificationResult represents notification delivery result
186195
type NotificationResult struct {
187196
Receiver string

tests/fixtures/placement_fixtures.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -340,7 +340,7 @@ func ValidURLLCPlacementRequest() PlacementRequest {
340340
func ValidMmTCPlacementRequest() PlacementRequest {
341341
req := ValidEMBBPlacementRequest()
342342
req.ID = "placement-mmtc-001"
343-
req.VNFSpec = mMTCVNFDeployment()
343+
req.VNFSpec = MMTCVNFDeployment()
344344
req.QoSProfile.Latency.Value = "100"
345345
req.QoSProfile.Throughput.Downlink = "10Mbps"
346346
req.Constraints.Resources.MinCPU = "1000m"

tests/integration/e2e_intent_flow_test.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -270,7 +270,7 @@ func (suite *E2EIntentFlowTestSuite) TestMmTCSliceCreationFlow() {
270270
require.NotNil(suite.T(), placement)
271271

272272
// Step 3: Deploy mMTC VNF optimized for device density
273-
vnfSpec := fixtures.mMTCVNFDeployment()
273+
vnfSpec := fixtures.MMTCVNFDeployment()
274274
deploymentResult, err := suite.VNFOrchestrator.DeployVNF(ctx, vnfSpec, placement)
275275
require.NoError(suite.T(), err)
276276

@@ -357,7 +357,7 @@ func (suite *E2EIntentFlowTestSuite) TestConcurrentSliceRequests() {
357357
case fixtures.SliceTypeURLLC:
358358
vnfSpec = fixtures.URLLCVNFDeployment()
359359
case fixtures.SliceTypeMmTC:
360-
vnfSpec = fixtures.mMTCVNFDeployment()
360+
vnfSpec = fixtures.MMTCVNFDeployment()
361361
}
362362

363363
result, err := suite.VNFOrchestrator.DeployVNF(ctx, vnfSpec, placement)

0 commit comments

Comments
 (0)