From 8b159b6235224bf14b9527ff61f5de7adc25dd3a Mon Sep 17 00:00:00 2001 From: David Pursehouse Date: Fri, 1 Dec 2017 10:56:18 +0900 Subject: [PATCH] Make local assert methods private in test classes Change-Id: I1bed28a1eac3c7f84cc40841853b9540c72be265 Signed-off-by: David Pursehouse --- .../org/eclipse/jgit/attributes/AttributesMatcherTest.java | 4 ++-- .../tst/org/eclipse/jgit/ignore/FastIgnoreRuleTest.java | 4 ++-- .../eclipse/jgit/ignore/IgnoreMatcherParametrizedTest.java | 4 ++-- 3 files changed, 6 insertions(+), 6 deletions(-) diff --git a/org.eclipse.jgit.test/tst/org/eclipse/jgit/attributes/AttributesMatcherTest.java b/org.eclipse.jgit.test/tst/org/eclipse/jgit/attributes/AttributesMatcherTest.java index 23c416a45b..05484ab671 100644 --- a/org.eclipse.jgit.test/tst/org/eclipse/jgit/attributes/AttributesMatcherTest.java +++ b/org.eclipse.jgit.test/tst/org/eclipse/jgit/attributes/AttributesMatcherTest.java @@ -391,7 +391,7 @@ public class AttributesMatcherTest { * @param target * Target file path relative to repository's GIT_DIR */ - public void assertMatched(String pattern, String target) { + private void assertMatched(String pattern, String target) { boolean value = match(pattern, target); assertTrue("Expected a match for: " + pattern + " with: " + target, value); @@ -406,7 +406,7 @@ public class AttributesMatcherTest { * @param target * Target file path relative to repository's GIT_DIR */ - public void assertNotMatched(String pattern, String target) { + private void assertNotMatched(String pattern, String target) { boolean value = match(pattern, target); assertFalse("Expected no match for: " + pattern + " with: " + target, value); diff --git a/org.eclipse.jgit.test/tst/org/eclipse/jgit/ignore/FastIgnoreRuleTest.java b/org.eclipse.jgit.test/tst/org/eclipse/jgit/ignore/FastIgnoreRuleTest.java index bcc8f7e47f..06164c8a91 100644 --- a/org.eclipse.jgit.test/tst/org/eclipse/jgit/ignore/FastIgnoreRuleTest.java +++ b/org.eclipse.jgit.test/tst/org/eclipse/jgit/ignore/FastIgnoreRuleTest.java @@ -465,7 +465,7 @@ public class FastIgnoreRuleTest { split("/a/b/c/", '/').toArray()); } - public void assertMatched(String pattern, String path) { + private void assertMatched(String pattern, String path) { boolean match = match(pattern, path); String result = path + " is " + (match ? "ignored" : "not ignored") + " via '" + pattern + "' rule"; @@ -485,7 +485,7 @@ public class FastIgnoreRuleTest { match); } - public void assertNotMatched(String pattern, String path) { + private void assertNotMatched(String pattern, String path) { boolean match = match(pattern, path); String result = path + " is " + (match ? "ignored" : "not ignored") + " via '" + pattern + "' rule"; diff --git a/org.eclipse.jgit.test/tst/org/eclipse/jgit/ignore/IgnoreMatcherParametrizedTest.java b/org.eclipse.jgit.test/tst/org/eclipse/jgit/ignore/IgnoreMatcherParametrizedTest.java index 529c75ff39..137230d09f 100644 --- a/org.eclipse.jgit.test/tst/org/eclipse/jgit/ignore/IgnoreMatcherParametrizedTest.java +++ b/org.eclipse.jgit.test/tst/org/eclipse/jgit/ignore/IgnoreMatcherParametrizedTest.java @@ -346,7 +346,7 @@ public class IgnoreMatcherParametrizedTest { * Target file path relative to repository's GIT_DIR * @param assume */ - public void assertMatched(String pattern, String target, Boolean... assume) { + private void assertMatched(String pattern, String target, Boolean... assume) { boolean value = match(pattern, target); if (assume.length == 0 || !assume[0].booleanValue()) assertTrue("Expected a match for: " + pattern + " with: " + target, @@ -366,7 +366,7 @@ public class IgnoreMatcherParametrizedTest { * Target file path relative to repository's GIT_DIR * @param assume */ - public void assertNotMatched(String pattern, String target, + private void assertNotMatched(String pattern, String target, Boolean... assume) { boolean value = match(pattern, target); if (assume.length == 0 || !assume[0].booleanValue()) -- 2.39.5