Skip to content

Commit d9b8a8d

Browse files
authored
Add TestCLIStatus to integration tests and fix assertions (#1368)
- TestCLIStatus isn't in the integration test. - Update the output to align with the current output. - Follow up #1193 --------- Signed-off-by: ChengHao Yang <17496418+tico88612@users.noreply.github.com>
1 parent 490cb69 commit d9b8a8d

2 files changed

Lines changed: 5 additions & 4 deletions

File tree

Makefile

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -193,6 +193,7 @@ integration: init-block
193193
{ \
194194
exit_code=0; \
195195
CLITEST_LOG_ROOT=$(LOG_ROOT) && export CLITEST_LOG_ROOT ; \
196+
$(SWIFT) test -c $(BUILD_CONFIGURATION) $(SWIFT_CONFIGURATION) --filter TestCLIStatus || exit_code=1 ; \
196197
$(SWIFT) test -c $(BUILD_CONFIGURATION) $(SWIFT_CONFIGURATION) --filter TestCLIVersion || exit_code=1 ; \
197198
$(SWIFT) test -c $(BUILD_CONFIGURATION) $(SWIFT_CONFIGURATION) --filter TestCLINetwork || exit_code=1 ; \
198199
$(SWIFT) test -c $(BUILD_CONFIGURATION) $(SWIFT_CONFIGURATION) --filter TestCLIRunLifecycle || exit_code=1 ; \

Tests/CLITests/Subcommands/System/TestCLIStatus.swift

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -52,10 +52,10 @@ final class TestCLIStatus: CLITest {
5252
let fullOutput = lines.joined(separator: "\n")
5353
#expect(fullOutput.contains("status"))
5454
#expect(fullOutput.contains("running"))
55-
#expect(fullOutput.contains("app-root"))
56-
#expect(fullOutput.contains("install-root"))
57-
#expect(fullOutput.contains("apiserver-version"))
58-
#expect(fullOutput.contains("apiserver-commit"))
55+
#expect(fullOutput.contains("appRoot"))
56+
#expect(fullOutput.contains("installRoot"))
57+
#expect(fullOutput.contains("apiserver.version"))
58+
#expect(fullOutput.contains("apiserver.commit"))
5959

6060
_ = data // silence unused warning if assertions short-circuit
6161
}

0 commit comments

Comments
 (0)