-
Notifications
You must be signed in to change notification settings - Fork 40
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
Added test_case related to anonymous namespace. #75
Added test_case related to anonymous namespace. #75
Conversation
Codecov Report
@@ Coverage Diff @@
## master #75 +/- ##
=======================================
Coverage 86.43% 86.43%
=======================================
Files 80 80
Lines 6346 6346
=======================================
Hits 5485 5485
Misses 861 861 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
@bveldhoen Thanks for the PR. First of all, please note that there is already a test case for anonymous namespace: Also your new test case does not crash or fail - my suspicion was that you had an anonymous namespace in the root namespace? As to the issue with running the test cases on your system, the error says:
Could it be a permissions issue? in any case - can you try |
make test output:
|
Abandoning because this scenario is already covered. |
I suspect the error has something to do with installed clang versions. On another machine, I didn't get this error while running the tests (without modifications). Also, on the machine on which I was getting the error, I needed to specifically use clang-cpp12 in order to get it to build (see change to CMakeLists.txt file in this PR). I probably need to compare the differences in installed clang packages on both machines. Will do so when I have the time. |
Added a unit test to try and reproduce a suspected issue. I'm running into a problem when executing tests. What am I doing wrong?