You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jmeter.apache.org by se...@apache.org on 2014/11/29 13:34:56 UTC
svn commit: r1642421 -
/jmeter/trunk/test/src/org/apache/jmeter/resources/PackageTest.java
Author: sebb
Date: Sat Nov 29 12:34:56 2014
New Revision: 1642421
URL: http://svn.apache.org/r1642421
Log:
Test needs to fail if no files found
Modified:
jmeter/trunk/test/src/org/apache/jmeter/resources/PackageTest.java
Modified: jmeter/trunk/test/src/org/apache/jmeter/resources/PackageTest.java
URL: http://svn.apache.org/viewvc/jmeter/trunk/test/src/org/apache/jmeter/resources/PackageTest.java?rev=1642421&r1=1642420&r2=1642421&view=diff
==============================================================================
--- jmeter/trunk/test/src/org/apache/jmeter/resources/PackageTest.java (original)
+++ jmeter/trunk/test/src/org/apache/jmeter/resources/PackageTest.java Sat Nov 29 12:34:56 2014
@@ -261,10 +261,7 @@ public class PackageTest extends TestCas
private static void findFile(File file, Set<String> set,
FilenameFilter filenameFilter) {
File[] foundFiles = file.listFiles(filenameFilter);
- if (foundFiles == null) { // Better error than NPE
- System.err.println("Not a directory: "+file);
- return;
- }
+ assertNotNull("Not a directory: "+file, foundFiles);
for (File file2 : foundFiles) {
if(file2.isDirectory()) {
findFile(file2, set, filenameFilter);