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

Replace marshaledBytesConsumer/Producer in tests with testBinary #62

Closed
dekarrin opened this issue Oct 11, 2023 · 0 comments
Closed

Replace marshaledBytesConsumer/Producer in tests with testBinary #62

dekarrin opened this issue Oct 11, 2023 · 0 comments
Milestone
2.3.0

Comments

@dekarrin
Copy link
Owner

Looks like a lot of duplication in the use of marshaledBytesConsumer/marshaledBytesProducer and just plain testBinary in tests. Tests originally used the marshaled consumer and producer to get around needing a specific struct, but eventually moved to simple testBinary usage once available. Update tests to use that instead of the marshaledBytesX types that are in rezi_test.go
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
2.3.0
Development

No branches or pull requests
1 participant
@dekarrin