Line endings and JavaDoc

Author: Ashley Frieze

docs/VariablesAndValues.md

@@ -94,8 +94,8 @@ E.g.
 Supplier<List<String>> list = let(ArrayList::new);
 
 it("can use the object", () -> {
-	list.get().add("Hello");
-	assertThat(list.get().get(0), is("Hello"));
+  list.get().add("Hello");
+  assertThat(list.get().get(0), is("Hello"));
 });
 ```
 
@@ -105,7 +105,7 @@ can be replaced by
 List<String> list = unbox(let(ArrayList::new), List.class);
 
 it("can use the object as though it was not in a supplier", () -> {
-	list.add("Hello");
-	assertThat(list.get(0), is("Hello"));
+  list.add("Hello");
+  assertThat(list.get(0), is("Hello"));
 });
 ```

src/main/java/com/greghaskins/spectrum/Unboxer.java

@@ -11,19 +11,20 @@
  * can be used with <code>let</code> and {@link Variable}
  */
 public interface Unboxer {
-	/**
-	 * Provide a proxy object to the contents of a supplier to reduce the number of
-	 * {@link Supplier#get} calls in your code. Note, if using {@link Variable} then you
-	 * will want to keep a reference to the original object so you can use {@link Variable#set}.
-	 * @param supplier supplier of object
-	 * @param asClass target type of the unboxer - must be interface
-	 * @param <T> type within the supplier
-	 * @param <R> type of the unboxer object
-	 * @return a proxy to the contents of the supplier
-	 */
-	@SuppressWarnings("unchecked")
-	static <T extends S, R extends S, S> R unbox(Supplier<T> supplier, Class<S> asClass) {
-		return (R) Proxy.newProxyInstance(asClass.getClassLoader(), new Class[] {asClass},
-			(Object proxy, Method method, Object[] args) -> method.invoke(supplier.get(), args));
-	}
+  /**
+   * Provide a proxy object to the contents of a supplier to reduce the number of
+   * {@link Supplier#get} calls in your code. Note, if using {@link Variable} then you
+   * will want to keep a reference to the original object so you can use {@link Variable#set}.
+   * @param supplier supplier of object
+   * @param asClass target type of the unboxer - must be interface
+   * @param <T> type within the supplier
+   * @param <R> (inferred) type of the unboxer object (allows generic types to be preserved)
+   * @param <S> class of the interface to return
+   * @return a proxy to the contents of the supplier
+   */
+  @SuppressWarnings("unchecked")
+  static <T extends S, R extends S, S> R unbox(Supplier<T> supplier, Class<S> asClass) {
+    return (R) Proxy.newProxyInstance(asClass.getClassLoader(), new Class[] {asClass},
+        (Object proxy, Method method, Object[] args) -> method.invoke(supplier.get(), args));
+  }
 }

src/test/java/specs/UnboxerSpecs.java

@@ -7,52 +7,53 @@
 import static org.hamcrest.core.Is.is;
 import static org.junit.Assert.assertThat;
 
-import java.util.ArrayList;
-import java.util.List;
-
 import com.greghaskins.spectrum.Spectrum;
 import com.greghaskins.spectrum.Variable;
+
 import org.junit.runner.RunWith;
 
+import java.util.ArrayList;
+import java.util.List;
+
 @RunWith(Spectrum.class)
 public class UnboxerSpecs {
-	{
-		describe("Using unboxer on let", () -> {
-			List<String> list = unbox(let(ArrayList::new), List.class);
-
-			it("can use the object as though it was not in a supplier", () -> {
-				list.add("Hello");
-				assertThat(list.get(0), is("Hello"));
-			});
-
-			it("can use multi-parameter methods correctly", () -> {
-				list.add("a");
-				list.add("b");
-				list.add(0, "_");
-
-				assertThat(list.size(), is(3));
-				assertThat(list.get(0), is("_"));
-			});
-		});
-
-		describe("Using unboxer with variable", () -> {
-			Variable<ArrayList<String>> listVariable = new Variable<>(new ArrayList<>());
-			List<String> list = unbox(listVariable, List.class);
-
-			it("can read the variable contents", () -> {
-				list.add("World");
-				assertThat(list.size(), is(1));
-				assertThat(list.get(0), is("World"));
-			});
-
-			it("can still read the same list in the next spec", () -> {
-				assertThat(list.size(), is(1));
-			});
-
-			it("can reset the content via the original variable object", () -> {
-				listVariable.set(new ArrayList<>());
-				assertThat(list.size(), is(0));
-			});
-		});
-	}
+  {
+    describe("Using unboxer on let", () -> {
+      List<String> list = unbox(let(ArrayList::new), List.class);
+
+      it("can use the object as though it was not in a supplier", () -> {
+        list.add("Hello");
+        assertThat(list.get(0), is("Hello"));
+      });
+
+      it("can use multi-parameter methods correctly", () -> {
+        list.add("a");
+        list.add("b");
+        list.add(0, "_");
+
+        assertThat(list.size(), is(3));
+        assertThat(list.get(0), is("_"));
+      });
+    });
+
+    describe("Using unboxer with variable", () -> {
+      Variable<ArrayList<String>> listVariable = new Variable<>(new ArrayList<>());
+      List<String> list = unbox(listVariable, List.class);
+
+      it("can read the variable contents", () -> {
+        list.add("World");
+        assertThat(list.size(), is(1));
+        assertThat(list.get(0), is("World"));
+      });
+
+      it("can still read the same list in the next spec", () -> {
+        assertThat(list.size(), is(1));
+      });
+
+      it("can reset the content via the original variable object", () -> {
+        listVariable.set(new ArrayList<>());
+        assertThat(list.size(), is(0));
+      });
+    });
+  }
 }