Merge pull request #399 from HDouss/#195

#195 Add a test case

src/main/resources/org/jpeek/metrics/TCC.xsl

@@ -53,11 +53,6 @@ SOFTWARE.
     <xsl:variable name="NC" select="$methods_count * ($methods_count - 1) div 2"/>
     <xsl:variable name="directly-related-pairs">
       <xsl:for-each select="$methods">
-        <!--
-        @todo #120:30min TCC: need to come back and refactor the following after
-         #156 is fixed. The ops for fields must be properly filtered to ensure
-         that they belong to the enclosing class.
-        -->
         <xsl:variable name="i" select="position()"/>
         <xsl:variable name="left" select="."/>
         <xsl:variable name="left_attrs" select="$attrs[. = $left/ops/op/text()]"/>

src/test/java/org/jpeek/MetricsTest.java

@@ -174,6 +174,7 @@ public static Collection<Object[]> targets() {
             new Object[] {"Foo", "OCC", 0.5d},
             new Object[] {"Bar", "TCC", 0.0d},
             new Object[] {"Foo", "TCC", 1.0d},
+            new Object[] {"ClassSameAsAnotherPublicField", "TCC", 0.0d},
             new Object[] {"MethodsWithDiffParamTypes", "TCC", 0.2d},
             new Object[] {"OverloadMethods", "TCC", 1.0d},
             new Object[] {"TwoCommonAttributes", "TCC", 0.0d},

src/test/resources/org/jpeek/samples/ClassSameAsAnotherPublicField.java

@@ -0,0 +1,17 @@
+package org.jpeek.samples;
+
+public final class ClassSameAsAnotherPublicField {
+    private String NAME = "hey";
+
+    public void test() {
+        ClassWithPublicField.NAME = "test";
+    }
+
+    public void foo() {
+        this.NAME = "test";
+    }
+}
+
+class ClassWithPublicField {
+    public static String NAME = "yoo";
+}