Skip to content
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.

Sign up for GitHub

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

Jump to bottom

PAL test file_io/GetFullPathNameA/test3/paltest_getfullpathnamea_test3 does not pass #42289

Closed
davidwrighton opened this issue Sep 16, 2020 · 2 comments · Fixed by #72492
Closed

PAL test file_io/GetFullPathNameA/test3/paltest_getfullpathnamea_test3 does not pass #42289

davidwrighton opened this issue Sep 16, 2020 · 2 comments · Fixed by #72492
Labels
area-PAL-coreclr
Milestone
7.0.0

Comments

@davidwrighton
Copy link
Member

Description

Run the pal test file_io/GetFullPathNameA/test3/paltest_getfullpathnamea_test3

Note that the following error is produced

ERROR :2: CreateFileA failed to create "/root/helix/work/workitem/testoutputtmp/test_directory../testing.tmp".

I believe this test has been failing forever; however, the test did not properly report failure, so the test never failed in CI before.
@Dotnet-GitSync-Bot Dotnet-GitSync-Bot added area-PAL-coreclr untriaged New issue has not been triaged by the area owner labels Sep 16, 2020
@davidwrighton davidwrighton added this to the 6.0.0 milestone Sep 16, 2020
@davidwrighton davidwrighton removed the untriaged New issue has not been triaged by the area owner label Sep 16, 2020
@janvorli janvorli modified the milestones: 6.0.0, 7.0.0 Jul 19, 2021
@mangod9
Copy link
Member

mangod9 commented Jul 19, 2022

@janvorli, is this still something which needs fixing?

@janvorli
Copy link
Member

It is a simple stupid bug in the test - missing path separator character. I will fix it.

@janvorli janvorli mentioned this issue Jul 19, 2022
Merged
@ghost ghost added the in-pr There is an active PR which will close this issue when it is merged label Jul 19, 2022
@ghost ghost removed the in-pr There is an active PR which will close this issue when it is merged label Jul 20, 2022
@ghost ghost locked as resolved and limited conversation to collaborators Aug 19, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
area-PAL-coreclr
Projects
None yet
Milestone
7.0.0
Development

Successfully merging a pull request may close this issue.

Fix PAL GetFullPathNameA test3 janvorli/runtime
4 participants
@janvorli @davidwrighton @Dotnet-GitSync-Bot @mangod9