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 testing to the Stroke Tessellator #193

Open
o0Ignition0o opened this issue Oct 23, 2017 · 1 comment
Open

Add testing to the Stroke Tessellator #193

o0Ignition0o opened this issue Oct 23, 2017 · 1 comment
Labels

Comments

@o0Ignition0o
Copy link
Contributor

o0Ignition0o commented Oct 23, 2017

Followup on this #192 (comment) it would be nice to add some tests to the tessellator.

Before trying to implement this, I'd love to bikeshed on what would be best to test the whole module.

I don't know who to cc on this, so feel free to add cc's and comments :)

@nical
Copy link
Owner

nical commented Oct 23, 2017

Fuzz tests ?

Fuzzing is already in place (from the cli directory, run cargo run -- fuzz -s), but it isn't very useful in its current state because there isn't a lot of assertions in the stroke tessellator for the fuzzer to find.

Reftests would be pretty great. It would pretty cool if the cli program had a sub-command to render fills and strokes to an image. We could build a script on top of that to compare the images and as a bonus log the errors using the same format as gecko/webrender to reuse the reftest analyser tool.

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

No branches or pull requests

2 participants