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

light-client: Verification Testing #330

Closed
brapse opened this issue Jun 16, 2020 · 1 comment · Fixed by #475
Closed

light-client: Verification Testing #330

brapse opened this issue Jun 16, 2020 · 1 comment · Fixed by #475
Assignees
Labels
light-client Issues/features which involve the light client tests
Milestone

Comments

@brapse
Copy link
Contributor

brapse commented Jun 16, 2020

part of #320

The verifier is composed of multiple predicates, each of which should be tested for both the positive and negative case using concrete types. operations should be mocked out.

Setup a simple test harness which facilitates to construction of concrete types to limit the dependency graph and make room for more general constructions.

@brapse brapse added light-client Issues/features which involve the light client tests labels Jun 16, 2020
@brapse brapse added this to the Light Node milestone Jun 16, 2020
@brapse brapse self-assigned this Jun 16, 2020
@brapse
Copy link
Contributor Author

brapse commented Jun 18, 2020

So this as it turns out is trickier than it seems. In order to effectively exercise the predicates mentioned above we need to actually construct concrete types which are very complex. One approach would be to provide factories for these concrete types which extend the fixtures provides in the json tests but allow overriding specific fields. This way the tests can focus on specific field interactions without getting lost in details.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
light-client Issues/features which involve the light client tests
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants