Skip to content

Commit 20bf23b

Browse files
based on the e2e test changed it
1 parent da615e5 commit 20bf23b

1 file changed

Lines changed: 5 additions & 4 deletions

File tree

integration/ocm_integration_test.go

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -91,10 +91,10 @@ func (s *integrationTestSuite) TestAddonStatusReporting() {
9191
addonsv1alpha1.Available: string(metav1.ConditionTrue),
9292
addonsv1alpha1.Installed: string(metav1.ConditionTrue),
9393
}
94-
for _, condition := range res.StatusConditions {
94+
for ind, condition := range res.StatusConditions {
9595
expectedVal, found := expectedConditions[condition.StatusType]
9696
s.Require().True(found && expectedVal == string(condition.StatusValue))
97-
s.Require().NotEmpty(condition.Message)
97+
s.Require().Equal(addon.Status.Conditions[ind].Message, condition.Message)
9898
}
9999
})
100100

@@ -119,16 +119,17 @@ func (s *integrationTestSuite) TestAddonStatusReporting() {
119119
s.Require().Equal(addon.Name, res.AddonID)
120120
s.Require().Equal(addon.Spec.Version, res.AddonVersion)
121121
s.Require().Equal(len(res.StatusConditions), 3)
122+
122123
expectedConditions := map[string]string{
123124
addonsv1alpha1.Available: string(metav1.ConditionTrue),
124125
addonsv1alpha1.Installed: string(metav1.ConditionTrue),
125126
addonsv1alpha1.Paused: string(metav1.ConditionTrue),
126127
}
127128

128-
for _, condition := range res.StatusConditions {
129+
for ind, condition := range res.StatusConditions {
129130
expectedVal, found := expectedConditions[condition.StatusType]
130131
s.Require().True(found && expectedVal == string(condition.StatusValue))
131-
s.Require().NotEmpty(condition.Message)
132+
s.Require().Equal(addon.Status.Conditions[ind].Message, condition.Message)
132133
}
133134

134135
// Assert that the addon object correctly updates its status block with the

0 commit comments

Comments
 (0)