package org.smoothbuild.task.exec;

import static org.smoothbuild.task.save.ArtifactPaths.artifactPath;

import java.util.Map;
import java.util.Map.Entry;
import java.util.TreeMap;

import javax.inject.Inject;

import org.smoothbuild.cli.Console;
import org.smoothbuild.lang.expr.Expression;
import org.smoothbuild.lang.function.base.Function;
import org.smoothbuild.lang.function.base.Name;
import org.smoothbuild.lang.message.Location;
import org.smoothbuild.lang.value.Value;
import org.smoothbuild.task.base.Task;
import org.smoothbuild.task.save.ArtifactSaver;
import org.smoothbuild.util.Dag;

public class ArtifactBuilder {
  private final ArtifactSaver artifactSaver;
  private final TaskBatch taskBatch;
  private final Console console;
  private final Map<Name, Dag<Task>> artifacts;

  @Inject
  public ArtifactBuilder(ArtifactSaver artifactSaver, TaskBatch taskBatch, Console console) {
    this.artifactSaver = artifactSaver;
    this.taskBatch = taskBatch;
    this.console = console;
    this.artifacts = new TreeMap<>((name1, name2) -> name1.toString().compareTo(name2.toString()));
  }

  public void addArtifact(Function function) {
    Expression expression = function.createCallExpression(false, Location.commandLine());
    Dag<Task> task = taskBatch.createTasks(new Dag<>(expression));
    artifacts.put(function.name(), task);
  }

  public void runBuild() {
    taskBatch.executeAll();
    if (!taskBatch.containsErrors()) {
      console.println("built artifact(s):");
      for (Entry<Name, Dag<Task>> artifact : artifacts.entrySet()) {
        Name name = artifact.getKey();
        Task task = artifact.getValue().elem();
        Value value = task.output().result();
        artifactSaver.save(name, value);
        console.println(name.toString() + " -> " + artifactPath(name));
      }
    }
  }
}

package org.smoothbuild.task.exec;

import static org.smoothbuild.task.save.ArtifactPaths.artifactPath;

import java.util.Map;
import java.util.Map.Entry;
import java.util.TreeMap;

import javax.inject.Inject;

import org.smoothbuild.cli.Console;
import org.smoothbuild.lang.expr.Expression;
import org.smoothbuild.lang.function.base.Function;
import org.smoothbuild.lang.function.base.Name;
import org.smoothbuild.lang.message.Location;
import org.smoothbuild.lang.value.Value;
import org.smoothbuild.task.base.Task;
import org.smoothbuild.task.save.ArtifactSaver;
import org.smoothbuild.util.Dag;

public class ArtifactBuilder {
  private final ArtifactSaver artifactSaver;
  private final TaskBatch taskBatch;
  private final Console console;
  private final Map<Name, Dag<Task>> artifacts;

  @Inject
  public ArtifactBuilder(ArtifactSaver artifactSaver, TaskBatch taskBatch, Console console) {
    this.artifactSaver = artifactSaver;
    this.taskBatch = taskBatch;
    this.console = console;
    this.artifacts = new TreeMap<>((name1, name2) -> name1.toString().compareTo(name2.toString()));
  }

  public void addArtifact(Function function) {
    Expression expression = function.createCallExpression(false, Location.commandLine());
    Dag<Task> task = taskBatch.createTasks(new Dag<>(expression));
    artifacts.put(function.name(), task);
  }

  public void runBuild() {
    taskBatch.executeAll();
    if (!taskBatch.containsErrors()) {
      console.println("\nbuilt artifact(s):");
      for (Entry<Name, Dag<Task>> artifact : artifacts.entrySet()) {
        Name name = artifact.getKey();
        Task task = artifact.getValue().elem();
        Value value = task.output().result();
        artifactSaver.save(name, value);
        console.println(name.toString() + " -> " + artifactPath(name));
      }
    }
  }
}

add empty line before printing "built artifacts" to console