You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@geode.apache.org by kl...@apache.org on 2016/04/28 18:45:54 UTC

incubator-geode git commit: Cleanup test

Repository: incubator-geode
Updated Branches:
  refs/heads/feature/GEODE-1276 e4f513ef2 -> 1e90a7225


Cleanup test


Project: http://git-wip-us.apache.org/repos/asf/incubator-geode/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-geode/commit/1e90a722
Tree: http://git-wip-us.apache.org/repos/asf/incubator-geode/tree/1e90a722
Diff: http://git-wip-us.apache.org/repos/asf/incubator-geode/diff/1e90a722

Branch: refs/heads/feature/GEODE-1276
Commit: 1e90a72257437ed333afd464859375b6f5c3ea23
Parents: e4f513e
Author: Kirk Lund <kl...@apache.org>
Authored: Thu Apr 28 09:45:25 2016 -0700
Committer: Kirk Lund <kl...@apache.org>
Committed: Thu Apr 28 09:45:25 2016 -0700

----------------------------------------------------------------------
 .../management/internal/configuration/ZipUtilsJUnitTest.java  | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-geode/blob/1e90a722/geode-core/src/test/java/com/gemstone/gemfire/management/internal/configuration/ZipUtilsJUnitTest.java
----------------------------------------------------------------------
diff --git a/geode-core/src/test/java/com/gemstone/gemfire/management/internal/configuration/ZipUtilsJUnitTest.java b/geode-core/src/test/java/com/gemstone/gemfire/management/internal/configuration/ZipUtilsJUnitTest.java
index d45c962..595bdeb 100755
--- a/geode-core/src/test/java/com/gemstone/gemfire/management/internal/configuration/ZipUtilsJUnitTest.java
+++ b/geode-core/src/test/java/com/gemstone/gemfire/management/internal/configuration/ZipUtilsJUnitTest.java
@@ -19,11 +19,9 @@ package com.gemstone.gemfire.management.internal.configuration;
 import static org.junit.Assert.*;
 
 import java.io.File;
-import java.io.FileNotFoundException;
 
 import org.apache.commons.io.FileUtils;
 import org.apache.commons.io.FilenameUtils;
-import org.junit.After;
 import org.junit.Before;
 import org.junit.Rule;
 import org.junit.Test;
@@ -31,14 +29,14 @@ import org.junit.experimental.categories.Category;
 import org.junit.rules.TemporaryFolder;
 
 import com.gemstone.gemfire.management.internal.configuration.utils.ZipUtils;
-import com.gemstone.gemfire.test.junit.categories.UnitTest;
+import com.gemstone.gemfire.test.junit.categories.IntegrationTest;
 
 /**
  * JUnit Test for {@link ZipUtils}
  * 
  * TODO: this is broken on Windows: see bug #52036 (GEODE-312)
  */
-@Category(UnitTest.class)
+@Category(IntegrationTest.class)
 public class ZipUtilsJUnitTest {
   
   private String destinationFolderName = "destination";
@@ -90,7 +88,6 @@ public class ZipUtilsJUnitTest {
     assertTrue(destinationFolder.isDirectory());
     
     File[] destinationSubDirs = destinationFolder.listFiles();
-    //assertTrue((destinationSubDirs != null) && (destinationSubDirs.length != 0));
     assertNotNull(destinationSubDirs);
     assertEquals(2, destinationSubDirs.length);