From d8c8ab30778cd179a6d6430bbd05eed5e777684a Mon Sep 17 00:00:00 2001
From: Michal Kordas <kontakt@michalkordas.com>
Date: Sun, 17 May 2015 23:06:03 +0200
Subject: [PATCH] Use EqualsVerifier to test FilterSet and remove obsolete
 code. #1088

---
 pom.xml                                       |  1 -
 .../tools/checkstyle/api/FilterSet.java       | 32 +++++++------------
 .../checkstyle/filters/FilterSetTest.java     |  8 +++++
 3 files changed, 20 insertions(+), 21 deletions(-)

diff --git a/pom.xml b/pom.xml
index 3b9d6c3d7..882c3aaab 100644
--- a/pom.xml
+++ b/pom.xml
@@ -665,7 +665,6 @@
           <regex><pattern>.*.api.DetailAST</pattern><branchRate>95</branchRate><lineRate>98</lineRate></regex>
           <regex><pattern>.*.api.FileContents</pattern><branchRate>96</branchRate><lineRate>94</lineRate></regex>
           <regex><pattern>.*.api.FileText</pattern><branchRate>50</branchRate><lineRate>59</lineRate></regex>
-          <regex><pattern>.*.api.FilterSet</pattern><branchRate>83</branchRate><lineRate>80</lineRate></regex>
           <regex><pattern>.*.api.FullIdent</pattern><branchRate>83</branchRate><lineRate>96</lineRate></regex>
           <regex><pattern>.*.api.JavadocTagInfo</pattern><branchRate>25</branchRate><lineRate>77</lineRate></regex>
           <regex><pattern>.*.api.JavadocTagInfo\$.*</pattern><branchRate>0</branchRate><lineRate>8</lineRate></regex>
diff --git a/src/main/java/com/puppycrawl/tools/checkstyle/api/FilterSet.java b/src/main/java/com/puppycrawl/tools/checkstyle/api/FilterSet.java
index 4b989ce8a..91b89a930 100644
--- a/src/main/java/com/puppycrawl/tools/checkstyle/api/FilterSet.java
+++ b/src/main/java/com/puppycrawl/tools/checkstyle/api/FilterSet.java
@@ -20,6 +20,8 @@
 package com.puppycrawl.tools.checkstyle.api;
 
 import com.google.common.collect.Sets;
+
+import java.util.Objects;
 import java.util.Set;
 
 /**
@@ -49,31 +51,21 @@ public class FilterSet
         filters.remove(filter);
     }
 
-    /**
-     * Returns the Filters of the filter set.
-     * @return the Filters of the filter set.
-     */
-    protected Set<Filter> getFilters() {
-        return filters;
-    }
-
     @Override
-    public String toString() {
-        return filters.toString();
+    public boolean equals(Object o) {
+        if (this == o) {
+            return true;
+        }
+        if (o == null || getClass() != o.getClass()) {
+            return false;
+        }
+        final FilterSet filterSet = (FilterSet) o;
+        return Objects.equals(filters, filterSet.filters);
     }
 
     @Override
     public int hashCode() {
-        return filters.hashCode();
-    }
-
-    @Override
-    public boolean equals(Object object) {
-        if (object instanceof FilterSet) {
-            final FilterSet other = (FilterSet) object;
-            return this.filters.equals(other.filters);
-        }
-        return false;
+        return Objects.hash(filters);
     }
 
     /** {@inheritDoc} */
diff --git a/src/test/java/com/puppycrawl/tools/checkstyle/filters/FilterSetTest.java b/src/test/java/com/puppycrawl/tools/checkstyle/filters/FilterSetTest.java
index da7813cb9..110bed1b8 100644
--- a/src/test/java/com/puppycrawl/tools/checkstyle/filters/FilterSetTest.java
+++ b/src/test/java/com/puppycrawl/tools/checkstyle/filters/FilterSetTest.java
@@ -21,6 +21,9 @@ package com.puppycrawl.tools.checkstyle.filters;
 
 import static org.junit.Assert.assertFalse;
 import static org.junit.Assert.assertTrue;
+
+import com.puppycrawl.tools.checkstyle.api.FilterSet;
+import nl.jqno.equalsverifier.EqualsVerifier;
 import org.junit.Before;
 import org.junit.Test;
 
@@ -54,4 +57,9 @@ public class FilterSetTest {
         filter.addFilter(new IntRangeFilter(3, 4));
         assertTrue("0 is in [3,4]", filter.accept(Integer.valueOf(0)));
     }
+
+    @Test
+    public void testEqualsAndHashCode() {
+        EqualsVerifier.forClass(FilterSet.class).usingGetClass().verify();
+    }
 }