From 5c173c632d184d9fc5dd7994b2eccb7c9d28511f Mon Sep 17 00:00:00 2001 From: Ivan Frade Date: Fri, 17 Nov 2023 10:11:17 -0800 Subject: [PATCH] Adjust javadoc to pass errorprone checks bazel build //org.eclipse.jgit.test:all doesn't build due to errorprone errors. This leds to disabling the checks and find the errors later in the jenkins builder. Fix javadoc errors reported by errorprone. This doesn't fix completely the build but it is a step towards it. Change-Id: Ic4ec93a2d71d4628bf3a05277c60157881156393 --- .../jgit/internal/storage/file/GcTestCase.java | 2 ++ .../storage/file/PackIndexTestCase.java | 2 ++ .../SampleDataRepositoryTestCase.java | 1 + .../transport/RequestValidatorTestCase.java | 18 +++++++++++++++++- 4 files changed, 22 insertions(+), 1 deletion(-) diff --git a/org.eclipse.jgit.test/tst/org/eclipse/jgit/internal/storage/file/GcTestCase.java b/org.eclipse.jgit.test/tst/org/eclipse/jgit/internal/storage/file/GcTestCase.java index 48f6e06385..8b27b829b2 100644 --- a/org.eclipse.jgit.test/tst/org/eclipse/jgit/internal/storage/file/GcTestCase.java +++ b/org.eclipse.jgit.test/tst/org/eclipse/jgit/internal/storage/file/GcTestCase.java @@ -61,6 +61,7 @@ public abstract class GcTestCase extends LocalDiskRepositoryTestCase { * the depth of the commit chain. * @return the commit that is the tip of the commit chain * @throws Exception + * if an error occurred */ protected RevCommit commitChain(int depth) throws Exception { if (depth <= 0) @@ -93,6 +94,7 @@ public abstract class GcTestCase extends LocalDiskRepositoryTestCase { * number of files added per commit * @return the commit that is the tip of the commit chain * @throws Exception + * if an error occurred */ protected RevCommit commitChain(int depth, int width) throws Exception { if (depth <= 0) { diff --git a/org.eclipse.jgit.test/tst/org/eclipse/jgit/internal/storage/file/PackIndexTestCase.java b/org.eclipse.jgit.test/tst/org/eclipse/jgit/internal/storage/file/PackIndexTestCase.java index 67bba18e2b..24bdc4a97a 100644 --- a/org.eclipse.jgit.test/tst/org/eclipse/jgit/internal/storage/file/PackIndexTestCase.java +++ b/org.eclipse.jgit.test/tst/org/eclipse/jgit/internal/storage/file/PackIndexTestCase.java @@ -66,7 +66,9 @@ public abstract class PackIndexTestCase extends RepositoryTestCase { * Verify CRC32 support. * * @throws MissingObjectException + * object is missing in the underlying index * @throws UnsupportedOperationException + * the index doesn't have CRC */ public abstract void testCRC32() throws MissingObjectException, UnsupportedOperationException; diff --git a/org.eclipse.jgit.test/tst/org/eclipse/jgit/test/resources/SampleDataRepositoryTestCase.java b/org.eclipse.jgit.test/tst/org/eclipse/jgit/test/resources/SampleDataRepositoryTestCase.java index 578128326f..4f5e35f129 100644 --- a/org.eclipse.jgit.test/tst/org/eclipse/jgit/test/resources/SampleDataRepositoryTestCase.java +++ b/org.eclipse.jgit.test/tst/org/eclipse/jgit/test/resources/SampleDataRepositoryTestCase.java @@ -35,6 +35,7 @@ public abstract class SampleDataRepositoryTestCase extends RepositoryTestCase { * @param repo * test repository to receive packfile copies * @throws IOException + * an error occurred */ public static void copyCGitTestPacks(FileRepository repo) throws IOException { final String[] packs = { diff --git a/org.eclipse.jgit.test/tst/org/eclipse/jgit/transport/RequestValidatorTestCase.java b/org.eclipse.jgit.test/tst/org/eclipse/jgit/transport/RequestValidatorTestCase.java index cc910b3b9f..f589a399a4 100644 --- a/org.eclipse.jgit.test/tst/org/eclipse/jgit/transport/RequestValidatorTestCase.java +++ b/org.eclipse.jgit.test/tst/org/eclipse/jgit/transport/RequestValidatorTestCase.java @@ -70,20 +70,30 @@ public abstract class RequestValidatorTestCase { } /** + * Check if the validator accepts a reachable commit + * * @return true if a commit reachable from a visible tip (but not directly * the tip) is valid */ protected abstract boolean isReachableCommitValid(); - /** @return true if a commit not reachable from any tip is valid */ + /** + * Check if the validator accepts an unreachable commit + * + * @return true if a commit not reachable from any tip is valid + **/ protected abstract boolean isUnreachableCommitValid(); /** + * Check if the validator accepts a previously advertised tip + * * @return true if the commit directly pointed by an advertised ref is valid */ protected abstract boolean isAdvertisedTipValid(); /** + * Check if the validator accepts a previous unadvertised tip + * * @return true if the object directly pointed by a non-advertised ref is * valid */ @@ -92,17 +102,23 @@ public abstract class RequestValidatorTestCase { // UploadPack doesn't allow to ask for blobs when there is no // bitmap. Test both cases separately. /** + * Check if the validator accepts a reachable blob (repo with bitmaps) + * * @return true if a reachable blob is valid (and the repo has bitmaps) */ protected abstract boolean isReachableBlobValid_withBitmaps(); /** + * Check if the validator accepts a reachable blob (repo without bitmaps) + * * @return true if a reachable blob is valid (and the repo does NOT have * bitmaps) */ protected abstract boolean isReachableBlobValid_withoutBitmaps(); /** + * Check if the validator accepts an unreachable blob + * * @return true if a blob unreachable from any tip is valid */ protected abstract boolean isUnreachableBlobValid(); -- 2.39.5