Polishing

junit-team · Sep 8, 2024 · 044f5cf · 044f5cf
1 parent 40bc1c4
commit 044f5cf
Show file tree

Hide file tree

Showing 3 changed files with 19 additions and 19 deletions.
diff --git a/documentation/src/docs/asciidoc/release-notes/release-notes-5.11.1.adoc b/documentation/src/docs/asciidoc/release-notes/release-notes-5.11.1.adoc
@@ -6,8 +6,8 @@
 *Scope:* minor bug fixes and improvements since 5.11.0
 
 For a complete list of all _closed_ issues and pull requests for this release, consult the
-link:{junit5-repo}+/milestone/79?closed=1+[5.11.1] milestone page in the
-JUnit repository on GitHub.
+link:{junit5-repo}+/milestone/79?closed=1+[5.11.1] milestone page in the JUnit repository
+on GitHub.
 
 
 [[release-notes-5.11.1-junit-platform]]
@@ -16,8 +16,8 @@ JUnit repository on GitHub.
 [[release-notes-5.11.1-junit-platform-bug-fixes]]
 ==== Bug Fixes
 
-* Fix potential locking issue with `ExclusiveResource` in the `HierarchicalTestExecutorService`.
-  This could lead to deadlocks in certain scenarios.
+* Fixed potential locking issue with `ExclusiveResource` in the
+  `HierarchicalTestExecutorService`, which could lead to deadlocks in certain scenarios.
 
 [[release-notes-5.11.1-junit-platform-deprecations-and-breaking-changes]]
 ==== Deprecations and Breaking Changes

diff --git a/...rm-engine/src/main/java/org/junit/platform/engine/support/hierarchical/CompositeLock.java b/...rm-engine/src/main/java/org/junit/platform/engine/support/hierarchical/CompositeLock.java
@@ -28,7 +28,7 @@ class CompositeLock implements ResourceLock {
 
 	// for tests only
 	List<Lock> getLocks() {
-		return locks;
+		return this.locks;
 	}
 
 	@Override
@@ -38,9 +38,9 @@ public ResourceLock acquire() throws InterruptedException {
 	}
 
 	private void acquireAllLocks() throws InterruptedException {
-		List<Lock> acquiredLocks = new ArrayList<>(locks.size());
+		List<Lock> acquiredLocks = new ArrayList<>(this.locks.size());
 		try {
-			for (Lock lock : locks) {
+			for (Lock lock : this.locks) {
 				lock.lockInterruptibly();
 				acquiredLocks.add(lock);
 			}
@@ -53,7 +53,7 @@ private void acquireAllLocks() throws InterruptedException {
 
 	@Override
 	public void release() {
-		release(locks);
+		release(this.locks);
 	}
 
 	private void release(List<Lock> acquiredLocks) {
@@ -64,20 +64,20 @@ private void release(List<Lock> acquiredLocks) {
 
 	private class CompositeLockManagedBlocker implements ForkJoinPool.ManagedBlocker {
 
-		private boolean acquired;
+		private volatile boolean acquired;
 
 		@Override
 		public boolean block() throws InterruptedException {
-			if (!acquired) {
+			if (!this.acquired) {
 				acquireAllLocks();
-				acquired = true;
+				this.acquired = true;
 			}
 			return true;
 		}
 
 		@Override
 		public boolean isReleasable() {
-			return acquired;
+			return this.acquired;
 		}
 
 	}

diff --git a/...tform-engine/src/main/java/org/junit/platform/engine/support/hierarchical/SingleLock.java b/...tform-engine/src/main/java/org/junit/platform/engine/support/hierarchical/SingleLock.java
@@ -26,7 +26,7 @@ class SingleLock implements ResourceLock {
 
 	// for tests only
 	Lock getLock() {
-		return lock;
+		return this.lock;
 	}
 
 	@Override
@@ -37,25 +37,25 @@ public ResourceLock acquire() throws InterruptedException {
 
 	@Override
 	public void release() {
-		lock.unlock();
+		this.lock.unlock();
 	}
 
 	private class SingleLockManagedBlocker implements ForkJoinPool.ManagedBlocker {
 
-		private boolean acquired;
+		private volatile boolean acquired;
 
 		@Override
 		public boolean block() throws InterruptedException {
-			if (!acquired) {
-				lock.lockInterruptibly();
-				acquired = true;
+			if (!this.acquired) {
+				SingleLock.this.lock.lockInterruptibly();
+				this.acquired = true;
 			}
 			return true;
 		}
 
 		@Override
 		public boolean isReleasable() {
-			return acquired || (acquired = lock.tryLock());
+			return this.acquired || (this.acquired = SingleLock.this.lock.tryLock());
 		}
 
 	}