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

Tests: merge SofaTests with Sofa.Testing #1885

Closed
hugtalbot opened this issue Mar 3, 2021 · 3 comments · Fixed by #3000
Closed

Tests: merge SofaTests with Sofa.Testing #1885

hugtalbot opened this issue Mar 3, 2021 · 3 comments · Fixed by #3000
Assignees
Labels
issue: discussion Open topic of discussion STC#13 Tasks for STC#13 coding sprint

Comments

@hugtalbot
Copy link
Contributor

hugtalbot commented Mar 3, 2021

Further #1834 :

Tests: merge SofaTests with Sofa.Testing
Make sure that examples (see in Plugin Example : https://github.com/sofa-framework/sofa/blob/master/applications/plugins/PluginExample/PluginExample_test/MyBehaviorModel_test.cpp) and doc are up to date: https://www.sofa-framework.org/community/doc/contributing-to-sofa/writing-tests/

Connected to #417 #489

and PRs: #455 #471

@fredroy fredroy added the issue: discussion Open topic of discussion label Mar 12, 2021
@fredroy fredroy added the STC#11 Simple tasks for STC#11 coding sprint label May 26, 2021
@fredroy fredroy self-assigned this Jun 2, 2021
@fredroy
Copy link
Contributor

fredroy commented Jun 2, 2021

A list of tasks to do:

  1. remove SofaTest use in Sofa*_test (first in the most important modules SofaBase* and then SofaCommon and ...)
  2. dispatch the existing tests in the SofaTest

I suppose 1) will involve to add/modify the new Sofa.Testing API.
This post will surely be updated heavily soon....

@hugtalbot
Copy link
Contributor Author

How far are we on this @fredroy? same point?

@hugtalbot hugtalbot added STC#13 Tasks for STC#13 coding sprint and removed STC#11 Simple tasks for STC#11 coding sprint STC#12 Tasks for STC#12 coding sprint labels May 10, 2022
@hugtalbot
Copy link
Contributor Author

congrats @fredroy !!!! 👏 👏 👏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
issue: discussion Open topic of discussion STC#13 Tasks for STC#13 coding sprint
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants