From 5cdecca0d6f5d6ca21bea5c414ad43bf0efed6a3 Mon Sep 17 00:00:00 2001 From: Henry Coles Date: Thu, 11 Mar 2021 09:10:21 +0000 Subject: [PATCH] upgrade equalsverifier --- pitest/pom.xml | 2 +- .../src/test/java/org/pitest/coverage/BlockLocationTest.java | 4 +++- pitest/src/test/java/org/pitest/coverage/TestInfoTest.java | 4 +++- .../test/java/org/pitest/coverage/analysis/BlockTest.java | 5 ++++- .../org/pitest/mutationtest/engine/MutationDetailsTest.java | 4 +++- pitest/src/test/java/org/pitest/plugin/FeatureTest.java | 4 +++- .../org/pitest/testapi/execute/MultipleTestGroupTest.java | 4 +++- 7 files changed, 20 insertions(+), 7 deletions(-) diff --git a/pitest/pom.xml b/pitest/pom.xml index 24af74b71..3c2cbb2be 100644 --- a/pitest/pom.xml +++ b/pitest/pom.xml @@ -220,7 +220,7 @@ nl.jqno.equalsverifier equalsverifier - 1.7.5 + 3.5.4 test diff --git a/pitest/src/test/java/org/pitest/coverage/BlockLocationTest.java b/pitest/src/test/java/org/pitest/coverage/BlockLocationTest.java index 400129f38..11aa7e252 100644 --- a/pitest/src/test/java/org/pitest/coverage/BlockLocationTest.java +++ b/pitest/src/test/java/org/pitest/coverage/BlockLocationTest.java @@ -8,7 +8,9 @@ public class BlockLocationTest { @Test public void shouldObeyHashcodeEqualsContract() { - EqualsVerifier.forClass(BlockLocation.class).verify(); + EqualsVerifier.forClass(BlockLocation.class) + .withOnlyTheseFields("block", "location") + .verify(); } } diff --git a/pitest/src/test/java/org/pitest/coverage/TestInfoTest.java b/pitest/src/test/java/org/pitest/coverage/TestInfoTest.java index 8789f441c..e86a8de38 100644 --- a/pitest/src/test/java/org/pitest/coverage/TestInfoTest.java +++ b/pitest/src/test/java/org/pitest/coverage/TestInfoTest.java @@ -8,6 +8,8 @@ public class TestInfoTest { @Test public void shouldObeyHashcodeEqualsContract() { - EqualsVerifier.forClass(TestInfo.class).verify(); + EqualsVerifier.forClass(TestInfo.class) + .withOnlyTheseFields("name", "definingClass") + .verify(); } } diff --git a/pitest/src/test/java/org/pitest/coverage/analysis/BlockTest.java b/pitest/src/test/java/org/pitest/coverage/analysis/BlockTest.java index 8be88aec7..ce25e800b 100644 --- a/pitest/src/test/java/org/pitest/coverage/analysis/BlockTest.java +++ b/pitest/src/test/java/org/pitest/coverage/analysis/BlockTest.java @@ -8,7 +8,10 @@ public class BlockTest { @Test public void shouldObeyHashcodeEqualsContract() { - EqualsVerifier.forClass(Block.class).verify(); + EqualsVerifier.forClass(Block.class) + .withOnlyTheseFields("firstInstruction") + .withOnlyTheseFields("lastInstruction") + .verify(); } } diff --git a/pitest/src/test/java/org/pitest/mutationtest/engine/MutationDetailsTest.java b/pitest/src/test/java/org/pitest/mutationtest/engine/MutationDetailsTest.java index 20861d0f7..8dd2156cc 100644 --- a/pitest/src/test/java/org/pitest/mutationtest/engine/MutationDetailsTest.java +++ b/pitest/src/test/java/org/pitest/mutationtest/engine/MutationDetailsTest.java @@ -38,7 +38,9 @@ public void shouldDefaultFilenameWhenNoneKnown() { @Test public void shouldObeyHashcodeEqualsContract() { - EqualsVerifier.forClass(MutationDetails.class).verify(); + EqualsVerifier.forClass(MutationDetails.class) + .withOnlyTheseFields("id") + .verify(); } } diff --git a/pitest/src/test/java/org/pitest/plugin/FeatureTest.java b/pitest/src/test/java/org/pitest/plugin/FeatureTest.java index c57b1e9f6..1eb7b6264 100644 --- a/pitest/src/test/java/org/pitest/plugin/FeatureTest.java +++ b/pitest/src/test/java/org/pitest/plugin/FeatureTest.java @@ -10,7 +10,9 @@ public class FeatureTest { @Test public void shouldObeyHashcodeEqualsContract() { - EqualsVerifier.forClass(Feature.class).verify(); + EqualsVerifier.forClass(Feature.class) + .withOnlyTheseFields("name") + .verify(); } @Test diff --git a/pitest/src/test/java/org/pitest/testapi/execute/MultipleTestGroupTest.java b/pitest/src/test/java/org/pitest/testapi/execute/MultipleTestGroupTest.java index 0a766552b..f3bb2b4a8 100644 --- a/pitest/src/test/java/org/pitest/testapi/execute/MultipleTestGroupTest.java +++ b/pitest/src/test/java/org/pitest/testapi/execute/MultipleTestGroupTest.java @@ -73,7 +73,9 @@ public void shouldStopExecutingWhenResultCollectorRequestsStop() { @Test public void shouldObeyHashcodeEqualsContract() { - EqualsVerifier.forClass(MultipleTestGroup.class).verify(); + EqualsVerifier.forClass(MultipleTestGroup.class) + .withOnlyTheseFields("children") + .verify(); } }