Skip to content

[5.5] Update BuildToolTests.testBuildCompleteMessage #3498

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
May 14, 2021

Conversation

yim-lee
Copy link
Contributor

@yim-lee yim-lee commented May 13, 2021

#3496 for 5.5

Motivation:
The build output on Linux has changed. Causing self hosted Linux test to fail ([example](https://ci.swift.org/job/swift-package-manager-self-hosted-Linux-smoke-test/2051/)):

```
21:29:49 Test Case 'BuildToolTests.testBuildCompleteMessage' started at 2021-05-13 04:29:06.495
21:29:49
/home/buildnode/jenkins/workspace/swift-package-manager-self-hosted-Linux-smoke-test/branch-main/swiftpm/Tests/CommandsTests/BuildToolTests.swift:256: error: BuildToolTests.testBuildCompleteMessage : XCTAssertTrue failed - [1/4] Compiling Bar Bar.swift
21:29:49 [3/6] Merging module Bar
21:29:49 [5/7] Wrapping AST for Bar for debugging
21:29:49 [6/9] Compiling Foo Foo.swift
21:29:49 [7/9] Compiling Foo main.swift
21:29:49 [9/11] Merging module Foo
21:29:49 [11/12] Wrapping AST for Foo for debugging
21:29:49 [12/12] Linking Foo
21:29:49 [12/12] Build complete!
21:29:49
21:29:49 Test Case 'BuildToolTests.testBuildCompleteMessage' failed (5.24 seconds)
```

Modification:
Update expected output in `BuildToolTests.testBuildCompleteMessage`.
@yim-lee
Copy link
Contributor Author

yim-lee commented May 13, 2021

@swift-ci please test

@tomerd
Copy link
Contributor

tomerd commented May 13, 2021

@swift-ci please test

@tomerd tomerd added the 5.5 label May 13, 2021
@yim-lee yim-lee merged commit 35bb9ad into swiftlang:release/5.5 May 14, 2021
@yim-lee yim-lee deleted the update-buildtooltest-5.5 branch May 14, 2021 00:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants