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

fix test_exprevaluator in parallel @vincentchabannes #1658

Merged
merged 1 commit into from
Jul 2, 2021

Conversation

romainhild
Copy link
Contributor

  • Have you checked to ensure there aren't other open Pull Requests for the same change?
  • Have you added an explanation of what your changes do and why you'd like us to include them?
  • Have you written new tests for your changes?
  • Have you successfully run the Feel++ testsuite with your changes locally?
  • Have you written Doxygen comments in your contribution ?

The test pass in sequential and in parallel now:

Test project /ssd/hild/builds/develop_rel/testsuite/feelvf
    Start 47: feelpp_test_exprevaluator-np-6
1/2 Test #47: feelpp_test_exprevaluator-np-6 ...   Passed    4.11 sec
    Start 48: feelpp_test_exprevaluator-np-1
2/2 Test #48: feelpp_test_exprevaluator-np-1 ...   Passed    3.91 sec

100% tests passed, 0 tests failed out of 2

@romainhild romainhild self-assigned this Jul 2, 2021
@vincentchabannes vincentchabannes merged commit dda9e8a into develop Jul 2, 2021
@vincentchabannes vincentchabannes deleted the romainhild/issue1652 branch July 2, 2021 07:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants