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

Test regressions with W3C SVG 1.1 Test Suite #1357

Merged
merged 11 commits into from Mar 4, 2021
Merged

Conversation

TrySound
Copy link
Member

@TrySound TrySound commented Feb 21, 2021

Visual regressions tests will give us more confidence in SVGO stability.
Test cases are downloaded from w3.org, screenshooted with playwright and
compared with pixelmatch.

See test suite https://www.w3.org/Graphics/SVG/Test/20110816/

Bugs detected with these tests:

@TrySound TrySound marked this pull request as draft February 21, 2021 13:57
@XhmikosR
Copy link
Contributor

Shouldn't this run on CI? If so, perhaps a separate job?

@TrySound
Copy link
Member Author

Yes, it will be. Not all tests are passing yet. This is why it's draft.

@sk-
Copy link
Contributor

sk- commented Feb 22, 2021

Great work @TrySound. I was working on something similar too.

I'm using:

See screenshots attached:

Screen Shot 2021-02-22 at 11 50 00 AM

Screen Shot 2021-02-22 at 11 50 31 AM

Maybe we should create an issue where we can discuss ideas.

@TrySound
Copy link
Member Author

Sure, feel free to open one

TrySound added a commit that referenced this pull request Feb 23, 2021
Ref https://github.com/svg/sax/releases/tag/v0.1.1

This fixes enother case in [regression tests](#1357).
TrySound added a commit that referenced this pull request Feb 23, 2021
@TrySound TrySound force-pushed the test-regression branch 3 times, most recently from a08b452 to 45d664d Compare March 1, 2021 10:11
Visual regressions tests will give us more confidence in SVGO stability.
Test cases are downloaded from w3.org, screenshooted with playwright and
compared with pixelmatch.
@TrySound TrySound marked this pull request as ready for review March 4, 2021 14:11
@TrySound
Copy link
Member Author

TrySound commented Mar 4, 2021

Current result
Skipped: 97
Mismatched: 0
Passed: 429

@TrySound TrySound merged commit 4d0ee0c into master Mar 4, 2021
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 this pull request may close these issues.

None yet

3 participants