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

Existing tests for tmpName/tmpNameSync do not cover the name option #212

Closed
silkentrance opened this issue Jan 24, 2020 · 0 comments · May be fixed by mike-north/devcert#45 or sekikn/avro#9
Closed
Assignees
Labels

Comments

@silkentrance
Copy link
Collaborator

silkentrance commented Jan 24, 2020

the name option must be tested against.

it is indirectly tested in standard inband tests.

@silkentrance silkentrance self-assigned this Jan 24, 2020
@silkentrance silkentrance changed the title existing tests for name or nameSync do not include the name option existing tests for name or do not include the name option Jan 24, 2020
@silkentrance silkentrance changed the title existing tests for name or do not include the name option existing tests for tmpName do not include the name option Jan 24, 2020
@silkentrance silkentrance changed the title existing tests for tmpName do not include the name option existing tests for tmpName do not cover the name option Jan 24, 2020
@silkentrance silkentrance changed the title existing tests for tmpName do not cover the name option Existing tests for tmpName/tmpNameSync do not cover the name option Jan 24, 2020
silkentrance added a commit that referenced this issue Jan 27, 2020
fix #212: enable direct name option test
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
1 participant