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

Update groovy TestFuzzer.java Enhance Fuzz Testing with Additional Custom Object Test Cases #12823

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

Conversation

Shivam7-1
Copy link
Contributor

PR Description: Added additional test cases for custom objects, including null handling, hash code consistency, toString validation, and large object array testing. Enhanced fuzz testing for equality, edge cases, and inheritance scenarios to improve robustness and reliability.

Key changes Done:

  1. Null and Invalid Comparisons:
  • testNullObjectEquality: Ensures that comparing a MyObject to null fails.
  • Tests null array handling.
  1. HashCode Consistency:
  • testHashCodeConsistency: Ensures that two equal MyObject instances return the same hash code.
  1. ToString Method:
  • testCustomObjectToString: Verifies that toString() returns the correct string representation.
  1. Large Object Array Handling:
  • testLargeObjectArrayHandling: Tests large arrays of MyObject to see how the system handles them.
  1. Inheritance Handling:
  • testCustomObjectEqualityWithInheritance: A placeholder for future inheritance tests.

@Shivam7-1 Shivam7-1 changed the title Update groovy TestFuzzer.java Update groovy TestFuzzer.java Enhance Fuzz Testing with Additional Custom Object Test Cases Dec 7, 2024
Copy link

github-actions bot commented Dec 7, 2024

Shivam7-1 is a new contributor to projects/groovy. The PR must be approved by known contributors before it can be merged. The past contributors are: henryrneh, fmeum, aschaich

@Shivam7-1
Copy link
Contributor Author

Shivam7-1 commented Dec 9, 2024

Hii @henryrneh @fmeum @aschaich Could Team Please Review This PR
Thanks

@Shivam7-1
Copy link
Contributor Author

Hii @fmeum Could You Please Review This PR
Thanks

@fmeum
Copy link
Contributor

fmeum commented Dec 10, 2024

Sorry, I no longer work on OSS-Fuzz.

@Shivam7-1
Copy link
Contributor Author

Hii @fmeum Thanks For Responding Do you anyone who Could Review this PR Could You Please ping them here
Thanks

@fmeum
Copy link
Contributor

fmeum commented Dec 10, 2024

@DavidKorczynski Can you help here? None of the suggested contributors are active anymore.

Copy link
Collaborator

@DavidKorczynski DavidKorczynski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! Can you help reach out to maintainers and get these harnesses landed upstream? That should help the reviewing process

@Shivam7-1
Copy link
Contributor Author

Shivam7-1 commented Dec 11, 2024

Hii @jonathanmetzman @DavidKorczynski @AdamKorcz Could Team please review or help in this PR
Thanks

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.

3 participants