Skip to content

Commit 8990b00

Browse files
authored
Merge pull request #1260 from RusJaI/4.3.x
Modify integration tests regarding to the improvement that allowing an API and a Common policy to have same name and version
2 parents 2e57d34 + e7ef87d commit 8990b00

File tree

8 files changed

+11
-7
lines changed

8 files changed

+11
-7
lines changed

import-export-cli/integration/testdata/TestArtifactDirectory/DevFirstUpdatedSampleCaseArtifacts/customAddLogMessage_v1.j2 renamed to import-export-cli/integration/testdata/TestArtifactDirectory/DevFirstUpdatedSampleCaseArtifacts/customAddLogMessage_v1_api.j2

File renamed without changes.

import-export-cli/integration/testdata/TestArtifactDirectory/DevFirstUpdatedSampleCaseArtifacts/customAddLogMessage_v1.yaml renamed to import-export-cli/integration/testdata/TestArtifactDirectory/DevFirstUpdatedSampleCaseArtifacts/customAddLogMessage_v1_api.yaml

File renamed without changes.

import-export-cli/integration/testdata/TestArtifactDirectory/DynamicDataSampleCaseArtifacts/dynamicAddLogMessage_v1.j2 renamed to import-export-cli/integration/testdata/TestArtifactDirectory/DynamicDataSampleCaseArtifacts/dynamicAddLogMessage_v1_api.j2

File renamed without changes.

import-export-cli/integration/testdata/TestArtifactDirectory/DynamicDataSampleCaseArtifacts/dynamicAddLogMessage_v1.yaml renamed to import-export-cli/integration/testdata/TestArtifactDirectory/DynamicDataSampleCaseArtifacts/dynamicAddLogMessage_v1_api.yaml

File renamed without changes.

import-export-cli/integration/testdata/TestArtifactDirectory/DynamicDataSampleCaseArtifacts/dynamicDataSubstitutedAddLogMessage_v1.j2 renamed to import-export-cli/integration/testdata/TestArtifactDirectory/DynamicDataSampleCaseArtifacts/dynamicDataSubstitutedAddLogMessage_v1_api.j2

File renamed without changes.

