JAL-1889 ignore 'synthetic' fields added for jacoco
authorgmungoc <g.m.carstairs@dundee.ac.uk>
Thu, 3 Jan 2019 09:16:15 +0000 (09:16 +0000)
committergmungoc <g.m.carstairs@dundee.ac.uk>
Thu, 3 Jan 2019 09:16:15 +0000 (09:16 +0000)
test/jalview/viewmodel/styles/ViewStyleTest.java

index bc74d8a..7deb2b0 100644 (file)
@@ -100,24 +100,23 @@ public class ViewStyleTest
   private boolean copyConstructorIgnores(Field field)
   {
     /*
-     * just instrumentation added by test coverage while testing
+     * ignore instrumentation added by jacoco for test coverage
      */
-    String type = field.getClass().toString();
-    if (type.toString().contains("com_atlassian_clover"))
+    if (field.isSynthetic())
     {
-      // instrumentation added for test coverage - ignore
       return true;
     }
+
     return false;
   }
-
+  
   /**
-   * Change the value of one field in a ViewStyle object
-   * 
-   * @param vs
-   * @param field
-   * @throws IllegalAccessException
-   */
+  * Change the value of one field in a ViewStyle object
+  * 
+  * @param vs
+  * @param field
+  * @throws IllegalAccessException
+  */
   protected void changeValue(ViewStyle vs, Field field)
           throws IllegalAccessException
   {