google#159 Refactor NonReversibleValidationException package

Categorize it into exception package and solve related
build dependencies.

Author: ignite1771

java/com/google/copybara/BUILD

@@ -99,7 +99,6 @@ BASE_SRCS = [
     "LocalParallelizer.java",
     "Metadata.java",
     "MigrationInfo.java",
-    "NonReversibleValidationException.java",
     "Option.java",
     "Options.java",
     "Origin.java",

java/com/google/copybara/Core.java

@@ -35,6 +35,7 @@
 import com.google.copybara.doc.annotations.Example;
 import com.google.copybara.doc.annotations.UsesFlags;
 import com.google.copybara.exception.EmptyChangeException;
+import com.google.copybara.exception.NonReversibleValidationException;
 import com.google.copybara.feedback.Action;
 import com.google.copybara.feedback.Feedback;
 import com.google.copybara.feedback.StarlarkAction;

java/com/google/copybara/MapMapper.java

@@ -19,6 +19,7 @@
 import com.google.common.base.Preconditions;
 import com.google.common.collect.ImmutableBiMap;
 import com.google.common.collect.ImmutableMap;
+import com.google.copybara.exception.NonReversibleValidationException;
 import com.google.copybara.transform.ReversibleFunction;
 
 public class MapMapper implements ReversibleFunction<String, String> {

java/com/google/copybara/Transformation.java

@@ -16,6 +16,7 @@
 
 package com.google.copybara;
 
+import com.google.copybara.exception.NonReversibleValidationException;
 import com.google.copybara.exception.RepoException;
 import com.google.copybara.exception.ValidationException;
 import java.io.IOException;

java/com/google/copybara/buildozer/BuildozerBatch.java

@@ -19,7 +19,7 @@
 import com.google.common.base.Preconditions;
 import com.google.common.collect.ImmutableList;
 import com.google.common.collect.Iterables;
-import com.google.copybara.NonReversibleValidationException;
+import com.google.copybara.exception.NonReversibleValidationException;
 import com.google.copybara.TransformWork;
 import com.google.copybara.Transformation;
 import com.google.copybara.WorkflowOptions;

java/com/google/copybara/buildozer/BuildozerDelete.java

@@ -20,7 +20,7 @@
 
 import com.google.common.base.MoreObjects;
 import com.google.common.collect.ImmutableList;
-import com.google.copybara.NonReversibleValidationException;
+import com.google.copybara.exception.NonReversibleValidationException;
 import com.google.copybara.TransformWork;
 import com.google.copybara.Transformation;
 import com.google.copybara.WorkflowOptions;

java/com/google/copybara/buildozer/BuildozerModify.java

@@ -20,7 +20,7 @@
 
 import com.google.common.base.MoreObjects;
 import com.google.common.collect.ImmutableList;
-import com.google.copybara.NonReversibleValidationException;
+import com.google.copybara.exception.NonReversibleValidationException;
 import com.google.copybara.TransformWork;
 import com.google.copybara.Transformation;
 import com.google.copybara.WorkflowOptions;

java/com/google/copybara/buildozer/Command.java

@@ -21,7 +21,7 @@
 import com.google.common.base.Joiner;
 import com.google.common.base.Preconditions;
 import com.google.common.base.Splitter;
-import com.google.copybara.NonReversibleValidationException;
+import com.google.copybara.exception.NonReversibleValidationException;
 import java.util.ArrayList;
 import java.util.Collections;
 import java.util.List;

java/com/google/copybara/exception/BUILD

@@ -28,5 +28,8 @@ java_library(
         ["**/*.java"],
     ),
     javacopts = JAVACOPTS,
-    deps = ["//third_party:error_prone"],
+    deps = [
+        "//third_party:starlark",
+        "//third_party:error_prone"
+    ],
 )

java/com/google/copybara/NonReversibleValidationException.java renamed to java/com/google/copybara/exception/NonReversibleValidationException.java

@@ -14,15 +14,13 @@
  * limitations under the License.
  */
 
-package com.google.copybara;
+package com.google.copybara.exception;
 
 import net.starlark.java.eval.EvalException;
 
 /**
- * Exception thrown when a {@link Transformation} is not reversible but the configuration asked for
- * the reverse.
- *
- * TODO(malcon): Move to the exception package
+ * Exception thrown when a {@link com.google.copybara.Transformation} is not reversible
+ * but the configuration asked for the reverse.
  */
 public class NonReversibleValidationException extends EvalException {
 

java/com/google/copybara/format/BuildifierFormat.java

@@ -27,7 +27,7 @@
 import com.google.common.collect.Iterables;
 import com.google.common.flogger.FluentLogger;
 import com.google.copybara.GeneralOptions;
-import com.google.copybara.NonReversibleValidationException;
+import com.google.copybara.exception.NonReversibleValidationException;
 import com.google.copybara.TransformWork;
 import com.google.copybara.Transformation;
 import com.google.copybara.WorkflowOptions;

java/com/google/copybara/transform/CopyOrMove.java

@@ -21,7 +21,7 @@
 import com.google.common.base.MoreObjects;
 import com.google.common.base.Preconditions;
 import com.google.common.collect.ImmutableList;
-import com.google.copybara.NonReversibleValidationException;
+import com.google.copybara.exception.NonReversibleValidationException;
 import com.google.copybara.TransformWork;
 import com.google.copybara.Transformation;
 import com.google.copybara.WorkflowOptions;

java/com/google/copybara/transform/FilterReplace.java

@@ -22,7 +22,7 @@
 import com.google.common.collect.Iterables;
 import com.google.common.flogger.FluentLogger;
 import com.google.copybara.LocalParallelizer.TransformFunc;
-import com.google.copybara.NonReversibleValidationException;
+import com.google.copybara.exception.NonReversibleValidationException;
 import com.google.copybara.TransformWork;
 import com.google.copybara.Transformation;
 import com.google.copybara.WorkflowOptions;

java/com/google/copybara/transform/Remove.java

@@ -20,7 +20,7 @@
 
 import com.google.common.base.Preconditions;
 import com.google.common.flogger.FluentLogger;
-import com.google.copybara.NonReversibleValidationException;
+import com.google.copybara.exception.NonReversibleValidationException;
 import com.google.copybara.TransformWork;
 import com.google.copybara.Transformation;
 import com.google.copybara.WorkflowOptions;

java/com/google/copybara/transform/Replace.java

@@ -25,7 +25,7 @@
 import com.google.common.collect.Iterables;
 import com.google.common.flogger.FluentLogger;
 import com.google.copybara.LocalParallelizer;
-import com.google.copybara.NonReversibleValidationException;
+import com.google.copybara.exception.NonReversibleValidationException;
 import com.google.copybara.TransformWork;
 import com.google.copybara.Transformation;
 import com.google.copybara.WorkflowOptions;

java/com/google/copybara/transform/ReplaceMapper.java

@@ -21,7 +21,7 @@
 import com.google.common.cache.CacheLoader;
 import com.google.common.cache.LoadingCache;
 import com.google.common.collect.ImmutableList;
-import com.google.copybara.NonReversibleValidationException;
+import com.google.copybara.exception.NonReversibleValidationException;
 import com.google.copybara.transform.RegexTemplateTokens.Replacer;
 import java.util.concurrent.ExecutionException;
 

java/com/google/copybara/transform/ReversibleFunction.java

@@ -16,7 +16,7 @@
 
 package com.google.copybara.transform;
 
-import com.google.copybara.NonReversibleValidationException;
+import com.google.copybara.exception.NonReversibleValidationException;
 import java.util.function.Function;
 import net.starlark.java.annot.StarlarkBuiltin;
 import net.starlark.java.eval.StarlarkValue;

java/com/google/copybara/transform/Sequence.java

@@ -19,7 +19,7 @@
 import com.google.common.annotations.VisibleForTesting;
 import com.google.common.base.Preconditions;
 import com.google.common.collect.ImmutableList;
-import com.google.copybara.NonReversibleValidationException;
+import com.google.copybara.exception.NonReversibleValidationException;
 import com.google.copybara.TransformWork;
 import com.google.copybara.Transformation;
 import com.google.copybara.exception.RepoException;

java/com/google/copybara/transform/SkylarkTransformation.java

@@ -21,7 +21,7 @@
 import com.google.common.base.Preconditions;
 import com.google.common.collect.ImmutableList;
 import com.google.common.collect.ImmutableMap;
-import com.google.copybara.NonReversibleValidationException;
+import com.google.copybara.exception.NonReversibleValidationException;
 import com.google.copybara.TransformWork;
 import com.google.copybara.Transformation;
 import com.google.copybara.exception.EmptyChangeException;

java/com/google/copybara/transform/TodoReplace.java

@@ -29,7 +29,7 @@
 import com.google.common.collect.ImmutableMap;
 import com.google.common.collect.Iterables;
 import com.google.copybara.LocalParallelizer;
-import com.google.copybara.NonReversibleValidationException;
+import com.google.copybara.exception.NonReversibleValidationException;
 import com.google.copybara.TransformWork;
 import com.google.copybara.Transformation;
 import com.google.copybara.exception.ValidationException;

java/com/google/copybara/transform/debug/TransformDebug.java

@@ -27,7 +27,7 @@
 import com.google.common.collect.Maps;
 import com.google.common.hash.Hashing;
 import com.google.common.io.BaseEncoding;
-import com.google.copybara.NonReversibleValidationException;
+import com.google.copybara.exception.NonReversibleValidationException;
 import com.google.copybara.TransformWork;
 import com.google.copybara.Transformation;
 import com.google.copybara.exception.RepoException;

java/com/google/copybara/transform/metadata/MapAuthor.java

@@ -23,7 +23,7 @@
 import com.google.common.collect.ImmutableBiMap;
 import com.google.common.collect.ImmutableMap;
 import com.google.copybara.Change;
-import com.google.copybara.NonReversibleValidationException;
+import com.google.copybara.exception.NonReversibleValidationException;
 import com.google.copybara.TransformWork;
 import com.google.copybara.Transformation;
 import com.google.copybara.authoring.Author;

java/com/google/copybara/transform/metadata/MetadataSquashNotes.java

@@ -18,7 +18,7 @@
 
 import com.google.common.collect.Lists;
 import com.google.copybara.Change;
-import com.google.copybara.NonReversibleValidationException;
+import com.google.copybara.exception.NonReversibleValidationException;
 import com.google.copybara.TransformWork;
 import com.google.copybara.Transformation;
 import com.google.copybara.exception.ValidationException;

java/com/google/copybara/transform/metadata/MetadataVerifyMatch.java

@@ -19,7 +19,7 @@
 import static com.google.copybara.exception.ValidationException.checkCondition;
 
 import com.google.common.base.Preconditions;
-import com.google.copybara.NonReversibleValidationException;
+import com.google.copybara.exception.NonReversibleValidationException;
 import com.google.copybara.TransformWork;
 import com.google.copybara.Transformation;
 import com.google.copybara.exception.ValidationException;

java/com/google/copybara/transform/metadata/RestoreOriginalAuthor.java

@@ -20,7 +20,7 @@
 import com.google.common.collect.ImmutableCollection;
 import com.google.common.collect.Iterables;
 import com.google.copybara.Change;
-import com.google.copybara.NonReversibleValidationException;
+import com.google.copybara.exception.NonReversibleValidationException;
 import com.google.copybara.TransformWork;
 import com.google.copybara.Transformation;
 import com.google.copybara.authoring.Author;

java/com/google/copybara/transform/metadata/SaveOriginalAuthor.java

@@ -17,7 +17,7 @@
 package com.google.copybara.transform.metadata;
 
 import com.google.common.base.Preconditions;
-import com.google.copybara.NonReversibleValidationException;
+import com.google.copybara.exception.NonReversibleValidationException;
 import com.google.copybara.TransformWork;
 import com.google.copybara.Transformation;
 import com.google.copybara.exception.ValidationException;

java/com/google/copybara/transform/metadata/Scrubber.java

@@ -17,7 +17,7 @@
 package com.google.copybara.transform.metadata;
 
 import com.google.common.base.Preconditions;
-import com.google.copybara.NonReversibleValidationException;
+import com.google.copybara.exception.NonReversibleValidationException;
 import com.google.copybara.TransformWork;
 import com.google.copybara.Transformation;
 import com.google.copybara.exception.ValidationException;

java/com/google/copybara/transform/metadata/UseLastChange.java

@@ -18,7 +18,7 @@
 
 import com.google.common.base.Preconditions;
 import com.google.copybara.Change;
-import com.google.copybara.NonReversibleValidationException;
+import com.google.copybara.exception.NonReversibleValidationException;
 import com.google.copybara.TransformWork;
 import com.google.copybara.Transformation;
 import com.google.copybara.exception.ValidationException;

javatests/com/google/copybara/CoreReverseTest.java

@@ -20,6 +20,7 @@
 import static org.junit.Assert.assertThrows;
 
 import com.google.common.collect.ImmutableSet;
+import com.google.copybara.exception.NonReversibleValidationException;
 import com.google.copybara.exception.ValidationException;
 import com.google.copybara.testing.OptionsBuilder;
 import com.google.copybara.testing.SkylarkTestExecutor;

javatests/com/google/copybara/transform/CopyOrMoveTest.java

@@ -23,7 +23,7 @@
 
 import com.google.common.collect.ImmutableList;
 import com.google.common.jimfs.Jimfs;
-import com.google.copybara.NonReversibleValidationException;
+import com.google.copybara.exception.NonReversibleValidationException;
 import com.google.copybara.Transformation;
 import com.google.copybara.exception.ValidationException;
 import com.google.copybara.testing.OptionsBuilder;

javatests/com/google/copybara/transform/ExplicitReversalTest.java

@@ -18,7 +18,7 @@
 
 import static com.google.common.truth.Truth.assertThat;
 import static java.nio.file.Paths.get;
-import com.google.copybara.NonReversibleValidationException;
+import com.google.copybara.exception.NonReversibleValidationException;
 import com.google.copybara.TransformWork;
 import com.google.copybara.Transformation;
 import com.google.copybara.exception.ValidationException;

javatests/com/google/copybara/transform/TodoReplaceTest.java

@@ -23,7 +23,7 @@
 
 import com.google.common.base.Joiner;
 import com.google.common.jimfs.Jimfs;
-import com.google.copybara.NonReversibleValidationException;
+import com.google.copybara.exception.NonReversibleValidationException;
 import com.google.copybara.TransformWork;
 import com.google.copybara.Transformation;
 import com.google.copybara.exception.ValidationException;

javatests/com/google/copybara/transform/metadata/MetadataModuleTest.java

@@ -29,7 +29,7 @@
 import com.google.common.collect.Iterables;
 import com.google.copybara.Change;
 import com.google.copybara.Changes;
-import com.google.copybara.NonReversibleValidationException;
+import com.google.copybara.exception.NonReversibleValidationException;
 import com.google.copybara.TransformWork;
 import com.google.copybara.Transformation;
 import com.google.copybara.Workflow;