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

Basic Text Read Tests for Ion C #62

Closed
almann opened this issue Aug 9, 2020 · 0 comments · Fixed by #66
Closed

Basic Text Read Tests for Ion C #62

almann opened this issue Aug 9, 2020 · 0 comments · Fixed by #66
Labels
enhancement New feature or request ion-c-sys Issues related to Ion C bindings

Comments

@almann
Copy link
Contributor

almann commented Aug 9, 2020

Towards #36. Replace the existing unit tests with some deeper tests using the current higher-level API

@almann almann added the enhancement New feature or request label Aug 9, 2020
almann added a commit that referenced this issue Aug 9, 2020
* Removes the original unit tests.  The doc tests and these
  tests cover what they did and more.
* Adds dev-dependency to `rstest` to get the parameterized
  functionality.

Resolves #62.
@almann almann added the ion-c-sys Issues related to Ion C bindings label Aug 9, 2020
@almann almann closed this as completed in #66 Aug 9, 2020
almann added a commit that referenced this issue Aug 9, 2020
* Implements parameterized text read tests for ion-c-sys.

* Removes the original unit tests.  The doc tests and these
  tests cover what they did and more.
* Adds dev-dependency to `rstest` to get the parameterized
  functionality.

Resolves #62.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request ion-c-sys Issues related to Ion C bindings
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant