FORGE-2671: Using UIRegionBuilder in SelectionsTest

forge · Aug 3, 2016 · 9718dd6 · 9718dd6
1 parent ce4aa39
commit 9718dd6
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 51 deletions.
diff --git a/ui/api/src/main/java/org/jboss/forge/addon/ui/context/UIRegionBuilder.java b/ui/api/src/main/java/org/jboss/forge/addon/ui/context/UIRegionBuilder.java
@@ -10,7 +10,8 @@
 import java.util.Optional;
 
 /**
- *
+ * Builder implementation of {@link UIRegion}
+ * 
  * @author <a href="mailto:ggastald@redhat.com">George Gastaldi</a>
  */
 public class UIRegionBuilder<SELECTIONTYPE> implements UIRegion<SELECTIONTYPE>

diff --git a/ui/api/src/test/java/org/jboss/forge/addon/ui/util/SelectionsTest.java b/ui/api/src/test/java/org/jboss/forge/addon/ui/util/SelectionsTest.java
@@ -9,9 +9,8 @@
 
 import java.util.Arrays;
 import java.util.List;
-import java.util.Optional;
 
-import org.jboss.forge.addon.ui.context.UIRegion;
+import org.jboss.forge.addon.ui.context.UIRegionBuilder;
 import org.jboss.forge.addon.ui.context.UISelection;
 import org.junit.Assert;
 import org.junit.Test;
@@ -22,59 +21,12 @@
  */
 public class SelectionsTest
 {
-
    @Test
    public void testRegionIsPresent()
    {
       List<String> list = Arrays.asList("A", "B", "C");
-      UISelection<String> selection = Selections.from(MockUIRegion::new, list);
+      UISelection<String> selection = Selections.from(r -> UIRegionBuilder.create(r).text(r), list);
       Assert.assertTrue(selection.getRegion().isPresent());
       Assert.assertEquals("A", selection.getRegion().get().getText().get());
    }
-
-   private class MockUIRegion implements UIRegion<String>
-   {
-      private String resource;
-
-      public MockUIRegion(String resource)
-      {
-         this.resource = resource;
-      }
-
-      @Override
-      public int getStartPosition()
-      {
-         return 0;
-      }
-
-      @Override
-      public int getEndPosition()
-      {
-         return 0;
-      }
-
-      @Override
-      public int getStartLine()
-      {
-         return 0;
-      }
-
-      @Override
-      public int getEndLine()
-      {
-         return 0;
-      }
-
-      @Override
-      public Optional<String> getText()
-      {
-         return Optional.of(resource);
-      }
-
-      @Override
-      public String getResource()
-      {
-         return resource;
-      }
-   }
 }