Skip to content

Conversation

kjarosh
Copy link
Member

@kjarosh kjarosh commented Oct 5, 2025

This patch adds possibility to run SWF tests from //tests/tests/swfs on web.

Currently web-specific renderers like webgl and canvas are largely being untested. This will make it possible in the future to test them properly and make sure they work the same way as wgpu.

This is mostly a (working) draft, because I have to come up with a (sane) way of controlling Ruffle through web for testing (e.g. manual ticking).

Please let me know what's your thoughts about this PR.

This makes it possible to use top-level await expressions.
This patch adds possibility to run SWF tests from //tests/tests/swfs
on web.

Currently web-specific renderers like webgl and canvas are largely being
untested. This will make it possible in the future to test them properly
and make sure they work the same way as wgpu.
@kjarosh kjarosh added A-tests Area: Tests & Test Framework T-feature Type: New Feature (that Flash doesn't have) labels Oct 5, 2025
@n0samu
Copy link
Member

n0samu commented Oct 5, 2025

How does this compare to #15243?

@kjarosh
Copy link
Member Author

kjarosh commented Oct 5, 2025

How does this compare to #15243?

Oh that's interesting, I didn't know about that PR.

This PR is a much simpler version of #15243 that leverages existing selfhosted tests for running selected SWF tests. I wonder why #15243 didn't get merged back in the day...

@n0samu
Copy link
Member

n0samu commented Oct 5, 2025

There are a lot of great PRs that seem to just be forgotten :((

@n0samu
Copy link
Member

n0samu commented Oct 5, 2025

Looking at old conversation on Discord, it doesn't look like anyone had any objections - people just didn't get around to reviewing the PR, as often happens

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

Labels

A-tests Area: Tests & Test Framework T-feature Type: New Feature (that Flash doesn't have)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants