https://github.com/symflower/eval-dev-quality/pull/155/files missing a test

Question

https://github.com/symflower/eval-dev-quality/pull/155/files missing a test

Closed this issue 4 months ago · 3 comments

All change need tests in TDD style. fail first, pass with changes. Same rules as internally.

Answer 1 · 2024-06-05T13:28:09.000Z

Well the testcase itself is using filepath.Join for the paths so if the test case is run on windows or macos or linux it is using the path separator for that particular OS

Answer 2 · 2024-06-05T16:46:24.000Z

What was the reason for changing this then? I thought you said something failed/did not work?

Answer 3 · 2024-06-06T14:27:42.000Z

The test case was wrong in the first place as it used hardcoded / for paths, because of the hardcoded nature the test succeeded on Windows even if it shouldn't.

By switching the testcase result to use filepath.Join instead of the hardcoded versions, it did in fact fail on Windows until I changed the implementation of TestFilePath to filepath.Join too.