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 2025/06/14 12:19:45 UTC

(commons-lang) branch master updated: Add missing test per JaCoCo test coverage

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-lang.git


The following commit(s) were added to refs/heads/master by this push:
     new 974458208 Add missing test per JaCoCo test coverage
974458208 is described below

commit 974458208df7c6ccf630442a79a51da8869e8f47
Author: Gary D. Gregory <ga...@gmail.com>
AuthorDate: Sat Jun 14 08:19:40 2025 -0400

    Add missing test per JaCoCo test coverage
    
    Fix Checkstyle
---
 .../apache/commons/lang3/concurrent/locks/LockingVisitorsTest.java   | 5 +++--
 src/test/java/org/apache/commons/lang3/tuple/MutablePairTest.java    | 4 ++--
 2 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/src/test/java/org/apache/commons/lang3/concurrent/locks/LockingVisitorsTest.java b/src/test/java/org/apache/commons/lang3/concurrent/locks/LockingVisitorsTest.java
index e9a1f6a25..3670337b8 100644
--- a/src/test/java/org/apache/commons/lang3/concurrent/locks/LockingVisitorsTest.java
+++ b/src/test/java/org/apache/commons/lang3/concurrent/locks/LockingVisitorsTest.java
@@ -49,9 +49,10 @@ protected boolean containsTrue(final boolean[] booleanArray) {
     }
 
     private void runTest(final Duration delay, final boolean exclusiveLock, final LongConsumer runTimeCheck,
-        final boolean[] booleanValues, final LockVisitor<boolean[], ?> visitor) throws InterruptedException {
+            final boolean[] booleanValues, final LockVisitor<boolean[], ?> visitor) throws InterruptedException {
+        assertNotNull(visitor.getLock());
+        assertNotNull(visitor.getObject());
         final boolean[] runningValues = new boolean[10];
-
         final long startTimeMillis = System.currentTimeMillis();
         for (int i = 0; i < booleanValues.length; i++) {
             final int index = i;
diff --git a/src/test/java/org/apache/commons/lang3/tuple/MutablePairTest.java b/src/test/java/org/apache/commons/lang3/tuple/MutablePairTest.java
index 5a13bf064..8a4a181a1 100644
--- a/src/test/java/org/apache/commons/lang3/tuple/MutablePairTest.java
+++ b/src/test/java/org/apache/commons/lang3/tuple/MutablePairTest.java
@@ -116,8 +116,8 @@ void testOfNonNull() {
 
     @Test
     void testPairOfMapEntry() {
-        assertNull( MutablePair.of(null).getLeft());
-        assertNull( MutablePair.of(null).getRight());
+        assertNull(MutablePair.of(null).getLeft());
+        assertNull(MutablePair.of(null).getRight());
         final HashMap<Integer, String> map = new HashMap<>();
         map.put(0, "foo");
         final Entry<Integer, String> entry = map.entrySet().iterator().next();