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

[ConvDiff] ConvectionDiffusionApplication testing utilities #11659

Merged
merged 2 commits into from
Oct 5, 2023

Conversation

rubenzorrilla
Copy link
Member

📝 Description
This PR creates a new ConvectionDiffusionTestingUtilities static class to be used in the cpp tests, allowing to remove the auxiliary copied&pasted function that prepares the model part in all tests.

@rubenzorrilla rubenzorrilla added Cleanup Applications Testing FastPR This Pr is simple and / or has been already tested and the revision should be fast labels Oct 4, 2023
@rubenzorrilla rubenzorrilla requested a review from a team as a code owner October 4, 2023 16:18
@rubenzorrilla rubenzorrilla self-assigned this Oct 4, 2023
Copy link
Member

@roigcarlo roigcarlo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@rubenzorrilla rubenzorrilla merged commit 2a052f2 into master Oct 5, 2023
@rubenzorrilla rubenzorrilla deleted the convdiff/test-utilities-creation branch October 5, 2023 07:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Applications Cleanup FastPR This Pr is simple and / or has been already tested and the revision should be fast Testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants