Skip to content

Conversation

@0xcaff
Copy link

@0xcaff 0xcaff commented Nov 23, 2019

Tests are failing on the async branch. This is the first step to fixing the failures.

It seems the errors from the rust compiler changed in a recent version, but the expectations for errors weren't updated. This PR updates the expectations.

This leaves two tests failing. It seems that sometimes environment specific file paths get used in the comparison of compiler errors causing it to always fail for a couple of tests:

This is likely caused by an upstream bug in compiletest (the crate). They may be already fixed upstream (in the rust project) and could be fixed fixed when this lands (in the crate): Manishearth/compiletest-rs#201

@jebrosen
Copy link
Collaborator

I'm hesitant to make these the "desired" errors; it's only due to an accident that I had trouble fixing (see 9c4167a#diff-e00e6dba1cc1a7479917f658ea6c9257R85 )

For the path issues, I started to fix this in compiletest in Manishearth/compiletest-rs#198 (comment) but didn't like where it was going. trybuild has a better solution in the form of $WORKSPACE normalization, but @SergioBenitez has tried it and run into some issues.

For the time being I'm inclined to disable the UI tests on the async branch until things have settled.

@0xcaff
Copy link
Author

0xcaff commented Nov 23, 2019

Oops, missed that context. +1 to disabling tests until things stabilize a bit more with compiletest. These tests are flaky. It seems like this fix doesn't work. The tests are still failing because line numbers for errors changed between the last nightly versions (Nov 22nd and Nov 21st).

@0xcaff 0xcaff closed this Nov 23, 2019
@jebrosen
Copy link
Collaborator

I ended up going ahead and updating these errors and others in addition to disabling UI tests, in the interest of having working CI for any future PRs to the async branch. I still hope to fix the error message location, but there are some strange interactions between async fn and function pointer syntax that are making it difficult.

@0xcaff
Copy link
Author

0xcaff commented Nov 24, 2019

Woo! Awesome! Looking forward to playing with the async stuff next.

@SergioBenitez SergioBenitez added the pr: closed This pull request was not merged label Feb 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr: closed This pull request was not merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants