Skip to content

Add Unit Tests for Matrix and Extend Test Coverage for Math Utilities #453

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

Open
jnyfah opened this issue May 25, 2025 · 1 comment · May be fixed by #454
Open

Add Unit Tests for Matrix and Extend Test Coverage for Math Utilities #453

jnyfah opened this issue May 25, 2025 · 1 comment · May be fixed by #454
Labels
area-macOS Work on macOS system area-window Work on Window system enhancement New feature or request good first issue Good for newcomers

Comments

@jnyfah
Copy link
Collaborator

jnyfah commented May 25, 2025

Currently, our Matrix class lacks dedicated unit tests. While there's some initial test structure and inspiration in matrix_test.cpp, we need to formalize and expand this coverage.

Tasks:

  • Add comprehensive unit tests for the Matrix class.
  • Validate operations such as construction, indexing, arithmetic, and transformations.
  • Use matrix_test.cpp as a reference or starting point.
  • If we introduce new math utilities like Quaternion, include corresponding test files and coverage as well.
@jnyfah
Copy link
Collaborator Author

jnyfah commented May 25, 2025

@AmanMenda

@JeanPhilippeKernel JeanPhilippeKernel added this to the Born baby (0.2.0) milestone May 25, 2025
@JeanPhilippeKernel JeanPhilippeKernel added enhancement New feature or request good first issue Good for newcomers area-window Work on Window system area-macOS Work on macOS system labels May 25, 2025
@JeanPhilippeKernel JeanPhilippeKernel moved this to In Progress in ZEngine Board Jun 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-macOS Work on macOS system area-window Work on Window system enhancement New feature or request good first issue Good for newcomers
Projects
Status: In Progress
Development

Successfully merging a pull request may close this issue.

2 participants