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: Modified unit test.
  • Loading branch information
andrey-qlogic committed Feb 12, 2019
commit 75a971436d4488a70eadb994fd4271f2bc983ae8
29 changes: 23 additions & 6 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 @@ -48,17 +48,34 @@ public void setFieldB(List<String> fieldB) {
}
}

private class GenericData1 extends GenericData {
public GenericData1() {
super(EnumSet.of(Flags.IGNORE_CASE));
}

@Key("FieldA")
public String fieldA;
}

private class GenericData2 extends GenericData {
public GenericData2() {
super(EnumSet.of(Flags.IGNORE_CASE));
}

@Key("FieldA")
public String fieldA;
}

public void testEquals_Symmetric() {
GenericData actual = new GenericData();
actual.set("foo", "bar");
GenericData expected = new GenericData();
GenericData actual = new GenericData1();
actual.set("fieldA", "bar");
GenericData expected = new GenericData2();
// Test that objects equal
expected.set("foo", "bar");
expected.set("fieldA", "bar");
assertNotSame(expected, actual);
assertEquals(expected.hashCode(), actual.hashCode());
assertTrue(expected.equals(expected) && actual.equals(actual));
// Test that objects not equal
expected.set("foo", "far");
expected.set("fieldA", "far");
assertFalse(expected.equals(actual) || actual.equals(expected));
assertFalse(expected.hashCode() == actual.hashCode());
}
Expand Down