You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by gg...@apache.org on 2022/11/16 14:27:44 UTC

[commons-bcel] 02/02: Test cases a public by convention in BCEL

This is an automated email from the ASF dual-hosted git repository.

ggregory pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-bcel.git

commit c9ee7fcec4c651f894bfe63431d1586ef58836a1
Author: Gary David Gregory (Code signing key) <gg...@apache.org>
AuthorDate: Wed Nov 16 09:27:36 2022 -0500

    Test cases a public by convention in BCEL
---
 src/main/java/org/apache/bcel/verifier/statics/Pass3aVerifier.java    | 4 ++--
 .../java/org/apache/bcel/verifier/VerifierArrayAccessTestCase.java    | 2 +-
 src/test/java/org/apache/bcel/verifier/VerifierReturnTestCase.java    | 2 +-
 3 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/main/java/org/apache/bcel/verifier/statics/Pass3aVerifier.java b/src/main/java/org/apache/bcel/verifier/statics/Pass3aVerifier.java
index f03914e6..3c8bd94f 100644
--- a/src/main/java/org/apache/bcel/verifier/statics/Pass3aVerifier.java
+++ b/src/main/java/org/apache/bcel/verifier/statics/Pass3aVerifier.java
@@ -870,8 +870,8 @@ public final class Pass3aVerifier extends PassVerifier {
                 }
 
                 if (f.isFinal() && !verifier.getClassName().equals(getObjectType(o).getClassName())) {
-                    constraintViolated(o, "Referenced field '" + f + "' is final and must therefore be declared in the current class '" + verifier.getClassName()
-                        + "' which is not the case: it is declared in '" + o.getReferenceType(constantPoolGen) + "'.");
+                    constraintViolated(o, "Referenced field '" + f + "' is final and must therefore be declared in the current class '"
+                            + verifier.getClassName() + "' which is not the case: it is declared in '" + o.getReferenceType(constantPoolGen) + "'.");
                 }
 
                 if (!f.isStatic()) {
diff --git a/src/test/java/org/apache/bcel/verifier/VerifierArrayAccessTestCase.java b/src/test/java/org/apache/bcel/verifier/VerifierArrayAccessTestCase.java
index 35b6004c..03b5561d 100644
--- a/src/test/java/org/apache/bcel/verifier/VerifierArrayAccessTestCase.java
+++ b/src/test/java/org/apache/bcel/verifier/VerifierArrayAccessTestCase.java
@@ -30,7 +30,7 @@ import org.apache.bcel.verifier.tests.TestArrayAccess04UnknownCreator;
 import org.junit.jupiter.api.Assertions;
 import org.junit.jupiter.api.Test;
 
-class VerifierArrayAccessTestCase extends AbstractVerifierTestCase {
+public class VerifierArrayAccessTestCase extends AbstractVerifierTestCase {
 
     @Test
     public void testInvalidArrayAccess() throws IOException, ClassNotFoundException {
diff --git a/src/test/java/org/apache/bcel/verifier/VerifierReturnTestCase.java b/src/test/java/org/apache/bcel/verifier/VerifierReturnTestCase.java
index 692c73a0..41dc2db9 100644
--- a/src/test/java/org/apache/bcel/verifier/VerifierReturnTestCase.java
+++ b/src/test/java/org/apache/bcel/verifier/VerifierReturnTestCase.java
@@ -31,7 +31,7 @@ import org.apache.bcel.verifier.tests.TestReturn03UnknownCreator;
 import org.junit.jupiter.api.Assertions;
 import org.junit.jupiter.api.Test;
 
-class VerifierReturnTestCase extends AbstractVerifierTestCase {
+public class VerifierReturnTestCase extends AbstractVerifierTestCase {
 
     @Test
     public void testInvalidReturn() throws IOException, ClassNotFoundException {