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

Test #37

Open
wants to merge 17 commits into
base: main
Choose a base branch
from
Open

Test #37

wants to merge 17 commits into from

Conversation

MEKBAL
Copy link
Contributor

@MEKBAL MEKBAL commented Jun 20, 2024

Unitary Tests with CI

@MEKBAL
Copy link
Contributor Author

MEKBAL commented Jun 26, 2024

@aneojgurhem @a-hamitouche j'ai apporté les modifications demandées. Pourriez-vous les revoir ?


enable_testing()

set(TEST_LIST test_binary_PENE test_binary_Verificarlo)
Copy link
Contributor

Choose a reason for hiding this comment

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

Can you remove verificarlo from the executed tests? The fix will be done later



if __name__ == "__main__":
pytest.main()
Copy link
Contributor

Choose a reason for hiding this comment

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

Can you make the execution of a frontend depending on arguments and only giving PENE as parameter ?

@@ -0,0 +1,73 @@

Copy link
Contributor

Choose a reason for hiding this comment

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

Isn't there a way to reduce redundancy of CMaksLists?

Copy link
Contributor Author

@MEKBAL MEKBAL Jul 1, 2024

Choose a reason for hiding this comment

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

No, there is non way.




if __name__ == "__main__":
Copy link
Contributor

Choose a reason for hiding this comment

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

Same comment about arguments

@@ -0,0 +1,74 @@

Copy link
Contributor

Choose a reason for hiding this comment

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

Same comment about CMakeLists

Test/Test_result/Test_Frontend_executable.cpp Show resolved Hide resolved
@@ -0,0 +1,161 @@

Copy link
Contributor

Choose a reason for hiding this comment

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

Same comments


sqrt_double:
*res=sqrt(b);
end_sqrt_double
Copy link
Contributor

Choose a reason for hiding this comment

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

Add end of ligne character



if __name__ == "__main__":
pytest.main()
Copy link
Contributor

Choose a reason for hiding this comment

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

Add end of line character

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.

2 participants