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.

Sign up for GitHub

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

Jump to bottom

Add Unit Tests for PVector in processing:core #986

Open
Rishab87 opened this issue Mar 20, 2025 · 1 comment · May be fixed by #987
Open

Add Unit Tests for PVector in processing:core #986

Rishab87 opened this issue Mar 20, 2025 · 1 comment · May be fixed by #987

Comments

@Rishab87
Copy link
Contributor

Currently processing does not have unit tests for PVector. So it would be great if we have unit tests for them. The tests should cover all possible cases to ensure everything works as expected
@Rishab87 Rishab87 linked a pull request Mar 20, 2025 that will close this issue
Open
@Rishab87
Copy link
Contributor Author

Hi I've created a PR for it.

Also, after this I was planning to add tests for PMatrix, PMatrix2D and PMatrix3D as these classes are getting used throughout the rendering pipeline ig, so I think its necessary to ensure they work correctly. What do you think?

I also noticed that these unit tests are not integrated in GitHub workflows, so should I integrate them?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Added Unit Tests for PVector in processing:core Rishab87/processing4
1 participant
@Rishab87