Skip to content

Add test cases for null handling in EqualsBuilder.reflectionEquals#1619

Open
shreyash24-tech wants to merge 1 commit intoapache:masterfrom
shreyash24-tech:add-null-test
Open

Add test cases for null handling in EqualsBuilder.reflectionEquals#1619
shreyash24-tech wants to merge 1 commit intoapache:masterfrom
shreyash24-tech:add-null-test

Conversation

@shreyash24-tech
Copy link

Changes

  • Added test cases for EqualsBuilder.reflectionEquals

Details

  • Added test to verify behavior when both objects are null
  • Added test to verify behavior when one object is null

Reason

These tests improve edge case coverage and ensure correct handling of null values.

Copy link
Member

@garydgregory garydgregory left a comment

Choose a reason for hiding this comment

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

Hello @shreyash24-tech

-1: This PR duplicates existing tests.

Did you look? For example: org.apache.commons.lang3.builder.EqualsBuilderTest.testReflectionEquals()

Do not change the order of imports, do not make cosmetic edits.

Be mindful that by creating a PR, you requests a maintainer's time.

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.

2 participants