Skip to content

Commit d3d8bfd

Browse files
authored
SANDBOX-1392: member-pair Drop dependency on Che instance (#1197)
* SANDBOX-1392: Drop dependency on Che instance , change the toolchain-status test to use console instead where che was required and used to determine the member status. Signed-off-by: Feny Mehta <fbm3307@gmail.com>
1 parent 26c2410 commit d3d8bfd

1 file changed

Lines changed: 8 additions & 8 deletions

File tree

test/e2e/toolchainstatus_test.go

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -20,22 +20,22 @@ func TestToolchainStatusUnready(t *testing.T) {
2020
VerifyToolchainStatus(t, hostAwait, memberAwait)
2121

2222
t.Run("verify updated toolchainconfig is synced - go to unready", func(t *testing.T) {
23-
// set the che required flag to true to force an error on the memberstatus (che is not installed in e2e test environments)
24-
memberConfigurationWithCheRequired := testconfig.ModifyMemberOperatorConfigObj(memberAwait.GetMemberOperatorConfig(t), testconfig.Che().Required(true))
25-
hostAwait.UpdateToolchainConfig(t, testconfig.Members().Default(memberConfigurationWithCheRequired.Spec))
23+
// set an invalid console route name to force an error on the memberstatus (console route won't be found)
24+
memberConfigurationWithInvalidConsole := testconfig.ModifyMemberOperatorConfigObj(memberAwait.GetMemberOperatorConfig(t), testconfig.Console().RouteName("nonexistent-console-route"))
25+
hostAwait.UpdateToolchainConfig(t, testconfig.Members().Default(memberConfigurationWithInvalidConsole.Spec))
2626

2727
err := memberAwait.WaitForMemberStatus(t,
2828
wait.UntilMemberStatusHasConditions(wait.ToolchainStatusComponentsNotReady("[routes]")))
29-
require.NoError(t, err, "failed while waiting for MemberStatus to contain error due to che being required")
29+
require.NoError(t, err, "failed while waiting for MemberStatus to contain error due to invalid console route")
3030

3131
_, err = hostAwait.WaitForToolchainStatus(t,
3232
wait.UntilToolchainStatusHasConditions(wait.ToolchainStatusComponentsNotReady("[members]"), wait.ToolchainStatusUnreadyNotificationNotCreated()))
33-
require.NoError(t, err, "failed while waiting for ToolchainStatus to contain error due to che being required")
33+
require.NoError(t, err, "failed while waiting for ToolchainStatus to contain error due to invalid console route")
3434

3535
t.Run("verify member and toolchain status go back to ready", func(t *testing.T) {
36-
// change che required flag back to true to resolve the error on the memberstatus
37-
memberConfigurationWithCheRequired = testconfig.ModifyMemberOperatorConfigObj(memberAwait.GetMemberOperatorConfig(t), testconfig.Che().Required(false))
38-
hostAwait.UpdateToolchainConfig(t, testconfig.Members().Default(memberConfigurationWithCheRequired.Spec))
36+
// change console route name back to default to resolve the error on the memberstatus
37+
memberConfigurationWithValidConsole := testconfig.ModifyMemberOperatorConfigObj(memberAwait.GetMemberOperatorConfig(t), testconfig.Console().RouteName("console"))
38+
hostAwait.UpdateToolchainConfig(t, testconfig.Members().Default(memberConfigurationWithValidConsole.Spec))
3939

4040
VerifyMemberStatus(t, memberAwait, consoleURL)
4141
VerifyToolchainStatus(t, hostAwait, memberAwait)

0 commit comments

Comments
 (0)