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

Refactor bindgen snapshot unit tests to use insta #1038

Closed
agostbiro opened this issue Jun 29, 2023 · 3 comments · Fixed by #1090
Closed

Refactor bindgen snapshot unit tests to use insta #1038

agostbiro opened this issue Jun 29, 2023 · 3 comments · Fixed by #1090
Assignees

Comments

@agostbiro
Copy link
Contributor

agostbiro commented Jun 29, 2023

As @frol suggested in #1036 (comment) we should refactor bindgen snapshot unit tests to use the insta crate.

Unit tests that could benefit from this:

@jaswinder6991
Copy link
Contributor

Hi @agostbiro, are you working on this issue? If not, I would like to try it out.

@iho
Copy link
Contributor

iho commented Sep 26, 2023

@agostbiro @jaswinder6991 @frol Do we have any progress? Can I take this task?

@frol
Copy link
Collaborator

frol commented Sep 26, 2023

@iho Please, go ahead! Take inspiration from borsh-rs setup.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants