Review

Detector:

Tikanga

Target:

project '

testng

' version

677302c

Tags:

Potential Misuse

Anomaly identified by the detector. Please review whether this anomaly corresponds to a misuse.

Finding:	finding-14
In File:	org/testng/reporters/EmailableReporter2.java
In Method:	groupResults(Set)
Code with Finding:	class EmailableReporter2.TestResult { /** * Groups test results by method and then by class. */ protected List<ClassResult> groupResults(Set<ITestResult> results) { List<ClassResult> classResults = Lists.newArrayList(); if (!results.isEmpty()) { List<MethodResult> resultsPerClass = Lists.newArrayList(); List<ITestResult> resultsPerMethod = Lists.newArrayList(); List<ITestResult> resultsList = Lists.newArrayList(results); Collections.sort(resultsList, RESULT_COMPARATOR); Iterator<ITestResult> resultsIterator = resultsList.iterator(); assert resultsIterator.hasNext(); ITestResult result = resultsIterator.next(); resultsPerMethod.add(result); String previousClassName = result.getTestClass().getName(); String previousMethodName = result.getMethod().getMethodName(); while (resultsIterator.hasNext()) { result = resultsIterator.next(); String className = result.getTestClass().getName(); if (!previousClassName.equals(className)) { // Different class implies different method assert !resultsPerMethod.isEmpty(); resultsPerClass.add(new MethodResult(resultsPerMethod)); resultsPerMethod = Lists.newArrayList(); assert !resultsPerClass.isEmpty(); classResults.add(new ClassResult(previousClassName, resultsPerClass)); resultsPerClass = Lists.newArrayList(); previousClassName = className; previousMethodName = result.getMethod().getMethodName(); } else { String methodName = result.getMethod().getMethodName(); if (!previousMethodName.equals(methodName)) { assert !resultsPerMethod.isEmpty(); resultsPerClass.add(new MethodResult(resultsPerMethod)); resultsPerMethod = Lists.newArrayList(); previousMethodName = methodName; } } resultsPerMethod.add(result); } assert !resultsPerMethod.isEmpty(); resultsPerClass.add(new MethodResult(resultsPerMethod)); assert !resultsPerClass.isEmpty(); classResults.add(new ClassResult(previousClassName, resultsPerClass)); } return classResults; } }

Metadata

Hit	Rank	Confidence	Defect Indicator	Missing Properties	Present Properties	Supporting Objects	Violation Types
?	14	1.0	2.48	AF Iterator.hasNext () : boolean @ (0)	AG (Iterator.next () : Object @ (0) => EX EF Iterator.next () : Object @ (0)) AF Iterator.next () : Object @ (0) AG (Iterator.next () : Object @ (0) => EX EF Iterator.hasNext () : boolean @ (0)) EF Iterator.hasNext () : boolean @ (0) AG (Iterator.hasNext () : boolean @ (0) => EX EF Iterator.hasNext () : boolean @ (0)) AG (Iterator.hasNext () : boolean @ (0) => EX EF Iterator.next () : Object @ (0)) EF Iterator.next () : Object @ (0) AG (Iterator.hasNext () : boolean @ (0) => AX AF Iterator.next () : Object @ (0))	Assert.assertEquals (Collection, Collection, String) : void : var #20@Iterator (line 558) Assert.assertEquals (Collection, Collection, String) : void : var #21@Iterator (line 559) Assert.assertEquals (Iterator, Iterator, String) : void : param #1@Iterator (line 590)