You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lens.apache.org by am...@apache.org on 2015/01/23 09:35:27 UTC

incubator-lens git commit: LENS-32 : Fix concurrency issues in CLI tests (amareshwari)

Repository: incubator-lens
Updated Branches:
  refs/heads/master db436ebd7 -> dbef65b9f


LENS-32 : Fix concurrency issues in CLI tests (amareshwari)


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

Branch: refs/heads/master
Commit: dbef65b9f1b023b82c0ab69149c76ccdd0dfb357
Parents: db436eb
Author: Amareshwari Sriramdasu <am...@inmobi.com>
Authored: Fri Jan 23 14:05:16 2015 +0530
Committer: Amareshwari Sriramdasu <am...@inmobi.com>
Committed: Fri Jan 23 14:05:16 2015 +0530

----------------------------------------------------------------------
 .../org/apache/lens/cli/TestLensDimensionTableCommands.java    | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-lens/blob/dbef65b9/lens-cli/src/test/java/org/apache/lens/cli/TestLensDimensionTableCommands.java
----------------------------------------------------------------------
diff --git a/lens-cli/src/test/java/org/apache/lens/cli/TestLensDimensionTableCommands.java b/lens-cli/src/test/java/org/apache/lens/cli/TestLensDimensionTableCommands.java
index 8ecb536..6f822b9 100644
--- a/lens-cli/src/test/java/org/apache/lens/cli/TestLensDimensionTableCommands.java
+++ b/lens-cli/src/test/java/org/apache/lens/cli/TestLensDimensionTableCommands.java
@@ -75,10 +75,10 @@ public class TestLensDimensionTableCommands extends LensCliApplicationTest {
    * @param storageName
    *          the storage name
    */
-  public static void addDim1Table(String tableName, String specName, String storageSpecName, String storageName) {
+  public synchronized static void addDim1Table(String tableName, String specName, String storageSpecName,
+      String storageName) {
     LensDimensionTableCommands command = getCommand();
     String dimList = command.showDimensionTables();
-    Assert.assertEquals("No Dimensions Found", dimList, "Dim tables should not be found");
     // add local storage before adding fact table
     TestLensStorageCommands.addLocalStorage(storageName);
     URL dimSpec = TestLensDimensionTableCommands.class.getClassLoader().getResource(specName);
@@ -93,7 +93,7 @@ public class TestLensDimensionTableCommands extends LensCliApplicationTest {
     }
 
     dimList = command.showDimensionTables();
-    Assert.assertEquals(tableName, dimList, "dim_table table should be found");
+    Assert.assertTrue(dimList.contains(tableName), "dim_table table should be found");
   }
 
   /**