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

Enhance identity matrix tests and improve exception handling #27

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

Conversation

ershadow786
Copy link
Collaborator

  • Summary :
    Add parameterized test data and fixes for identity matrix creation:

  • Introduced static data providers using Stream<Arguments> for parameterized tests:

    • provideValidEyeData: Validates the creation of identity matrices with various sizes, including edge cases (1x1 and 5x5 matrices).

    • provideInvalidEyeDimensions: Tests for expected exceptions when creating identity matrices with invalid dimensions (negative and zero values).

  • Fixed identified issues in the eye function and improved exception handling in tests.

  • Enhanced assertions to ensure correct matrix shape and behavior when exceptions are encountered.

Enhance tests for identity matrix creation and invalid dimensions

- Added parameterized tests for the `eye` function to verify creation of identity matrices of various sizes, including edge cases (1x1 and 5x5 matrices).
- Implemented test for expected exceptions when invalid dimensions (negative and zero values) are provided.
- Updated `provideValidEyeData` and `provideInvalidEyeDimensions` methods to supply comprehensive test cases for the identity matrix creation.
- Improved assertions to validate the shape of the resulting matrices and confirm correct behavior when exceptions are expected.
- Commented out `assertArrayEquals` for future implementation once array comparison logic is determined.
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