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

Add tests for LocalFunctionStatementSyntaxWrapper #2384

Closed
sharwell opened this issue Jun 10, 2017 · 3 comments
Closed

Add tests for LocalFunctionStatementSyntaxWrapper #2384

sharwell opened this issue Jun 10, 2017 · 3 comments

Comments

@sharwell
Copy link
Member

This wrapper type was added in #2370 without tests. It's not too difficult to add them; see commit c31beae and other examples from that pull request for how to implement good coverage of these types.

@bjornhellander
Copy link
Contributor

I can have a look at this

@sharwell
Copy link
Member Author

sharwell commented Jun 13, 2017

@bjornhellander Maybe start with #2376 (a simpler case) and then do this one? I won't stop you if you start with this though. 😆

@bjornhellander
Copy link
Contributor

😄 Yes, I just randomly picked one and I see that there are smaller ones available, so I'll have a look at some other wrappers first and come back to this soon. You can keep it assigned to me for now if you want.

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

No branches or pull requests

2 participants