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.

Sign up for GitHub

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

Jump to bottom

Test RecordSet.SubsetLimit() #28

Open
zacsketches opened this issue Dec 20, 2018 · 1 comment
Open

Test RecordSet.SubsetLimit() #28

zacsketches opened this issue Dec 20, 2018 · 1 comment
Labels
Testing Issue with the automated test suite

Comments

@zacsketches
Copy link
Contributor

This function is used in almost every utility. This is the highest priority for testing as of mid-dec 2018.
@zacsketches zacsketches added help wanted Extra attention is needed Testing Issue with the automated test suite labels Dec 20, 2018
@zacsketches zacsketches self-assigned this Dec 20, 2018
@zacsketches
Copy link
Contributor Author

commit 0c45b6b makes progress on this issue. From the commit message getting the last few error values covered is non-trivial because it may require an API breaking change so that errors can be generated on the RecordSet that is now created internal to the function.

@zacsketches zacsketches removed the help wanted Extra attention is needed label Dec 27, 2018
@zacsketches zacsketches removed their assignment Dec 28, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Testing Issue with the automated test suite
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@zacsketches