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.

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

Nested SECTIONs in tests do not appear to work #71

Open
fmahebert opened this issue May 11, 2023 · 2 comments
Open

Nested SECTIONs in tests do not appear to work #71

fmahebert opened this issue May 11, 2023 · 2 comments
Assignees
Labels
bug Something isn't working

Comments

@fmahebert
Copy link
Contributor

What happened?

The test case test_polygon uses nested SECTION scopes, and it appears that lines of code in inner SECTION scopes are not run. I think test_polygon may be the only eckit test using this nesting.

What are the steps to reproduce the bug?

  1. In file test_polygon.cc below line 240 (inside nested SECTION "Contains edge"), add new line of code EXPECT(false);. Expect test ctest -R eckit_test_geometry_polygon to fail, but it passes.
  2. To show expected behavior, move this new line of code to above line 240 (in outer SECTION "Simple rectangular polygon"). Expect test to fail, and it does.

Version

v1.20.2

Platform (OS and architecture)

macOS 12.6.4; intel i7

Relevant log output

No response

Accompanying data

No response

Organisation

No response

@fmahebert fmahebert added the bug Something isn't working label May 11, 2023
@wdeconinck wdeconinck mentioned this issue May 6, 2024
@wdeconinck
Copy link
Member

@pmaciel do you have a fix for this in another branch?

@pmaciel
Copy link
Member

pmaciel commented Jun 12, 2024

@pmaciel do you have a fix for this in another branch?

My fix for it was to flatten the hierarchy, ie remove the nested SECTIONs. The work I have isn't ready to be merged, so I suggest to flatten here as well?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants