Remove TODO comment

rneatherway · rneatherway · commit 7850d67a782d · 2019-11-06T17:47:02.000Z
I've checked Hamcrest versions 1.3, 2.0, 2.1 and 2.2
diff --git a/java/ql/src/semmle/code/java/frameworks/Assertions.qll b/java/ql/src/semmle/code/java/frameworks/Assertions.qll
@@ -30,7 +30,7 @@ private predicate assertionMethod(Method m, AssertKind kind) {
   or
   exists(RefType hamcrest |
     m.getDeclaringType() = hamcrest and
-    hamcrest.hasQualifiedName("org.hamcrest", "MatcherAssert") // TODO: Check some older versions of hamcrest
+    hamcrest.hasQualifiedName("org.hamcrest", "MatcherAssert")
   |
     m.hasName("assertThat") and kind = AssertKindThat()
   )

Original file line number	Diff line number	Diff line change
`@@ -30,7 +30,7 @@ private predicate assertionMethod(Method m, AssertKind kind) {`
`30`	`30`	`or`
`31`	`31`	`exists(RefType hamcrest \|`
`32`	`32`	`m.getDeclaringType() = hamcrest and`
`33`		`- hamcrest.hasQualifiedName("org.hamcrest", "MatcherAssert") // TODO: Check some older versions of hamcrest`
	`33`	`+ hamcrest.hasQualifiedName("org.hamcrest", "MatcherAssert")`
`34`	`34`	`\|`
`35`	`35`	`m.hasName("assertThat") and kind = AssertKindThat()`
`36`	`36`	`)`