rename Routes.mapper() to Routes.map() fix #385

Author: jknack

coverage-report/src/test/java/org/jooby/assets/AssetsBase.java

@@ -17,7 +17,7 @@ public List<Object> list(final Object... args) {
     return Arrays.asList(args);
   }
 
-  public Map<String, Object> map(final Object... args) {
+  public Map<String, Object> hash(final Object... args) {
     ImmutableMap.Builder<String, Object> builder = ImmutableMap.builder();
     for (int i = 0; i < args.length; i += 2) {
       builder.put(args[i].toString(), args[i + 1]);
@@ -27,6 +27,6 @@ public Map<String, Object> map(final Object... args) {
 
   public Config assets(final String dist, final Object... args) {
     return ConfigFactory.empty()
-        .withValue("assets", ConfigValueFactory.fromMap(map(args)));
+        .withValue("assets", ConfigValueFactory.fromMap(hash(args)));
   }
 }

coverage-report/src/test/java/org/jooby/assets/NoPipelineFeature.java

@@ -9,7 +9,7 @@ public class NoPipelineFeature extends AssetsBase {
   {
 
     use(assets("dev", "basedir", "org/jooby/assets", "fileset",
-        map("home", list("js/index.js", "css/index.css")), "watch", false));
+        hash("home", list("js/index.js", "css/index.css")), "watch", false));
 
     use(new Hbs("/org/jooby/assets"));
     use(new Assets());

coverage-report/src/test/java/org/jooby/issues/Issue367.java

@@ -11,9 +11,9 @@
 public class Issue367 extends ServerFeature {
 
   {
-    mapper(Rx.rx());
+    map(Rx.rx());
 
-    mapper(Reactor.reactor());
+    map(Reactor.reactor());
 
     get("/rx", () -> Observable.just("reactive"));
 

coverage-report/src/test/java/org/jooby/issues/Issue384a.java

@@ -8,8 +8,8 @@ public class Issue384a extends ServerFeature {
 
   {
 
-    mapper(Route.Mapper.create("counter", (final Integer v) -> v + 1));
-    mapper(Route.Mapper.create("counter", (final Integer v) -> v + 1));
+    map(Route.Mapper.create("counter", (final Integer v) -> v + 1));
+    map(Route.Mapper.create("counter", (final Integer v) -> v + 1));
 
     get("/counter", () -> 1);
   }

coverage-report/src/test/java/org/jooby/issues/Issue384b.java

@@ -7,9 +7,9 @@ public class Issue384b extends ServerFeature {
 
   {
 
-    mapper(((final Integer value) -> value * 2));
+    map(((final Integer value) -> value * 2));
 
-    mapper((v -> Integer.parseInt(v.toString())));
+    map((v -> Integer.parseInt(v.toString())));
 
     get("/4", () -> {
       return "2";

jooby-mongodb-rx/src/main/java/org/jooby/mongodb/MongoRx.java

@@ -389,8 +389,8 @@ public void configure(final Env env, final Config conf, final Binder binder) {
 
     /** mapper */
     env.routes()
-        .mapper(Rx.rx())
-        .mapper(mapper());
+        .map(Rx.rx())
+        .map(mapper());
 
     log.info("Started {}", cstr);
 

jooby-mongodb-rx/src/test/java/org/jooby/mongodb/MongodbRxTest.java

@@ -108,8 +108,8 @@ public class MongodbRxTest {
     expect(Rx.rx()).andReturn(rx);
 
     Routes routes = unit.mock(Routes.class);
-    expect(routes.mapper(rx)).andReturn(routes);
-    expect(routes.mapper(unit.capture(Route.Mapper.class))).andReturn(routes);
+    expect(routes.map(rx)).andReturn(routes);
+    expect(routes.map(unit.capture(Route.Mapper.class))).andReturn(routes);
 
     Env env = unit.get(Env.class);
     expect(env.routes()).andReturn(routes);

jooby/src/main/java/org/jooby/Jooby.java

@@ -3551,7 +3551,7 @@ public void start(final String[] args, final Consumer<List<Route.Definition>> ro
 
   @Override
   @SuppressWarnings("unchecked")
-  public Jooby mapper(final Mapper<?> mapper) {
+  public Jooby map(final Mapper<?> mapper) {
     requireNonNull(mapper, "Mapper is required.");
     if (mappers.add(mapper.name())) {
       this.mapper = Optional.ofNullable(this.mapper)

jooby/src/main/java/org/jooby/Routes.java

@@ -2477,12 +2477,12 @@ Route.Collection complete(String method, String pattern, Route.Complete handler,
    * }
    * }</pre>
    *
-   * A call to <code>/four</code> outputs <code>4</code>
+   * A call to <code>/four</code> outputs <code>4</code>.
    *
-   * @param mapper
+   * @param mapper Route mapper to append.
    * @return This instance.
    */
-  Routes mapper(final Mapper<?> mapper);
+  Routes map(final Mapper<?> mapper);
 
   /**
    * Setup a route error handler. Default error handler {@link Err.DefHandler} does content