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: Take advantage of TAP test driver #135

Merged
merged 6 commits into from
Mar 29, 2018
Merged

Conversation

ueno
Copy link
Member

@ueno ueno commented Mar 27, 2018

This makes test output finer grained and also enables to write skipped/expected-to-fail tests in a single test case.

@coveralls
Copy link

coveralls commented Mar 27, 2018

Coverage Status

Coverage increased (+0.8%) to 81.219% when pulling 8ef701e on ueno:wip/dueno/tap into e8d5690 on p11-glue:master.

@@ -0,0 +1,70 @@
/*
* Copyright (c) 2012 Stefan Walter
Copy link
Contributor

Choose a reason for hiding this comment

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

I think it should be red hat here.

* THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH
* DAMAGE.
*
* Author: Stef Walter <stef@thewalter.net>
Copy link
Contributor

Choose a reason for hiding this comment

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

Author info seems inaccurate.

assert_todo ("not implemented", "CKR_CRYPTOKI_ALREADY_INITIALIZED handling");
}
finalize_and_free_modules (modules);
}
Copy link
Contributor

@nmav nmav Mar 27, 2018

Choose a reason for hiding this comment

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

r+

@ueno ueno force-pushed the wip/dueno/tap branch 2 times, most recently from a37a936 to a3a8b82 Compare March 28, 2018 06:12
@ueno ueno merged commit 7827e65 into p11-glue:master Mar 29, 2018
@ueno
Copy link
Member Author

ueno commented Mar 29, 2018

Thank you for the review!

@ueno ueno deleted the wip/dueno/tap branch March 31, 2018 08:34
@ueno ueno added this to the 0.23.11 milestone May 7, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants