Add modernizer check for assertEquals(Iterable, Iterable[, String])

This prevents false positive test results when comparing
iterables (most notably: MaterializedResult instances) due to
TestNG bug (testng-team/testng#543).

False-positive results were found in ~20 tests and are fixed
in the following commits.

Author: ArturGajowy

pom.xml

@@ -883,6 +883,9 @@
                     <groupId>org.gaul</groupId>
                     <artifactId>modernizer-maven-plugin</artifactId>
                     <configuration>
+                        <violationsFiles>
+                            <violationsFile>${air.main.basedir}/src/modernizer/violations.xml</violationsFile>
+                        </violationsFiles>
                         <exclusionPatterns>
                             <exclusionPattern>org/joda/time/.*</exclusionPattern>
                         </exclusionPatterns>

presto-main/pom.xml

@@ -335,6 +335,7 @@
                      <!-- To allow usage of java.util.Hashtable -->
                      <ignorePackages>
                          <ignorePackage>com.facebook.presto.server.security.util.jndi</ignorePackage>
+                         <ignorePackage>com.facebook.presto.testing.assertions</ignorePackage>
                      </ignorePackages>
                 </configuration>
             </plugin>

src/modernizer/violations.xml

@@ -0,0 +1,14 @@
+<?xml version="1.0"?>
+<modernizer>
+    <violation>
+        <name>org/testng/Assert.assertEquals:(Ljava/lang/Iterable;Ljava/lang/Iterable;)V</name>
+        <version>1.8</version>
+        <comment>Use com.facebook.presto.testing.assertions.Assert.assertEquals due to TestNG #543</comment>
+    </violation>
+
+    <violation>
+        <name>org/testng/Assert.assertEquals:(Ljava/lang/Iterable;Ljava/lang/Iterable;Ljava/lang/String;)V</name>
+        <version>1.8</version>
+        <comment>Use com.facebook.presto.testing.assertions.Assert.assertEquals due to TestNG #543</comment>
+    </violation>
+</modernizer>