Add Monthly report for September #28
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
mingness
approved these changes
Oct 3, 2025
| - VisualTestRunner: The central orchestration class that manages the entire test lifecycle, from capturing a sketch to comparing it against a baseline image and generating a final test result. | ||
| - ImageComparator: The custom, self-contained pixel matching engine that uses a pixelmatch-inspired algorithm implemented entirely in Java, eliminating external dependencies. | ||
| - SketchRunner: A dedicated PApplet instance that manages the execution of a user's sketch in an off-screen buffer, ensuring consistent, repeatable rendering for testing purposes. | ||
| - ProcessingTestSuite: A class for organizing and running multiple tests, simplifying the management of large test suites. |
There was a problem hiding this comment.
Can you add comments about these test classes (VisualTestRunner and ProcessingTestSuite) and JUnit? You mentioned you had some difficulty with JUnit in the call, could you go into detail here? In our chats, I thought these would work WITH JUnit, but now I'm not sure, after seeing your future work bullet point on JUnit and not using these classes. We ourselves talked about how we would use JUnit, but from the call yesterday, it is apparent that it is not clear that we will use JUnit, and that there is a preference for using JUnit.
There was a problem hiding this comment.
Sure, I'll add the required explanations.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
cc @mingness @SableRaf