Move JGitPath into a seperate package This is a step towards splitting the monolithic plugin target across java packages. Also-by: David Ostrovsky <david@ostrovsky.org> Signed-off-by: Edwin Kempin <ekempin@google.com> Change-Id: I263f395fad4109627f9e3e20703e0c8b92fb4852
diff --git a/java/com/google/gerrit/plugins/codeowners/acceptance/AbstractCodeOwnersTest.java b/java/com/google/gerrit/plugins/codeowners/acceptance/AbstractCodeOwnersTest.java index 0ed3041..3580969 100644 --- a/java/com/google/gerrit/plugins/codeowners/acceptance/AbstractCodeOwnersTest.java +++ b/java/com/google/gerrit/plugins/codeowners/acceptance/AbstractCodeOwnersTest.java
@@ -34,11 +34,11 @@ import com.google.gerrit.extensions.common.ChangeInput; import com.google.gerrit.extensions.common.LabelDefinitionInput; import com.google.gerrit.extensions.restapi.RestApiException; -import com.google.gerrit.plugins.codeowners.JgitPath; import com.google.gerrit.plugins.codeowners.acceptance.testsuite.CodeOwnerConfigOperations; import com.google.gerrit.plugins.codeowners.acceptance.testsuite.TestCodeOwnerConfigCreation.Builder; import com.google.gerrit.plugins.codeowners.config.CodeOwnersPluginConfiguration; import com.google.gerrit.plugins.codeowners.config.StatusConfig; +import com.google.gerrit.plugins.codeowners.util.JgitPath; import com.google.inject.Inject; import java.nio.file.Path; import java.util.Map;
diff --git a/java/com/google/gerrit/plugins/codeowners/acceptance/testsuite/CodeOwnerConfigOperationsImpl.java b/java/com/google/gerrit/plugins/codeowners/acceptance/testsuite/CodeOwnerConfigOperationsImpl.java index 8462d52..317098d 100644 --- a/java/com/google/gerrit/plugins/codeowners/acceptance/testsuite/CodeOwnerConfigOperationsImpl.java +++ b/java/com/google/gerrit/plugins/codeowners/acceptance/testsuite/CodeOwnerConfigOperationsImpl.java
@@ -16,7 +16,6 @@ import static com.google.common.base.Preconditions.checkState; -import com.google.gerrit.plugins.codeowners.JgitPath; import com.google.gerrit.plugins.codeowners.backend.CodeOwnerConfig; import com.google.gerrit.plugins.codeowners.backend.CodeOwnerConfigImportModification; import com.google.gerrit.plugins.codeowners.backend.CodeOwnerConfigUpdate; @@ -24,6 +23,7 @@ import com.google.gerrit.plugins.codeowners.backend.CodeOwners; import com.google.gerrit.plugins.codeowners.backend.CodeOwnersUpdate; import com.google.gerrit.plugins.codeowners.config.BackendConfig; +import com.google.gerrit.plugins.codeowners.util.JgitPath; import com.google.gerrit.server.ServerInitiated; import com.google.gerrit.server.git.GitRepositoryManager; import com.google.gerrit.server.project.ProjectCache;
diff --git a/java/com/google/gerrit/plugins/codeowners/backend/ChangedFile.java b/java/com/google/gerrit/plugins/codeowners/backend/ChangedFile.java index 82655cf..5838d1f 100644 --- a/java/com/google/gerrit/plugins/codeowners/backend/ChangedFile.java +++ b/java/com/google/gerrit/plugins/codeowners/backend/ChangedFile.java
@@ -22,7 +22,7 @@ import com.google.common.collect.Maps; import com.google.gerrit.common.Nullable; import com.google.gerrit.entities.Patch; -import com.google.gerrit.plugins.codeowners.JgitPath; +import com.google.gerrit.plugins.codeowners.util.JgitPath; import com.google.gerrit.server.patch.PatchListEntry; import java.nio.file.Path; import java.util.Optional;
diff --git a/java/com/google/gerrit/plugins/codeowners/backend/CodeOwnerConfigFile.java b/java/com/google/gerrit/plugins/codeowners/backend/CodeOwnerConfigFile.java index 5ecddb3..4cceb13 100644 --- a/java/com/google/gerrit/plugins/codeowners/backend/CodeOwnerConfigFile.java +++ b/java/com/google/gerrit/plugins/codeowners/backend/CodeOwnerConfigFile.java
@@ -19,7 +19,7 @@ import com.google.common.annotations.VisibleForTesting; import com.google.common.base.Strings; -import com.google.gerrit.plugins.codeowners.JgitPath; +import com.google.gerrit.plugins.codeowners.util.JgitPath; import com.google.gerrit.server.git.meta.MetaDataUpdate; import com.google.gerrit.server.git.meta.VersionedMetaData; import java.io.IOException;
diff --git a/java/com/google/gerrit/plugins/codeowners/backend/CodeOwnerConfigTreeWalk.java b/java/com/google/gerrit/plugins/codeowners/backend/CodeOwnerConfigTreeWalk.java index 1ba8f74..0836a53 100644 --- a/java/com/google/gerrit/plugins/codeowners/backend/CodeOwnerConfigTreeWalk.java +++ b/java/com/google/gerrit/plugins/codeowners/backend/CodeOwnerConfigTreeWalk.java
@@ -21,7 +21,7 @@ import com.google.gerrit.common.Nullable; import com.google.gerrit.entities.BranchNameKey; import com.google.gerrit.entities.Project; -import com.google.gerrit.plugins.codeowners.JgitPath; +import com.google.gerrit.plugins.codeowners.util.JgitPath; import java.io.IOException; import java.nio.file.FileSystems; import java.nio.file.Path;
diff --git a/java/com/google/gerrit/plugins/codeowners/backend/CodeOwnersOnPostReview.java b/java/com/google/gerrit/plugins/codeowners/backend/CodeOwnersOnPostReview.java index 403b2f8..829058a 100644 --- a/java/com/google/gerrit/plugins/codeowners/backend/CodeOwnersOnPostReview.java +++ b/java/com/google/gerrit/plugins/codeowners/backend/CodeOwnersOnPostReview.java
@@ -23,10 +23,10 @@ import com.google.gerrit.entities.Account; import com.google.gerrit.entities.PatchSet; import com.google.gerrit.extensions.restapi.ResourceConflictException; -import com.google.gerrit.plugins.codeowners.JgitPath; import com.google.gerrit.plugins.codeowners.api.CodeOwnerStatus; import com.google.gerrit.plugins.codeowners.config.CodeOwnersPluginConfiguration; import com.google.gerrit.plugins.codeowners.config.RequiredApproval; +import com.google.gerrit.plugins.codeowners.util.JgitPath; import com.google.gerrit.server.IdentifiedUser; import com.google.gerrit.server.notedb.ChangeNotes; import com.google.gerrit.server.patch.PatchListNotAvailableException;
diff --git a/java/com/google/gerrit/plugins/codeowners/config/CodeOwnersPluginConfigValidator.java b/java/com/google/gerrit/plugins/codeowners/config/CodeOwnersPluginConfigValidator.java index 6a76360..bddfdc9 100644 --- a/java/com/google/gerrit/plugins/codeowners/config/CodeOwnersPluginConfigValidator.java +++ b/java/com/google/gerrit/plugins/codeowners/config/CodeOwnersPluginConfigValidator.java
@@ -19,8 +19,8 @@ import com.google.gerrit.entities.Project; import com.google.gerrit.entities.RefNames; import com.google.gerrit.extensions.annotations.PluginName; -import com.google.gerrit.plugins.codeowners.JgitPath; import com.google.gerrit.plugins.codeowners.backend.ChangedFiles; +import com.google.gerrit.plugins.codeowners.util.JgitPath; import com.google.gerrit.server.events.CommitReceivedEvent; import com.google.gerrit.server.git.GitRepositoryManager; import com.google.gerrit.server.git.validators.CommitValidationException;
diff --git a/java/com/google/gerrit/plugins/codeowners/restapi/CheckCodeOwner.java b/java/com/google/gerrit/plugins/codeowners/restapi/CheckCodeOwner.java index 86adb2b..a125d34 100644 --- a/java/com/google/gerrit/plugins/codeowners/restapi/CheckCodeOwner.java +++ b/java/com/google/gerrit/plugins/codeowners/restapi/CheckCodeOwner.java
@@ -25,7 +25,6 @@ import com.google.gerrit.extensions.restapi.Response; import com.google.gerrit.extensions.restapi.RestReadView; import com.google.gerrit.extensions.restapi.TopLevelResource; -import com.google.gerrit.plugins.codeowners.JgitPath; import com.google.gerrit.plugins.codeowners.api.CodeOwnerCheckInfo; import com.google.gerrit.plugins.codeowners.backend.CodeOwner; import com.google.gerrit.plugins.codeowners.backend.CodeOwnerConfigHierarchy; @@ -36,6 +35,7 @@ import com.google.gerrit.plugins.codeowners.backend.PathCodeOwners; import com.google.gerrit.plugins.codeowners.backend.PathCodeOwnersResult; import com.google.gerrit.plugins.codeowners.config.CodeOwnersPluginConfiguration; +import com.google.gerrit.plugins.codeowners.util.JgitPath; import com.google.gerrit.server.IdentifiedUser; import com.google.gerrit.server.permissions.PermissionBackend; import com.google.gerrit.server.permissions.PermissionBackendException;
diff --git a/java/com/google/gerrit/plugins/codeowners/restapi/CodeOwnerStatusInfoJson.java b/java/com/google/gerrit/plugins/codeowners/restapi/CodeOwnerStatusInfoJson.java index 284990b..bba3dbe 100644 --- a/java/com/google/gerrit/plugins/codeowners/restapi/CodeOwnerStatusInfoJson.java +++ b/java/com/google/gerrit/plugins/codeowners/restapi/CodeOwnerStatusInfoJson.java
@@ -24,12 +24,12 @@ import com.google.common.collect.Maps; import com.google.gerrit.entities.PatchSet; import com.google.gerrit.extensions.common.ChangeType; -import com.google.gerrit.plugins.codeowners.JgitPath; import com.google.gerrit.plugins.codeowners.api.CodeOwnerStatusInfo; import com.google.gerrit.plugins.codeowners.api.FileCodeOwnerStatusInfo; import com.google.gerrit.plugins.codeowners.api.PathCodeOwnerStatusInfo; import com.google.gerrit.plugins.codeowners.backend.FileCodeOwnerStatus; import com.google.gerrit.plugins.codeowners.backend.PathCodeOwnerStatus; +import com.google.gerrit.plugins.codeowners.util.JgitPath; import java.util.Comparator; import org.eclipse.jgit.diff.DiffEntry;
diff --git a/java/com/google/gerrit/plugins/codeowners/testing/backend/TestCodeOwnerConfigStorage.java b/java/com/google/gerrit/plugins/codeowners/testing/backend/TestCodeOwnerConfigStorage.java index 6a9c483..772277b 100644 --- a/java/com/google/gerrit/plugins/codeowners/testing/backend/TestCodeOwnerConfigStorage.java +++ b/java/com/google/gerrit/plugins/codeowners/testing/backend/TestCodeOwnerConfigStorage.java
@@ -14,9 +14,9 @@ package com.google.gerrit.plugins.codeowners.testing.backend; -import com.google.gerrit.plugins.codeowners.JgitPath; import com.google.gerrit.plugins.codeowners.backend.CodeOwnerConfig; import com.google.gerrit.plugins.codeowners.backend.CodeOwnerConfigParser; +import com.google.gerrit.plugins.codeowners.util.JgitPath; import com.google.gerrit.server.git.GitRepositoryManager; import com.google.inject.Inject; import com.google.inject.assistedinject.Assisted;
diff --git a/java/com/google/gerrit/plugins/codeowners/JgitPath.java b/java/com/google/gerrit/plugins/codeowners/util/JgitPath.java similarity index 97% rename from java/com/google/gerrit/plugins/codeowners/JgitPath.java rename to java/com/google/gerrit/plugins/codeowners/util/JgitPath.java index fd6c5fd..18b77be 100644 --- a/java/com/google/gerrit/plugins/codeowners/JgitPath.java +++ b/java/com/google/gerrit/plugins/codeowners/util/JgitPath.java
@@ -12,7 +12,7 @@ // See the License for the specific language governing permissions and // limitations under the License. -package com.google.gerrit.plugins.codeowners; +package com.google.gerrit.plugins.codeowners.util; import static java.util.Objects.requireNonNull;
diff --git a/java/com/google/gerrit/plugins/codeowners/validation/CodeOwnerConfigValidator.java b/java/com/google/gerrit/plugins/codeowners/validation/CodeOwnerConfigValidator.java index b2dd664..17c1255 100644 --- a/java/com/google/gerrit/plugins/codeowners/validation/CodeOwnerConfigValidator.java +++ b/java/com/google/gerrit/plugins/codeowners/validation/CodeOwnerConfigValidator.java
@@ -29,7 +29,6 @@ import com.google.gerrit.entities.PatchSet; import com.google.gerrit.entities.Project; import com.google.gerrit.exceptions.StorageException; -import com.google.gerrit.plugins.codeowners.JgitPath; import com.google.gerrit.plugins.codeowners.api.CodeOwnerConfigValidationPolicy; import com.google.gerrit.plugins.codeowners.api.MergeCommitStrategy; import com.google.gerrit.plugins.codeowners.backend.ChangedFile; @@ -43,6 +42,7 @@ import com.google.gerrit.plugins.codeowners.backend.PathCodeOwners; import com.google.gerrit.plugins.codeowners.config.CodeOwnersPluginConfiguration; import com.google.gerrit.plugins.codeowners.config.InvalidPluginConfigurationException; +import com.google.gerrit.plugins.codeowners.util.JgitPath; import com.google.gerrit.server.IdentifiedUser; import com.google.gerrit.server.PatchSetUtil; import com.google.gerrit.server.events.CommitReceivedEvent;
diff --git a/javatests/com/google/gerrit/plugins/codeowners/acceptance/api/CheckCodeOwnerConfigFilesInRevisionIT.java b/javatests/com/google/gerrit/plugins/codeowners/acceptance/api/CheckCodeOwnerConfigFilesInRevisionIT.java index 98a071c..b67f0f0 100644 --- a/javatests/com/google/gerrit/plugins/codeowners/acceptance/api/CheckCodeOwnerConfigFilesInRevisionIT.java +++ b/javatests/com/google/gerrit/plugins/codeowners/acceptance/api/CheckCodeOwnerConfigFilesInRevisionIT.java
@@ -23,7 +23,6 @@ import com.google.gerrit.common.Nullable; import com.google.gerrit.extensions.api.config.ConsistencyCheckInfo.ConsistencyProblemInfo; import com.google.gerrit.extensions.restapi.RestApiException; -import com.google.gerrit.plugins.codeowners.JgitPath; import com.google.gerrit.plugins.codeowners.acceptance.AbstractCodeOwnersIT; import com.google.gerrit.plugins.codeowners.backend.CodeOwnerBackend; import com.google.gerrit.plugins.codeowners.backend.CodeOwnerConfig; @@ -33,6 +32,7 @@ import com.google.gerrit.plugins.codeowners.backend.proto.ProtoBackend; import com.google.gerrit.plugins.codeowners.backend.proto.ProtoCodeOwnerConfigParser; import com.google.gerrit.plugins.codeowners.config.BackendConfig; +import com.google.gerrit.plugins.codeowners.util.JgitPath; import java.util.HashMap; import java.util.List; import java.util.Map;
diff --git a/javatests/com/google/gerrit/plugins/codeowners/acceptance/api/CheckCodeOwnerIT.java b/javatests/com/google/gerrit/plugins/codeowners/acceptance/api/CheckCodeOwnerIT.java index fc5fddf..5cd6e88 100644 --- a/javatests/com/google/gerrit/plugins/codeowners/acceptance/api/CheckCodeOwnerIT.java +++ b/javatests/com/google/gerrit/plugins/codeowners/acceptance/api/CheckCodeOwnerIT.java
@@ -31,7 +31,6 @@ import com.google.gerrit.extensions.restapi.AuthException; import com.google.gerrit.extensions.restapi.BadRequestException; import com.google.gerrit.extensions.restapi.RestApiException; -import com.google.gerrit.plugins.codeowners.JgitPath; import com.google.gerrit.plugins.codeowners.acceptance.AbstractCodeOwnersIT; import com.google.gerrit.plugins.codeowners.api.CodeOwnerCheckInfo; import com.google.gerrit.plugins.codeowners.backend.CodeOwnerBackend; @@ -43,6 +42,7 @@ import com.google.gerrit.plugins.codeowners.backend.proto.ProtoBackend; import com.google.gerrit.plugins.codeowners.config.BackendConfig; import com.google.gerrit.plugins.codeowners.restapi.CheckCodeOwnerCapability; +import com.google.gerrit.plugins.codeowners.util.JgitPath; import com.google.gerrit.server.ServerInitiated; import com.google.gerrit.server.account.AccountsUpdate; import com.google.gerrit.server.account.externalids.ExternalId;
diff --git a/javatests/com/google/gerrit/plugins/codeowners/acceptance/api/GetCodeOwnerConfigFilesIT.java b/javatests/com/google/gerrit/plugins/codeowners/acceptance/api/GetCodeOwnerConfigFilesIT.java index c17e4e2..b9df0b3 100644 --- a/javatests/com/google/gerrit/plugins/codeowners/acceptance/api/GetCodeOwnerConfigFilesIT.java +++ b/javatests/com/google/gerrit/plugins/codeowners/acceptance/api/GetCodeOwnerConfigFilesIT.java
@@ -20,7 +20,6 @@ import com.google.gerrit.acceptance.TestAccount; import com.google.gerrit.entities.RefNames; import com.google.gerrit.extensions.restapi.BadRequestException; -import com.google.gerrit.plugins.codeowners.JgitPath; import com.google.gerrit.plugins.codeowners.acceptance.AbstractCodeOwnersIT; import com.google.gerrit.plugins.codeowners.backend.CodeOwnerBackend; import com.google.gerrit.plugins.codeowners.backend.CodeOwnerConfig; @@ -28,6 +27,7 @@ import com.google.gerrit.plugins.codeowners.backend.findowners.FindOwnersBackend; import com.google.gerrit.plugins.codeowners.backend.proto.ProtoBackend; import com.google.gerrit.plugins.codeowners.config.BackendConfig; +import com.google.gerrit.plugins.codeowners.util.JgitPath; import org.junit.Before; import org.junit.Test;
diff --git a/javatests/com/google/gerrit/plugins/codeowners/acceptance/api/GetCodeOwnerConfigForPathInBranchIT.java b/javatests/com/google/gerrit/plugins/codeowners/acceptance/api/GetCodeOwnerConfigForPathInBranchIT.java index 8ccb3ba..3048707 100644 --- a/javatests/com/google/gerrit/plugins/codeowners/acceptance/api/GetCodeOwnerConfigForPathInBranchIT.java +++ b/javatests/com/google/gerrit/plugins/codeowners/acceptance/api/GetCodeOwnerConfigForPathInBranchIT.java
@@ -20,10 +20,10 @@ import com.google.gerrit.acceptance.config.GerritConfig; import com.google.gerrit.extensions.restapi.MethodNotAllowedException; -import com.google.gerrit.plugins.codeowners.JgitPath; import com.google.gerrit.plugins.codeowners.acceptance.AbstractCodeOwnersIT; import com.google.gerrit.plugins.codeowners.api.CodeOwnerConfigInfo; import com.google.gerrit.plugins.codeowners.backend.CodeOwnerConfig; +import com.google.gerrit.plugins.codeowners.util.JgitPath; import java.util.Optional; import org.junit.Test;
diff --git a/javatests/com/google/gerrit/plugins/codeowners/acceptance/api/GetCodeOwnersForPathInChangeIT.java b/javatests/com/google/gerrit/plugins/codeowners/acceptance/api/GetCodeOwnersForPathInChangeIT.java index d5a840c..27587e2 100644 --- a/javatests/com/google/gerrit/plugins/codeowners/acceptance/api/GetCodeOwnersForPathInChangeIT.java +++ b/javatests/com/google/gerrit/plugins/codeowners/acceptance/api/GetCodeOwnersForPathInChangeIT.java
@@ -30,9 +30,9 @@ import com.google.gerrit.extensions.restapi.BadRequestException; import com.google.gerrit.extensions.restapi.ResourceNotFoundException; import com.google.gerrit.extensions.restapi.RestApiException; -import com.google.gerrit.plugins.codeowners.JgitPath; import com.google.gerrit.plugins.codeowners.api.CodeOwnerInfo; import com.google.gerrit.plugins.codeowners.api.CodeOwners; +import com.google.gerrit.plugins.codeowners.util.JgitPath; import com.google.inject.Inject; import java.util.List; import org.eclipse.jgit.internal.storage.dfs.InMemoryRepository;
diff --git a/javatests/com/google/gerrit/plugins/codeowners/acceptance/restapi/GetCodeOwnerStatusRestIT.java b/javatests/com/google/gerrit/plugins/codeowners/acceptance/restapi/GetCodeOwnerStatusRestIT.java index bcbb0cf..7894bf2 100644 --- a/javatests/com/google/gerrit/plugins/codeowners/acceptance/restapi/GetCodeOwnerStatusRestIT.java +++ b/javatests/com/google/gerrit/plugins/codeowners/acceptance/restapi/GetCodeOwnerStatusRestIT.java
@@ -19,11 +19,11 @@ import com.google.gerrit.acceptance.RestResponse; import com.google.gerrit.acceptance.config.GerritConfig; import com.google.gerrit.extensions.restapi.IdString; -import com.google.gerrit.plugins.codeowners.JgitPath; import com.google.gerrit.plugins.codeowners.acceptance.AbstractCodeOwnersIT; import com.google.gerrit.plugins.codeowners.acceptance.AbstractCodeOwnersTest; import com.google.gerrit.plugins.codeowners.acceptance.testsuite.CodeOwnerConfigOperations; import com.google.gerrit.plugins.codeowners.backend.CodeOwnerConfig; +import com.google.gerrit.plugins.codeowners.util.JgitPath; import org.junit.Before; import org.junit.Test;
diff --git a/javatests/com/google/gerrit/plugins/codeowners/acceptance/restapi/GetCodeOwnersForPathInChangeRestIT.java b/javatests/com/google/gerrit/plugins/codeowners/acceptance/restapi/GetCodeOwnersForPathInChangeRestIT.java index 3c88f2d..92a7e25 100644 --- a/javatests/com/google/gerrit/plugins/codeowners/acceptance/restapi/GetCodeOwnersForPathInChangeRestIT.java +++ b/javatests/com/google/gerrit/plugins/codeowners/acceptance/restapi/GetCodeOwnersForPathInChangeRestIT.java
@@ -19,7 +19,7 @@ import com.google.gerrit.acceptance.RestResponse; import com.google.gerrit.acceptance.TestAccount; import com.google.gerrit.extensions.restapi.IdString; -import com.google.gerrit.plugins.codeowners.JgitPath; +import com.google.gerrit.plugins.codeowners.util.JgitPath; import org.junit.Before; import org.junit.Test;
diff --git a/javatests/com/google/gerrit/plugins/codeowners/backend/ChangedFilesTest.java b/javatests/com/google/gerrit/plugins/codeowners/backend/ChangedFilesTest.java index 19d5fdd..db73a2f 100644 --- a/javatests/com/google/gerrit/plugins/codeowners/backend/ChangedFilesTest.java +++ b/javatests/com/google/gerrit/plugins/codeowners/backend/ChangedFilesTest.java
@@ -32,9 +32,9 @@ import com.google.gerrit.extensions.api.projects.BranchInput; import com.google.gerrit.extensions.restapi.IdString; import com.google.gerrit.extensions.restapi.TopLevelResource; -import com.google.gerrit.plugins.codeowners.JgitPath; import com.google.gerrit.plugins.codeowners.acceptance.AbstractCodeOwnersTest; import com.google.gerrit.plugins.codeowners.api.MergeCommitStrategy; +import com.google.gerrit.plugins.codeowners.util.JgitPath; import com.google.gerrit.server.change.ChangeResource; import com.google.gerrit.server.change.RevisionResource; import com.google.gerrit.server.restapi.change.ChangesCollection;
diff --git a/javatests/com/google/gerrit/plugins/codeowners/backend/CodeOwnerApprovalCheckForAccountTest.java b/javatests/com/google/gerrit/plugins/codeowners/backend/CodeOwnerApprovalCheckForAccountTest.java index e392e47..216504f 100644 --- a/javatests/com/google/gerrit/plugins/codeowners/backend/CodeOwnerApprovalCheckForAccountTest.java +++ b/javatests/com/google/gerrit/plugins/codeowners/backend/CodeOwnerApprovalCheckForAccountTest.java
@@ -21,11 +21,11 @@ import com.google.gerrit.acceptance.testsuite.request.RequestScopeOperations; import com.google.gerrit.entities.Account; import com.google.gerrit.entities.Change; -import com.google.gerrit.plugins.codeowners.JgitPath; import com.google.gerrit.plugins.codeowners.acceptance.AbstractCodeOwnersTest; import com.google.gerrit.plugins.codeowners.acceptance.testsuite.CodeOwnerConfigOperations; import com.google.gerrit.plugins.codeowners.api.CodeOwnerStatus; import com.google.gerrit.plugins.codeowners.testing.FileCodeOwnerStatusSubject; +import com.google.gerrit.plugins.codeowners.util.JgitPath; import com.google.gerrit.server.notedb.ChangeNotes; import com.google.inject.Inject; import java.nio.file.Path;
diff --git a/javatests/com/google/gerrit/plugins/codeowners/backend/CodeOwnerApprovalCheckTest.java b/javatests/com/google/gerrit/plugins/codeowners/backend/CodeOwnerApprovalCheckTest.java index c300583..f8edcd1 100644 --- a/javatests/com/google/gerrit/plugins/codeowners/backend/CodeOwnerApprovalCheckTest.java +++ b/javatests/com/google/gerrit/plugins/codeowners/backend/CodeOwnerApprovalCheckTest.java
@@ -35,11 +35,11 @@ import com.google.gerrit.extensions.api.projects.DeleteBranchesInput; import com.google.gerrit.extensions.common.LabelDefinitionInput; import com.google.gerrit.extensions.restapi.ResourceConflictException; -import com.google.gerrit.plugins.codeowners.JgitPath; import com.google.gerrit.plugins.codeowners.acceptance.AbstractCodeOwnersTest; import com.google.gerrit.plugins.codeowners.acceptance.testsuite.CodeOwnerConfigOperations; import com.google.gerrit.plugins.codeowners.api.CodeOwnerStatus; import com.google.gerrit.plugins.codeowners.testing.FileCodeOwnerStatusSubject; +import com.google.gerrit.plugins.codeowners.util.JgitPath; import com.google.gerrit.server.notedb.ChangeNotes; import com.google.inject.Inject; import java.nio.file.Path;
diff --git a/javatests/com/google/gerrit/plugins/codeowners/backend/CodeOwnerApprovalCheckWithAllUsersAsFallbackCodeOwnersTest.java b/javatests/com/google/gerrit/plugins/codeowners/backend/CodeOwnerApprovalCheckWithAllUsersAsFallbackCodeOwnersTest.java index d2d2c5b..dd40600 100644 --- a/javatests/com/google/gerrit/plugins/codeowners/backend/CodeOwnerApprovalCheckWithAllUsersAsFallbackCodeOwnersTest.java +++ b/javatests/com/google/gerrit/plugins/codeowners/backend/CodeOwnerApprovalCheckWithAllUsersAsFallbackCodeOwnersTest.java
@@ -20,12 +20,12 @@ import com.google.gerrit.acceptance.config.GerritConfig; import com.google.gerrit.acceptance.testsuite.request.RequestScopeOperations; import com.google.gerrit.entities.Change; -import com.google.gerrit.plugins.codeowners.JgitPath; import com.google.gerrit.plugins.codeowners.acceptance.AbstractCodeOwnersTest; import com.google.gerrit.plugins.codeowners.acceptance.testsuite.CodeOwnerConfigOperations; import com.google.gerrit.plugins.codeowners.api.CodeOwnerStatus; import com.google.gerrit.plugins.codeowners.config.GeneralConfig; import com.google.gerrit.plugins.codeowners.testing.FileCodeOwnerStatusSubject; +import com.google.gerrit.plugins.codeowners.util.JgitPath; import com.google.gerrit.server.notedb.ChangeNotes; import com.google.gerrit.testing.ConfigSuite; import com.google.inject.Inject;
diff --git a/javatests/com/google/gerrit/plugins/codeowners/backend/CodeOwnerApprovalCheckWithSelfApprovalsIgnoredTest.java b/javatests/com/google/gerrit/plugins/codeowners/backend/CodeOwnerApprovalCheckWithSelfApprovalsIgnoredTest.java index 9ba465a..253cfbc 100644 --- a/javatests/com/google/gerrit/plugins/codeowners/backend/CodeOwnerApprovalCheckWithSelfApprovalsIgnoredTest.java +++ b/javatests/com/google/gerrit/plugins/codeowners/backend/CodeOwnerApprovalCheckWithSelfApprovalsIgnoredTest.java
@@ -22,11 +22,11 @@ import com.google.gerrit.entities.Change; import com.google.gerrit.extensions.api.changes.ReviewInput; import com.google.gerrit.extensions.common.LabelDefinitionInput; -import com.google.gerrit.plugins.codeowners.JgitPath; import com.google.gerrit.plugins.codeowners.acceptance.AbstractCodeOwnersTest; import com.google.gerrit.plugins.codeowners.api.CodeOwnerStatus; import com.google.gerrit.plugins.codeowners.config.OverrideApprovalConfig; import com.google.gerrit.plugins.codeowners.testing.FileCodeOwnerStatusSubject; +import com.google.gerrit.plugins.codeowners.util.JgitPath; import com.google.gerrit.server.notedb.ChangeNotes; import com.google.gerrit.testing.ConfigSuite; import com.google.inject.Inject;
diff --git a/javatests/com/google/gerrit/plugins/codeowners/backend/CodeOwnerConfigFileUpdateScannerTest.java b/javatests/com/google/gerrit/plugins/codeowners/backend/CodeOwnerConfigFileUpdateScannerTest.java index 839c77b..26cb4e8 100644 --- a/javatests/com/google/gerrit/plugins/codeowners/backend/CodeOwnerConfigFileUpdateScannerTest.java +++ b/javatests/com/google/gerrit/plugins/codeowners/backend/CodeOwnerConfigFileUpdateScannerTest.java
@@ -25,9 +25,9 @@ import com.google.gerrit.acceptance.TestAccount; import com.google.gerrit.acceptance.testsuite.project.ProjectOperations; import com.google.gerrit.entities.BranchNameKey; -import com.google.gerrit.plugins.codeowners.JgitPath; import com.google.gerrit.plugins.codeowners.acceptance.AbstractCodeOwnersTest; import com.google.gerrit.plugins.codeowners.acceptance.testsuite.CodeOwnerConfigOperations; +import com.google.gerrit.plugins.codeowners.util.JgitPath; import com.google.inject.Inject; import java.nio.file.Path; import java.nio.file.Paths;
diff --git a/javatests/com/google/gerrit/plugins/codeowners/BUILD b/javatests/com/google/gerrit/plugins/codeowners/util/BUILD similarity index 91% rename from javatests/com/google/gerrit/plugins/codeowners/BUILD rename to javatests/com/google/gerrit/plugins/codeowners/util/BUILD index 0a2f23e..40ae290 100644 --- a/javatests/com/google/gerrit/plugins/codeowners/BUILD +++ b/javatests/com/google/gerrit/plugins/codeowners/util/BUILD
@@ -2,7 +2,7 @@ acceptance_tests( srcs = glob(["*Test.java"]), - group = "codeowners", + group = "util", deps = [ "//plugins/code-owners:code-owners__plugin", "//plugins/code-owners/java/com/google/gerrit/plugins/codeowners/acceptance",
diff --git a/javatests/com/google/gerrit/plugins/codeowners/JgitPathTest.java b/javatests/com/google/gerrit/plugins/codeowners/util/JgitPathTest.java similarity index 95% rename from javatests/com/google/gerrit/plugins/codeowners/JgitPathTest.java rename to javatests/com/google/gerrit/plugins/codeowners/util/JgitPathTest.java index f2c3e81..25f4035 100644 --- a/javatests/com/google/gerrit/plugins/codeowners/JgitPathTest.java +++ b/javatests/com/google/gerrit/plugins/codeowners/util/JgitPathTest.java
@@ -12,13 +12,14 @@ // See the License for the specific language governing permissions and // limitations under the License. -package com.google.gerrit.plugins.codeowners; +package com.google.gerrit.plugins.codeowners.util; import static com.google.common.truth.Truth.assertThat; import static com.google.common.truth.Truth8.assertThat; import static com.google.gerrit.testing.GerritJUnit.assertThrows; import com.google.gerrit.plugins.codeowners.acceptance.AbstractCodeOwnersTest; +import com.google.gerrit.plugins.codeowners.util.JgitPath; import java.nio.file.Path; import java.nio.file.Paths; import org.junit.Test;