import-export-cli/integration/testutils/devFirst_testUtils.go

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -386,14 +386,15 @@ func ValidateAPIWithUpdatedSequenceIsExported(t *testing.T, args *InitTestArgs,
386386
base.Unzip(relativePath, exportedPath)
387387

388388
// Check whether the exported operation policy is equivalent to the latest operation policy
389-
exportedApiSequencePath := relativePath + TestDefaultExtractedFileName + DevFirstSampleCaseDestPolicy1PathSuffix
389+
exportedApiSequencePath := relativePath + TestDefaultExtractedFileName + DevFirstSampleCaseDestExportedPolicy1PathSuffix
390+
390391
lastUpdatedSequencePath, _ := filepath.Abs(DevFirstUpdatedSampleCasePolicy1Path)
391392
isSequenceUpdated := base.IsFileContentIdentical(exportedApiSequencePath, lastUpdatedSequencePath)
392393
base.Log("Exported operation policy is updated", isSequenceUpdated)
393394
assert.Equal(t, true, isSequenceUpdated, "Error while updating the operation policy of API")
394395

395396
// Check whether the exported operation policy definition is equivalent to the latest operation policy definition
396-
exportedApiSequenceDefinitionPath := relativePath + TestDefaultExtractedFileName + DevFirstSampleCaseDestPolicyDefinition1PathSuffix
397+
exportedApiSequenceDefinitionPath := relativePath + TestDefaultExtractedFileName + DevFirstSampleCaseDestExportedPolicyDefinition1PathSuffix
397398
lastUpdatedSequenceDefinitionPath, _ := filepath.Abs(DevFirstUpdatedSampleCasePolicyDefinition1Path)
398399
isSequenceDefinitionUpdated := base.IsFileContentIdentical(exportedApiSequenceDefinitionPath, lastUpdatedSequenceDefinitionPath)
399400
base.Log("Exported operation policy definition is updated", isSequenceDefinitionUpdated)

import-export-cli/integration/testutils/dynamicData_testUtils.go

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -108,6 +108,7 @@ func AddSequenceWithDynamicDataToAPIProject(t *testing.T, args *InitTestArgs) ap
108108
operationPolicies := apim.OperationPolicies{
109109
Request: append(requestPolicies, map[string]interface{}{
110110
"policyName": TestSampleDynamicDataPolicyName,
111+
"policyType": "api",
111112
}),
112113
Response: []string{},
113114
Fault: []string{},

import-export-cli/integration/testutils/testConstants.go

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -190,10 +190,12 @@ const DevSampleCaseMalformedOperationPolicyDefinitionPath = DevFirstSampleCaseMa
190190
const DevSampleCaseMalformedOperationPolicyPath = DevFirstSampleCaseMalformedOperationPolicyArtifactPath + "/customAddLogMessage.j2"
191191
const DevFirstSampleCaseDestPolicy1PathSuffix = PoliciesDirectory + "/customAddLogMessage_v1.j2"
192192
const DevFirstSampleCaseDestPolicy2PathSuffix = PoliciesDirectory + "/customAddLogMessage_v2.j2"
193+
const DevFirstSampleCaseDestExportedPolicy1PathSuffix = PoliciesDirectory + "/customAddLogMessage_v1_api.j2"
193194
const DevFirstSampleCaseDestPolicyDefinition1PathSuffix = PoliciesDirectory + "/customAddLogMessage_v1.yaml"
194195
const DevFirstSampleCaseDestPolicyDefinition2PathSuffix = PoliciesDirectory + "/customAddLogMessage_v2.yaml"
195-
const DevFirstUpdatedSampleCasePolicy1Path = DevFirstUpdatedSampleCaseArtifactPath + "/customAddLogMessage_v1.j2"
196-
const DevFirstUpdatedSampleCasePolicyDefinition1Path = DevFirstUpdatedSampleCaseArtifactPath + "/customAddLogMessage_v1.yaml"
196+
const DevFirstSampleCaseDestExportedPolicyDefinition1PathSuffix = PoliciesDirectory + "/customAddLogMessage_v1_api.yaml"
197+
const DevFirstUpdatedSampleCasePolicy1Path = DevFirstUpdatedSampleCaseArtifactPath + "/customAddLogMessage_v1_api.j2"
198+
const DevFirstUpdatedSampleCasePolicyDefinition1Path = DevFirstUpdatedSampleCaseArtifactPath + "/customAddLogMessage_v1_api.yaml"
197199
const TestSynapseChoreoConnectPolicyDefinitionPath = TestSynapseChoreoConnectPolicyArtifactsPath + "/testSynapseChoreoConnectPolicy.yaml"
198200
const TestSynapseChoreoConnectPolicyPathForSynapseType = TestSynapseChoreoConnectPolicyArtifactsPath + "/testSynapseChoreoConnectPolicy.j2"
199201
const TestSynapseChoreoConnectPolicyPathForChoreoConnectType = TestSynapseChoreoConnectPolicyArtifactsPath + "/testSynapseChoreoConnectPolicy.gotmpl"
@@ -214,9 +216,9 @@ const (
214216

215217
// Constants for sequence of the dynamic data test case
216218
const DynamicDataSampleCaseArtifactPath = "testdata/TestArtifactDirectory/DynamicDataSampleCaseArtifacts"
217-
const DynamicDataInSequence = "dynamicAddLogMessage_v1.j2"
218-
const DynamicDataInSequenceDefinition = "dynamicAddLogMessage_v1.yaml"
219-
const DynamicDataSubstitutedInSequence = DynamicDataSampleCaseArtifactPath + "/dynamicDataSubstitutedAddLogMessage_v1.j2"
219+
const DynamicDataInSequence = "dynamicAddLogMessage_v1_api.j2"
220+
const DynamicDataInSequenceDefinition = "dynamicAddLogMessage_v1_api.yaml"
221+
const DynamicDataSubstitutedInSequence = DynamicDataSampleCaseArtifactPath + "/dynamicDataSubstitutedAddLogMessage_v1_api.j2"
220222

221223
const (
222224
TestSampleDynamicDataOperationTarget = "/menu"

0 commit comments

Comments
 (0)