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

Github Actions: Implement a "Mesa Compliance Testing" #1575

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

MeFisto94
Copy link
Member

@MeFisto94 MeFisto94 commented Jun 11, 2021

using https://github.com/MeFisto94/jme3-testing that runs functional tests on a software renderer to validate rendering works correctly.

Scope

This is only the tracking PR about the Github Actions integration of this feature. Anything related to the feature itself may happen on the hub topic https://hub.jmonkeyengine.org/t/help-testing-jmonkeyengine/42856 or the jme3-testing repository.

Note: Currently when merging the PR, none of the tests will work, because they all time out, which means app.stop isn't called correctly. This used to work on an older revision of the engine, maybe context changes have caused the issue, I will investigate it, but it's unrelated to this PR basically.

Security

In theory jme3-testing and the buildenvs should use specific commit hashes to protect against their maintainers introducing evil code (or host them into the main organization, also from the aspect of it not disappearing), since this is a dedicated workflow, unrelated to releasing binaries or something, only the second aspect may be worth considering.

I'll move this PR in draft state until the issues in jme3-testing have been solved, feel free to comment on both this and jme3-testing in general.

Edit: Feel free to try jme3-testing locally, for me it almost works (testSimpleBlueCube10sDebugStats fails for lwjgl2), so it's something with the headless environment again.

…ithub.com/MeFisto94/jme3-testing that runs functional tests on a software renderer to validate rendering works correctly
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.

1 participant