WIP: Add gitHub workflow to run tests #152
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
@vsevolod-misiul: it would be great to get someone else's eyes on this too :-) @mcfletch -- you need to approve having this workflow run .... Meanwhile, you all can see the run results on my fork. |
|
NOTE: It may be fine to only run the accelerate tests -- those are the more tricky ones, what with cross platform issues and all that. Though even a subset of the rest would be good. -CHB |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This is just a start:
It's running the tests in the multiple platforms, but most aren't passing:
Not doing Ubuntu -- the test runners don't have GL -- though Mesa could be added, I just haven't tried that yet.
A bunch of the PyOpenGL tests are failing. I haven't debugged that yet, as a bunch of tests are failing locally for me (on OS-X 14 Intel) so if I can't get them to pass locally -- what hope do I have on the CI ?
We'll probably need to run only a subset of the tests anyway -- I think there's some interaction with the display that's going to be problematic regardless.
I added the patch from Fix import error on aarch64: define 'long' using ctypes.c_long #146 -- to get Accelerate to compile
The accelerate tests are passing -- at least on Windows, OS-X tests aren't running at all right now ....
It would be nice to get this going though -- automated tests are a great thing!