Skip to content

Commit

Permalink
Merge pull request #2315 from xueli181114/master
Browse files Browse the repository at this point in the history
OCM-9954 | test: update id:57102 to also check replicas number when edit label/taint
  • Loading branch information
openshift-merge-bot[bot] authored Aug 2, 2024
2 parents dd05a9f + e75a924 commit 13e9759
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion tests/e2e/test_rosacli_machine_pool.go
Original file line number Diff line number Diff line change
Expand Up @@ -1014,7 +1014,7 @@ var _ = Describe("Edit machinepool",
"--labels", common.ReplaceCommaSpaceWithComma(workerPool.Labels),
"--taints", common.ReplaceCommaSpaceWithComma(workerPool.Taints),
)

originalReplicas := workerPool.Replicas
for key, flags := range reqFlags {
By("Edit machinepools to the cluster")
_, err := machinePoolService.EditMachinePool(clusterID, con.DefaultClassicWorkerPool, flags...)
Expand All @@ -1029,20 +1029,23 @@ var _ = Describe("Edit machinepool",
Equal(strings.Join(common.ParseCommaSeparatedStrings(labels_1), ", ")))
Expect(out.Machinepool(con.DefaultClassicWorkerPool).Taints).To(
Equal(strings.Join(common.ParseCommaSeparatedStrings(taints_1), ", ")))
Expect(out.Machinepool(con.DefaultClassicWorkerPool).Replicas).To(Equal(originalReplicas))
}

if key == "empty_label_value" {
Expect(out.Machinepool(con.DefaultClassicWorkerPool).Labels).To(
Equal(strings.Join(common.ParseCommaSeparatedStrings(labels_2), ", ")))
Expect(out.Machinepool(con.DefaultClassicWorkerPool).Taints).To(
Equal(strings.Join(common.ParseCommaSeparatedStrings(taints_2), ", ")))
Expect(out.Machinepool(con.DefaultClassicWorkerPool).Replicas).To(Equal(originalReplicas))
}

if key == "empty_label_and_taint" {
Expect(out.Machinepool(con.DefaultClassicWorkerPool).Labels).To(
Equal(""))
Expect(out.Machinepool(con.DefaultClassicWorkerPool).Taints).To(
Equal(""))
Expect(out.Machinepool(con.DefaultClassicWorkerPool).Replicas).To(Equal(originalReplicas))
}
}
})
Expand Down

0 comments on commit 13e9759

Please sign in to comment.