Skip to content

Implement equals and Hashcode on GenericData #589

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

Merged
merged 7 commits into from
Feb 13, 2019
Merged

Implement equals and Hashcode on GenericData #589

Changes from 1 commit
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
a9ae85a
936: Added hashCode(), toString() and equals() methods relaing on cla…
Feb 12, 2019
1fe20d8
936: Changes after review.
Feb 12, 2019
75a9714
936: Modified unit test.
Feb 12, 2019
4413b65
936: Added more unit tests.
Feb 12, 2019
20daa8f
936: GenericJsonTest fixed because toString was updated in GenericData.
Feb 12, 2019
fbc0256
936: Changes after review.
Feb 13, 2019
3c3468d
Update GenericData.java
sduskis Feb 13, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
936: Added more unit tests.
  • Loading branch information
andrey-qlogic committed Feb 12, 2019
commit 4413b65cfdd6dd87fccf119fb5821a5309f5c2ee
26 changes: 24 additions & 2 deletions google-http-client/src/test/java/com/google/api/client/util/GenericDataTest.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -70,16 +70,38 @@ public void testEquals_Symmetric() {
GenericData actual = new GenericData1();
actual.set("fieldA", "bar");
GenericData expected = new GenericData2();
// Test that objects equal
// Test that objects are equal
expected.set("fieldA", "bar");
assertNotSame(expected, actual);
assertTrue(expected.equals(expected) && actual.equals(actual));
// Test that objects not equal
// Test that objects not are equal
expected.set("fieldA", "far");
assertFalse(expected.equals(actual) || actual.equals(expected));
assertFalse(expected.hashCode() == actual.hashCode());
}

public void testEquals_SymmetricWithSameClass() {
GenericData actual = new MyData();
actual.set("fieldA", "bar");
GenericData expected = new MyData();
// Test that objects are equal
expected.set("fieldA", "bar");
assertNotSame(expected, actual);
assertTrue(expected.equals(expected) && actual.equals(actual));
assertTrue(expected.hashCode() == expected.hashCode());
}

public void testNotEquals_SymmetricWithSameClass() {
GenericData actual = new MyData();
actual.set("fieldA", "bar");
GenericData expected = new MyData();
// Test that objects are not equal
expected.set("fieldA", "far");
assertNotSame(expected, actual);
assertFalse(expected.equals(actual) || actual.equals(expected));
assertFalse(expected.hashCode() == actual.hashCode());
}

public void testClone_changingEntrySet() {
GenericData data = new GenericData();
assertEquals("GenericData{classInfo=[], {}}", data.toString());
Expand Down