getDirectorySeparator() returns "\\" on Windows but it sometimes should return "/"
Nobody has claimed this yet.
Assessment
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Newbie friendliness
- 35/100
- Issue type
- Bug
- Clarity
- Mostly clear
- Activity status
- Stale
- Tech stack
- cpp
- Domain
- testing-qa
Research direction
Start in SystemUtils.cpp by reading getDirectorySeparator() and the surrounding path-handling code. Reproduce the directory-creation failure in WSL or Windows WorkSpaces Applications, then run the affected tests to verify that paths use the appropriate separator and directory creation succeeds.
Written by the indexing model from the issue text.
Description
In SystemUtils.cpp there is a function "getDirectorySeparator()". On Windows (when #ifdef _WIN32 returns true) it returns "\".
When you are running under WSL it needs to return "/".
When you are running on a Windows machine under AWS "WorkSpaces Applications" it also needs to return "/".
If you have the wrong directory separator then your tests fail with this message:
Failure:
test case THREW exception:
Unable to create directory:
What it has failed to do is work out the path to put the received file in, since it can't properly parse the directory the test case is in.
I have patched this problem locally where I have seen it just by replacing "\" with "/" in the code. I don't know how to properly determine which directory separator to use on Windows, but it seems that #ifdef _WIN32 is not a good enough guide.
- Dominant language
- C++
- Stars
- 339
- Forks
- 55
- PR merge metrics
- No merged PRs in 30d
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from approvals/ApprovalTests.cpp
-
Difficulty 2/5 1-3 hours Newbie friendliness 74/100
approvals/ApprovalTests.cpp#247 ·
-
Difficulty 3/5 1-2 days Newbie friendliness 45/100
approvals/ApprovalTests.cpp#242 ·
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 45/100
approvals/ApprovalTests.cpp#226 · 2 comments ·
-
bug namers
Difficulty 3/5 1-2 days Newbie friendliness 45/100
approvals/ApprovalTests.cpp#225 · 5 comments ·
-
Difficulty 5/5 Over a week Newbie friendliness 25/100
approvals/ApprovalTests.cpp#217 · 4 comments ·
All issues in approvals/ApprovalTests.cpp
Similar issues
-
Difficulty 1/5 Under an hour Newbie friendliness 90/100
AXERA-TECH/ax-llm#77 ·
-
Difficulty 1/5 Under an hour Newbie friendliness 90/100
games-on-whales/wolf#509 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 74/100
-
bug-unconfirmed
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 74/100
NVIDIA/cuda-samples#453 ·