-
Notifications
You must be signed in to change notification settings - Fork 53
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
converted from g_test_add() to g_test_add_func() #422
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tests are failing because the code autoformatter is making changes. Please run ninja test
to format that code and commit the changes.
I did run |
modulemd/tests/test-dirty.py
Outdated
@@ -11,6 +11,7 @@ | |||
# For more information on the license, see COPYING. | |||
# For more information on free software, see | |||
# <https://www.gnu.org/philosophy/free-sw.en.html>. | |||
#comment |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is the point of this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I just removed it, I will be pushing it after the tests complete.
fixed errors fixed errors in functions solved comment issue
I have been running |
converted from g_test_add() to g_test_add_func() Resolves: #417 Signed-off-by: Stephen Gallagher <sgallagh@redhat.com>
corrected the previous syntax and also modified the required functions to enable compatibility with g_test_add_func()