ViewVC Help
View File | Revision Log | Show Annotations | Download File | Root Listing
root/jsr166/jsr166/src/test/tck/SemaphoreTest.java
(Generate patch)

Comparing jsr166/src/test/tck/SemaphoreTest.java (file contents):
Revision 1.20 by jsr166, Sat Nov 21 02:07:27 2009 UTC vs.
Revision 1.39 by jsr166, Sun May 14 03:31:53 2017 UTC

# Line 1 | Line 1
1   /*
2   * Written by Doug Lea with assistance from members of JCP JSR-166
3   * Expert Group and released to the public domain, as explained at
4 < * http://creativecommons.org/licenses/publicdomain
4 > * http://creativecommons.org/publicdomain/zero/1.0/
5   * Other contributors include Andrew Wright, Jeffrey Hayes,
6   * Pat Fisher, Mike Judd.
7   */
8  
9 import junit.framework.*;
10 import java.util.*;
11 import java.util.concurrent.*;
9   import static java.util.concurrent.TimeUnit.MILLISECONDS;
10 < import java.io.*;
10 >
11 > import java.util.Collection;
12 > import java.util.concurrent.CountDownLatch;
13 > import java.util.concurrent.Semaphore;
14 > import java.util.concurrent.ThreadLocalRandom;
15 >
16 > import junit.framework.AssertionFailedError;
17 > import junit.framework.Test;
18 > import junit.framework.TestSuite;
19  
20   public class SemaphoreTest extends JSR166TestCase {
21      public static void main(String[] args) {
22 <        junit.textui.TestRunner.run (suite());
22 >        main(suite(), args);
23      }
24      public static Test suite() {
25          return new TestSuite(SemaphoreTest.class);
# Line 24 | Line 29 | public class SemaphoreTest extends JSR16
29       * Subclass to expose protected methods
30       */
31      static class PublicSemaphore extends Semaphore {
32 <        PublicSemaphore(int p, boolean f) { super(p, f); }
32 >        PublicSemaphore(int permits) { super(permits); }
33 >        PublicSemaphore(int permits, boolean fair) { super(permits, fair); }
34          public Collection<Thread> getQueuedThreads() {
35              return super.getQueuedThreads();
36          }
37 <        public void reducePermits(int p) {
38 <            super.reducePermits(p);
37 >        public boolean hasQueuedThread(Thread t) {
38 >            return super.getQueuedThreads().contains(t);
39 >        }
40 >        public void reducePermits(int reduction) {
41 >            super.reducePermits(reduction);
42          }
43      }
44  
# Line 38 | Line 47 | public class SemaphoreTest extends JSR16
47       */
48      class InterruptibleLockRunnable extends CheckedRunnable {
49          final Semaphore lock;
50 <        InterruptibleLockRunnable(Semaphore l) { lock = l; }
50 >        InterruptibleLockRunnable(Semaphore s) { lock = s; }
51          public void realRun() {
52              try {
53                  lock.acquire();
# Line 47 | Line 56 | public class SemaphoreTest extends JSR16
56          }
57      }
58  
50
59      /**
60 <     * A runnable calling acquire that expects to be
53 <     * interrupted
60 >     * A runnable calling acquire that expects to be interrupted
61       */
62      class InterruptedLockRunnable extends CheckedInterruptedRunnable {
63          final Semaphore lock;
64 <        InterruptedLockRunnable(Semaphore l) { lock = l; }
64 >        InterruptedLockRunnable(Semaphore s) { lock = s; }
65          public void realRun() throws InterruptedException {
66              lock.acquire();
67          }
68      }
69  
70      /**
71 <     * Zero, negative, and positive initial values are allowed in constructor
71 >     * Spin-waits until s.hasQueuedThread(t) becomes true.
72 >     */
73 >    void waitForQueuedThread(PublicSemaphore s, Thread t) {
74 >        long startTime = System.nanoTime();
75 >        while (!s.hasQueuedThread(t)) {
76 >            if (millisElapsedSince(startTime) > LONG_DELAY_MS)
77 >                throw new AssertionFailedError("timed out");
78 >            Thread.yield();
79 >        }
80 >        assertTrue(s.hasQueuedThreads());
81 >        assertTrue(t.isAlive());
82 >    }
83 >
84 >    /**
85 >     * Spin-waits until s.hasQueuedThreads() becomes true.
86       */
87 <    public void testConstructor() {
88 <        for (int permits : new int[] { -1, 0, 1 }) {
89 <            for (boolean fair : new boolean[] { false, true }) {
90 <                Semaphore s = new Semaphore(permits, fair);
91 <                assertEquals(permits, s.availablePermits());
92 <                assertEquals(fair, s.isFair());
87 >    void waitForQueuedThreads(Semaphore s) {
88 >        long startTime = System.nanoTime();
89 >        while (!s.hasQueuedThreads()) {
90 >            if (millisElapsedSince(startTime) > LONG_DELAY_MS)
91 >                throw new AssertionFailedError("timed out");
92 >            Thread.yield();
93 >        }
94 >    }
95 >
96 >    enum AcquireMethod {
97 >        acquire() {
98 >            void acquire(Semaphore s) throws InterruptedException {
99 >                s.acquire();
100 >            }
101 >        },
102 >        acquireN() {
103 >            void acquire(Semaphore s, int permits) throws InterruptedException {
104 >                s.acquire(permits);
105 >            }
106 >        },
107 >        acquireUninterruptibly() {
108 >            void acquire(Semaphore s) {
109 >                s.acquireUninterruptibly();
110 >            }
111 >        },
112 >        acquireUninterruptiblyN() {
113 >            void acquire(Semaphore s, int permits) {
114 >                s.acquireUninterruptibly(permits);
115              }
116 +        },
117 +        tryAcquire() {
118 +            void acquire(Semaphore s) {
119 +                assertTrue(s.tryAcquire());
120 +            }
121 +        },
122 +        tryAcquireN() {
123 +            void acquire(Semaphore s, int permits) {
124 +                assertTrue(s.tryAcquire(permits));
125 +            }
126 +        },
127 +        tryAcquireTimed() {
128 +            void acquire(Semaphore s) throws InterruptedException {
129 +                assertTrue(s.tryAcquire(2 * LONG_DELAY_MS, MILLISECONDS));
130 +            }
131 +            Thread.State parkedState() { return Thread.State.TIMED_WAITING; }
132 +        },
133 +        tryAcquireTimedN {
134 +            void acquire(Semaphore s, int permits) throws InterruptedException {
135 +                assertTrue(s.tryAcquire(permits, 2 * LONG_DELAY_MS, MILLISECONDS));
136 +            }
137 +            Thread.State parkedState() { return Thread.State.TIMED_WAITING; }
138 +        };
139 +
140 +        // Intentionally meta-circular
141 +
142 +        /** Acquires 1 permit. */
143 +        void acquire(Semaphore s) throws InterruptedException {
144 +            acquire(s, 1);
145 +        }
146 +        /** Acquires the given number of permits. */
147 +        void acquire(Semaphore s, int permits) throws InterruptedException {
148 +            for (int i = 0; i < permits; i++)
149 +                acquire(s);
150 +        }
151 +        Thread.State parkedState() { return Thread.State.WAITING; }
152 +    }
153 +
154 +    /**
155 +     * Zero, negative, and positive initial values are allowed in constructor
156 +     */
157 +    public void testConstructor()      { testConstructor(false); }
158 +    public void testConstructor_fair() { testConstructor(true); }
159 +    public void testConstructor(boolean fair) {
160 +        for (int permits : new int[] { -42, -1, 0, 1, 42 }) {
161 +            Semaphore s = new Semaphore(permits, fair);
162 +            assertEquals(permits, s.availablePermits());
163 +            assertEquals(fair, s.isFair());
164          }
165      }
166  
167      /**
168       * Constructor without fairness argument behaves as nonfair
169       */
170 <    public void testConstructor2() {
171 <        for (int permits : new int[] { -1, 0, 1 }) {
170 >    public void testConstructorDefaultsToNonFair() {
171 >        for (int permits : new int[] { -42, -1, 0, 1, 42 }) {
172              Semaphore s = new Semaphore(permits);
173              assertEquals(permits, s.availablePermits());
174              assertFalse(s.isFair());
# Line 87 | Line 178 | public class SemaphoreTest extends JSR16
178      /**
179       * tryAcquire succeeds when sufficient permits, else fails
180       */
181 <    public void testTryAcquireInSameThread() {
182 <        Semaphore s = new Semaphore(2, false);
181 >    public void testTryAcquireInSameThread()      { testTryAcquireInSameThread(false); }
182 >    public void testTryAcquireInSameThread_fair() { testTryAcquireInSameThread(true); }
183 >    public void testTryAcquireInSameThread(boolean fair) {
184 >        Semaphore s = new Semaphore(2, fair);
185          assertEquals(2, s.availablePermits());
186          assertTrue(s.tryAcquire());
187          assertTrue(s.tryAcquire());
188          assertEquals(0, s.availablePermits());
189          assertFalse(s.tryAcquire());
190 +        assertFalse(s.tryAcquire());
191 +        assertEquals(0, s.availablePermits());
192      }
193  
194      /**
195 <     * Acquire and release of semaphore succeed if initially available
101 <     */
102 <    public void testAcquireReleaseInSameThread()
103 <        throws InterruptedException {
104 <        Semaphore s = new Semaphore(1, false);
105 <        s.acquire();
106 <        s.release();
107 <        s.acquire();
108 <        s.release();
109 <        s.acquire();
110 <        s.release();
111 <        s.acquire();
112 <        s.release();
113 <        s.acquire();
114 <        s.release();
115 <        assertEquals(1, s.availablePermits());
116 <    }
117 <
118 <    /**
119 <     * Uninterruptible acquire and release of semaphore succeed if
120 <     * initially available
195 >     * timed tryAcquire times out
196       */
197 <    public void testAcquireUninterruptiblyReleaseInSameThread()
198 <        throws InterruptedException {
199 <        Semaphore s = new Semaphore(1, false);
200 <        s.acquireUninterruptibly();
201 <        s.release();
202 <        s.acquireUninterruptibly();
203 <        s.release();
204 <        s.acquireUninterruptibly();
205 <        s.release();
206 <        s.acquireUninterruptibly();
207 <        s.release();
208 <        s.acquireUninterruptibly();
209 <        s.release();
210 <        assertEquals(1, s.availablePermits());
197 >    public void testTryAcquire_timeout() {
198 >        final ThreadLocalRandom rnd = ThreadLocalRandom.current();
199 >        final boolean fair = rnd.nextBoolean();
200 >        final Semaphore s = new Semaphore(0, fair);
201 >        final long startTime = System.nanoTime();
202 >        try { assertFalse(s.tryAcquire(timeoutMillis(), MILLISECONDS)); }
203 >        catch (InterruptedException e) { threadUnexpectedException(e); }
204 >        assertTrue(millisElapsedSince(startTime) >= timeoutMillis());
205 >    }
206 >
207 >    /**
208 >     * timed tryAcquire(N) times out
209 >     */
210 >    public void testTryAcquireN_timeout() {
211 >        final ThreadLocalRandom rnd = ThreadLocalRandom.current();
212 >        final boolean fair = rnd.nextBoolean();
213 >        final Semaphore s = new Semaphore(2, fair);
214 >        final long startTime = System.nanoTime();
215 >        try { assertFalse(s.tryAcquire(3, timeoutMillis(), MILLISECONDS)); }
216 >        catch (InterruptedException e) { threadUnexpectedException(e); }
217 >        assertTrue(millisElapsedSince(startTime) >= timeoutMillis());
218 >    }
219 >
220 >    /**
221 >     * acquire(), acquire(N), timed tryAcquired, timed tryAcquire(N)
222 >     * are interruptible
223 >     */
224 >    public void testInterruptible_acquire()               { testInterruptible(false, AcquireMethod.acquire); }
225 >    public void testInterruptible_acquire_fair()          { testInterruptible(true,  AcquireMethod.acquire); }
226 >    public void testInterruptible_acquireN()              { testInterruptible(false, AcquireMethod.acquireN); }
227 >    public void testInterruptible_acquireN_fair()         { testInterruptible(true,  AcquireMethod.acquireN); }
228 >    public void testInterruptible_tryAcquireTimed()       { testInterruptible(false, AcquireMethod.tryAcquireTimed); }
229 >    public void testInterruptible_tryAcquireTimed_fair()  { testInterruptible(true,  AcquireMethod.tryAcquireTimed); }
230 >    public void testInterruptible_tryAcquireTimedN()      { testInterruptible(false, AcquireMethod.tryAcquireTimedN); }
231 >    public void testInterruptible_tryAcquireTimedN_fair() { testInterruptible(true,  AcquireMethod.tryAcquireTimedN); }
232 >    public void testInterruptible(boolean fair, final AcquireMethod acquirer) {
233 >        final PublicSemaphore s = new PublicSemaphore(0, fair);
234 >        final java.util.concurrent.CyclicBarrier pleaseInterrupt
235 >            = new java.util.concurrent.CyclicBarrier(2);
236 >        Thread t = newStartedThread(new CheckedRunnable() {
237 >            public void realRun() {
238 >                // Interrupt before acquire
239 >                Thread.currentThread().interrupt();
240 >                try {
241 >                    acquirer.acquire(s);
242 >                    shouldThrow();
243 >                } catch (InterruptedException success) {}
244 >                assertFalse(Thread.interrupted());
245 >
246 >                // Interrupt before acquire(N)
247 >                Thread.currentThread().interrupt();
248 >                try {
249 >                    acquirer.acquire(s, 3);
250 >                    shouldThrow();
251 >                } catch (InterruptedException success) {}
252 >                assertFalse(Thread.interrupted());
253 >
254 >                // Interrupt during acquire
255 >                await(pleaseInterrupt);
256 >                try {
257 >                    acquirer.acquire(s);
258 >                    shouldThrow();
259 >                } catch (InterruptedException success) {}
260 >                assertFalse(Thread.interrupted());
261 >
262 >                // Interrupt during acquire(N)
263 >                await(pleaseInterrupt);
264 >                try {
265 >                    acquirer.acquire(s, 3);
266 >                    shouldThrow();
267 >                } catch (InterruptedException success) {}
268 >                assertFalse(Thread.interrupted());
269 >            }});
270 >
271 >        for (int n = 2; n-->0; ) {
272 >            await(pleaseInterrupt);
273 >            assertThreadBlocks(t, acquirer.parkedState());
274 >            t.interrupt();
275 >        }
276 >        
277 >        awaitTermination(t);
278      }
279  
280      /**
281 <     * Timed Acquire and release of semaphore succeed if
282 <     * initially available
281 >     * acquireUninterruptibly(), acquireUninterruptibly(N) are
282 >     * uninterruptible
283       */
284 <    public void testTimedAcquireReleaseInSameThread()
285 <        throws InterruptedException {
286 <        Semaphore s = new Semaphore(1, false);
287 <        assertTrue(s.tryAcquire(SHORT_DELAY_MS, MILLISECONDS));
288 <        s.release();
289 <        assertTrue(s.tryAcquire(SHORT_DELAY_MS, MILLISECONDS));
290 <        s.release();
149 <        assertTrue(s.tryAcquire(SHORT_DELAY_MS, MILLISECONDS));
150 <        s.release();
151 <        assertTrue(s.tryAcquire(SHORT_DELAY_MS, MILLISECONDS));
152 <        s.release();
153 <        assertTrue(s.tryAcquire(SHORT_DELAY_MS, MILLISECONDS));
154 <        s.release();
155 <        assertEquals(1, s.availablePermits());
156 <    }
284 >    public void testUninterruptible_acquireUninterruptibly()       { testUninterruptible(false, AcquireMethod.acquireUninterruptibly); }
285 >    public void testUninterruptible_acquireUninterruptibly_fair()  { testUninterruptible(true,  AcquireMethod.acquireUninterruptibly); }
286 >    public void testUninterruptible_acquireUninterruptiblyN()      { testUninterruptible(false, AcquireMethod.acquireUninterruptiblyN); }
287 >    public void testUninterruptible_acquireUninterruptiblyN_fair() { testUninterruptible(true,  AcquireMethod.acquireUninterruptiblyN); }
288 >    public void testUninterruptible(boolean fair, final AcquireMethod acquirer) {
289 >        final PublicSemaphore s = new PublicSemaphore(0, fair);
290 >        final Semaphore pleaseInterrupt = new Semaphore(-1, fair);
291  
292 <    /**
159 <     * A release in one thread enables an acquire in another thread
160 <     */
161 <    public void testAcquireReleaseInDifferentThreads()
162 <        throws InterruptedException {
163 <        final Semaphore s = new Semaphore(0, false);
164 <        Thread t = new Thread(new CheckedRunnable() {
292 >        Thread t1 = newStartedThread(new CheckedRunnable() {
293              public void realRun() throws InterruptedException {
294 <                s.acquire();
295 <                s.release();
296 <                s.release();
297 <                s.acquire();
294 >                // Interrupt before acquire
295 >                pleaseInterrupt.release();
296 >                Thread.currentThread().interrupt();
297 >                acquirer.acquire(s);
298 >                assertTrue(Thread.interrupted());
299              }});
300  
301 <        t.start();
173 <        Thread.sleep(SHORT_DELAY_MS);
174 <        s.release();
175 <        s.release();
176 <        s.acquire();
177 <        s.acquire();
178 <        s.release();
179 <        t.join();
180 <    }
181 <
182 <    /**
183 <     * A release in one thread enables an uninterruptible acquire in another thread
184 <     */
185 <    public void testUninterruptibleAcquireReleaseInDifferentThreads()
186 <        throws InterruptedException {
187 <        final Semaphore s = new Semaphore(0, false);
188 <        Thread t = new Thread(new CheckedRunnable() {
189 <            public void realRun() throws InterruptedException {
190 <                s.acquireUninterruptibly();
191 <                s.release();
192 <                s.release();
193 <                s.acquireUninterruptibly();
194 <            }});
195 <
196 <        t.start();
197 <        Thread.sleep(SHORT_DELAY_MS);
198 <        s.release();
199 <        s.release();
200 <        s.acquireUninterruptibly();
201 <        s.acquireUninterruptibly();
202 <        s.release();
203 <        t.join();
204 <    }
205 <
206 <
207 <    /**
208 <     *  A release in one thread enables a timed acquire in another thread
209 <     */
210 <    public void testTimedAcquireReleaseInDifferentThreads()
211 <        throws InterruptedException {
212 <        final Semaphore s = new Semaphore(1, false);
213 <        Thread t = new Thread(new CheckedRunnable() {
301 >        Thread t2 = newStartedThread(new CheckedRunnable() {
302              public void realRun() throws InterruptedException {
303 <                s.release();
304 <                threadAssertTrue(s.tryAcquire(SHORT_DELAY_MS, MILLISECONDS));
305 <                s.release();
306 <                threadAssertTrue(s.tryAcquire(SHORT_DELAY_MS, MILLISECONDS));
303 >                // Interrupt during acquire
304 >                pleaseInterrupt.release();
305 >                acquirer.acquire(s);
306 >                assertTrue(Thread.interrupted());
307              }});
308  
309 <        t.start();
310 <        assertTrue(s.tryAcquire(SHORT_DELAY_MS, MILLISECONDS));
311 <        s.release();
312 <        assertTrue(s.tryAcquire(SHORT_DELAY_MS, MILLISECONDS));
225 <        s.release();
226 <        s.release();
227 <        t.join();
228 <    }
229 <
230 <    /**
231 <     * A waiting acquire blocks interruptibly
232 <     */
233 <    public void testAcquire_InterruptedException()
234 <        throws InterruptedException {
235 <        final Semaphore s = new Semaphore(0, false);
236 <        Thread t = new Thread(new CheckedInterruptedRunnable() {
237 <            public void realRun() throws InterruptedException {
238 <                s.acquire();
239 <            }});
309 >        await(pleaseInterrupt);
310 >        waitForQueuedThread(s, t1);
311 >        waitForQueuedThread(s, t2);
312 >        t2.interrupt();
313  
314 <        t.start();
315 <        Thread.sleep(SHORT_DELAY_MS);
243 <        t.interrupt();
244 <        t.join();
245 <    }
314 >        assertThreadBlocks(t1, Thread.State.WAITING);
315 >        assertThreadBlocks(t2, Thread.State.WAITING);
316  
317 <    /**
248 <     *  A waiting timed acquire blocks interruptibly
249 <     */
250 <    public void testTryAcquire_InterruptedException()
251 <        throws InterruptedException {
252 <        final Semaphore s = new Semaphore(0, false);
253 <        Thread t = new Thread(new CheckedInterruptedRunnable() {
254 <            public void realRun() throws InterruptedException {
255 <                s.tryAcquire(MEDIUM_DELAY_MS, MILLISECONDS);
256 <            }});
317 >        s.release(2);
318  
319 <        t.start();
320 <        Thread.sleep(SHORT_DELAY_MS);
260 <        t.interrupt();
261 <        t.join();
319 >        awaitTermination(t1);
320 >        awaitTermination(t2);
321      }
322  
323      /**
324       * hasQueuedThreads reports whether there are waiting threads
325       */
326 <    public void testHasQueuedThreads() throws InterruptedException {
327 <        final Semaphore lock = new Semaphore(1, false);
328 <        Thread t1 = new Thread(new InterruptedLockRunnable(lock));
329 <        Thread t2 = new Thread(new InterruptibleLockRunnable(lock));
326 >    public void testHasQueuedThreads()      { testHasQueuedThreads(false); }
327 >    public void testHasQueuedThreads_fair() { testHasQueuedThreads(true); }
328 >    public void testHasQueuedThreads(boolean fair) {
329 >        final PublicSemaphore lock = new PublicSemaphore(1, fair);
330          assertFalse(lock.hasQueuedThreads());
331          lock.acquireUninterruptibly();
332 <        t1.start();
333 <        Thread.sleep(SHORT_DELAY_MS);
332 >        Thread t1 = newStartedThread(new InterruptedLockRunnable(lock));
333 >        waitForQueuedThread(lock, t1);
334          assertTrue(lock.hasQueuedThreads());
335 <        t2.start();
336 <        Thread.sleep(SHORT_DELAY_MS);
335 >        Thread t2 = newStartedThread(new InterruptibleLockRunnable(lock));
336 >        waitForQueuedThread(lock, t2);
337          assertTrue(lock.hasQueuedThreads());
338          t1.interrupt();
339 <        Thread.sleep(SHORT_DELAY_MS);
339 >        awaitTermination(t1);
340          assertTrue(lock.hasQueuedThreads());
341          lock.release();
342 <        Thread.sleep(SHORT_DELAY_MS);
342 >        awaitTermination(t2);
343          assertFalse(lock.hasQueuedThreads());
285        t1.join();
286        t2.join();
344      }
345  
346      /**
347       * getQueueLength reports number of waiting threads
348       */
349 <    public void testGetQueueLength() throws InterruptedException {
350 <        final Semaphore lock = new Semaphore(1, false);
351 <        Thread t1 = new Thread(new InterruptedLockRunnable(lock));
352 <        Thread t2 = new Thread(new InterruptibleLockRunnable(lock));
349 >    public void testGetQueueLength()      { testGetQueueLength(false); }
350 >    public void testGetQueueLength_fair() { testGetQueueLength(true); }
351 >    public void testGetQueueLength(boolean fair) {
352 >        final PublicSemaphore lock = new PublicSemaphore(1, fair);
353          assertEquals(0, lock.getQueueLength());
354          lock.acquireUninterruptibly();
355 <        t1.start();
356 <        Thread.sleep(SHORT_DELAY_MS);
355 >        Thread t1 = newStartedThread(new InterruptedLockRunnable(lock));
356 >        waitForQueuedThread(lock, t1);
357          assertEquals(1, lock.getQueueLength());
358 <        t2.start();
359 <        Thread.sleep(SHORT_DELAY_MS);
358 >        Thread t2 = newStartedThread(new InterruptibleLockRunnable(lock));
359 >        waitForQueuedThread(lock, t2);
360          assertEquals(2, lock.getQueueLength());
361          t1.interrupt();
362 <        Thread.sleep(SHORT_DELAY_MS);
362 >        awaitTermination(t1);
363          assertEquals(1, lock.getQueueLength());
364          lock.release();
365 <        Thread.sleep(SHORT_DELAY_MS);
365 >        awaitTermination(t2);
366          assertEquals(0, lock.getQueueLength());
310        t1.join();
311        t2.join();
367      }
368  
369      /**
370       * getQueuedThreads includes waiting threads
371       */
372 <    public void testGetQueuedThreads() throws InterruptedException {
373 <        final PublicSemaphore lock = new PublicSemaphore(1, false);
374 <        Thread t1 = new Thread(new InterruptedLockRunnable(lock));
375 <        Thread t2 = new Thread(new InterruptibleLockRunnable(lock));
372 >    public void testGetQueuedThreads()      { testGetQueuedThreads(false); }
373 >    public void testGetQueuedThreads_fair() { testGetQueuedThreads(true); }
374 >    public void testGetQueuedThreads(boolean fair) {
375 >        final PublicSemaphore lock = new PublicSemaphore(1, fair);
376          assertTrue(lock.getQueuedThreads().isEmpty());
377          lock.acquireUninterruptibly();
378          assertTrue(lock.getQueuedThreads().isEmpty());
379 <        t1.start();
380 <        Thread.sleep(SHORT_DELAY_MS);
379 >        Thread t1 = newStartedThread(new InterruptedLockRunnable(lock));
380 >        waitForQueuedThread(lock, t1);
381          assertTrue(lock.getQueuedThreads().contains(t1));
382 <        t2.start();
383 <        Thread.sleep(SHORT_DELAY_MS);
382 >        Thread t2 = newStartedThread(new InterruptibleLockRunnable(lock));
383 >        waitForQueuedThread(lock, t2);
384          assertTrue(lock.getQueuedThreads().contains(t1));
385          assertTrue(lock.getQueuedThreads().contains(t2));
386          t1.interrupt();
387 <        Thread.sleep(SHORT_DELAY_MS);
387 >        awaitTermination(t1);
388          assertFalse(lock.getQueuedThreads().contains(t1));
389          assertTrue(lock.getQueuedThreads().contains(t2));
390          lock.release();
391 <        Thread.sleep(SHORT_DELAY_MS);
391 >        awaitTermination(t2);
392          assertTrue(lock.getQueuedThreads().isEmpty());
338        t1.join();
339        t2.join();
393      }
394  
395      /**
396       * drainPermits reports and removes given number of permits
397       */
398 <    public void testDrainPermits() {
399 <        Semaphore s = new Semaphore(0, false);
398 >    public void testDrainPermits()      { testDrainPermits(false); }
399 >    public void testDrainPermits_fair() { testDrainPermits(true); }
400 >    public void testDrainPermits(boolean fair) {
401 >        Semaphore s = new Semaphore(0, fair);
402          assertEquals(0, s.availablePermits());
403          assertEquals(0, s.drainPermits());
404          s.release(10);
# Line 354 | Line 409 | public class SemaphoreTest extends JSR16
409      }
410  
411      /**
412 <     * reducePermits reduces number of permits
412 >     * release(-N) throws IllegalArgumentException
413       */
414 <    public void testReducePermits() {
415 <        PublicSemaphore s = new PublicSemaphore(10, false);
416 <        assertEquals(10, s.availablePermits());
417 <        s.reducePermits(1);
418 <        assertEquals(9, s.availablePermits());
419 <        s.reducePermits(10);
420 <        assertEquals(-1, s.availablePermits());
414 >    public void testReleaseIAE()      { testReleaseIAE(false); }
415 >    public void testReleaseIAE_fair() { testReleaseIAE(true); }
416 >    public void testReleaseIAE(boolean fair) {
417 >        Semaphore s = new Semaphore(10, fair);
418 >        try {
419 >            s.release(-1);
420 >            shouldThrow();
421 >        } catch (IllegalArgumentException success) {}
422      }
423  
424      /**
425 <     * a deserialized serialized semaphore has same number of permits
426 <     */
427 <    public void testSerialization() throws Exception {
428 <        Semaphore l = new Semaphore(3, false);
429 <        l.acquire();
430 <        l.release();
431 <        ByteArrayOutputStream bout = new ByteArrayOutputStream(10000);
432 <        ObjectOutputStream out = new ObjectOutputStream(new BufferedOutputStream(bout));
433 <        out.writeObject(l);
434 <        out.close();
379 <
380 <        ByteArrayInputStream bin = new ByteArrayInputStream(bout.toByteArray());
381 <        ObjectInputStream in = new ObjectInputStream(new BufferedInputStream(bin));
382 <        Semaphore r = (Semaphore) in.readObject();
383 <        assertEquals(3, r.availablePermits());
384 <        assertFalse(r.isFair());
385 <        r.acquire();
386 <        r.release();
425 >     * reducePermits(-N) throws IllegalArgumentException
426 >     */
427 >    public void testReducePermitsIAE()      { testReducePermitsIAE(false); }
428 >    public void testReducePermitsIAE_fair() { testReducePermitsIAE(true); }
429 >    public void testReducePermitsIAE(boolean fair) {
430 >        PublicSemaphore s = new PublicSemaphore(10, fair);
431 >        try {
432 >            s.reducePermits(-1);
433 >            shouldThrow();
434 >        } catch (IllegalArgumentException success) {}
435      }
436  
389
437      /**
438 <     * Zero, negative, and positive initial values are allowed in constructor
438 >     * reducePermits reduces number of permits
439       */
440 <    public void testConstructor_fair() {
441 <        Semaphore s0 = new Semaphore(0, true);
442 <        assertEquals(0, s0.availablePermits());
443 <        assertTrue(s0.isFair());
444 <        Semaphore s1 = new Semaphore(-1, true);
445 <        assertEquals(-1, s1.availablePermits());
446 <        Semaphore s2 = new Semaphore(-1, true);
447 <        assertEquals(-1, s2.availablePermits());
440 >    public void testReducePermits()      { testReducePermits(false); }
441 >    public void testReducePermits_fair() { testReducePermits(true); }
442 >    public void testReducePermits(boolean fair) {
443 >        PublicSemaphore s = new PublicSemaphore(10, fair);
444 >        assertEquals(10, s.availablePermits());
445 >        s.reducePermits(0);
446 >        assertEquals(10, s.availablePermits());
447 >        s.reducePermits(1);
448 >        assertEquals(9, s.availablePermits());
449 >        s.reducePermits(10);
450 >        assertEquals(-1, s.availablePermits());
451 >        s.reducePermits(10);
452 >        assertEquals(-11, s.availablePermits());
453 >        s.reducePermits(0);
454 >        assertEquals(-11, s.availablePermits());
455      }
456  
457      /**
458 <     * tryAcquire succeeds when sufficient permits, else fails
459 <     */
460 <    public void testTryAcquireInSameThread_fair() {
461 <        Semaphore s = new Semaphore(2, true);
462 <        assertEquals(2, s.availablePermits());
463 <        assertTrue(s.tryAcquire());
464 <        assertTrue(s.tryAcquire());
465 <        assertEquals(0, s.availablePermits());
466 <        assertFalse(s.tryAcquire());
458 >     * a reserialized semaphore has same number of permits and
459 >     * fairness, but no queued threads
460 >     */
461 >    public void testSerialization()      { testSerialization(false); }
462 >    public void testSerialization_fair() { testSerialization(true); }
463 >    public void testSerialization(boolean fair) {
464 >        try {
465 >            Semaphore s = new Semaphore(3, fair);
466 >            s.acquire();
467 >            s.acquire();
468 >            s.release();
469 >
470 >            Semaphore clone = serialClone(s);
471 >            assertEquals(fair, s.isFair());
472 >            assertEquals(fair, clone.isFair());
473 >            assertEquals(2, s.availablePermits());
474 >            assertEquals(2, clone.availablePermits());
475 >            clone.acquire();
476 >            clone.acquire();
477 >            clone.release();
478 >            assertEquals(2, s.availablePermits());
479 >            assertEquals(1, clone.availablePermits());
480 >            assertFalse(s.hasQueuedThreads());
481 >            assertFalse(clone.hasQueuedThreads());
482 >        } catch (InterruptedException e) { threadUnexpectedException(e); }
483 >
484 >        {
485 >            PublicSemaphore s = new PublicSemaphore(0, fair);
486 >            Thread t = newStartedThread(new InterruptibleLockRunnable(s));
487 >            // waitForQueuedThreads(s); // suffers from "flicker", so ...
488 >            waitForQueuedThread(s, t);  // ... we use this instead
489 >            PublicSemaphore clone = serialClone(s);
490 >            assertEquals(fair, s.isFair());
491 >            assertEquals(fair, clone.isFair());
492 >            assertEquals(0, s.availablePermits());
493 >            assertEquals(0, clone.availablePermits());
494 >            assertTrue(s.hasQueuedThreads());
495 >            assertFalse(clone.hasQueuedThreads());
496 >            s.release();
497 >            awaitTermination(t);
498 >            assertFalse(s.hasQueuedThreads());
499 >            assertFalse(clone.hasQueuedThreads());
500 >        }
501      }
502  
503      /**
504       * tryAcquire(n) succeeds when sufficient permits, else fails
505       */
506 <    public void testTryAcquireNInSameThread_fair() {
507 <        Semaphore s = new Semaphore(2, true);
506 >    public void testTryAcquireNInSameThread()      { testTryAcquireNInSameThread(false); }
507 >    public void testTryAcquireNInSameThread_fair() { testTryAcquireNInSameThread(true); }
508 >    public void testTryAcquireNInSameThread(boolean fair) {
509 >        Semaphore s = new Semaphore(2, fair);
510 >        assertEquals(2, s.availablePermits());
511 >        assertFalse(s.tryAcquire(3));
512          assertEquals(2, s.availablePermits());
513          assertTrue(s.tryAcquire(2));
514          assertEquals(0, s.availablePermits());
515 <        assertFalse(s.tryAcquire());
516 <    }
517 <
426 <    /**
427 <     * Acquire and release of semaphore succeed if initially available
428 <     */
429 <    public void testAcquireReleaseInSameThread_fair()
430 <        throws InterruptedException {
431 <        Semaphore s = new Semaphore(1, true);
432 <        s.acquire();
433 <        s.release();
434 <        s.acquire();
435 <        s.release();
436 <        s.acquire();
437 <        s.release();
438 <        s.acquire();
439 <        s.release();
440 <        s.acquire();
441 <        s.release();
442 <        assertEquals(1, s.availablePermits());
443 <    }
444 <
445 <    /**
446 <     * Acquire(n) and release(n) of semaphore succeed if initially available
447 <     */
448 <    public void testAcquireReleaseNInSameThread_fair()
449 <        throws InterruptedException {
450 <        Semaphore s = new Semaphore(1, true);
451 <        s.release(1);
452 <        s.acquire(1);
453 <        s.release(2);
454 <        s.acquire(2);
455 <        s.release(3);
456 <        s.acquire(3);
457 <        s.release(4);
458 <        s.acquire(4);
459 <        s.release(5);
460 <        s.acquire(5);
461 <        assertEquals(1, s.availablePermits());
462 <    }
463 <
464 <    /**
465 <     * Acquire(n) and release(n) of semaphore succeed if initially available
466 <     */
467 <    public void testAcquireUninterruptiblyReleaseNInSameThread_fair() {
468 <        Semaphore s = new Semaphore(1, true);
469 <        s.release(1);
470 <        s.acquireUninterruptibly(1);
471 <        s.release(2);
472 <        s.acquireUninterruptibly(2);
473 <        s.release(3);
474 <        s.acquireUninterruptibly(3);
475 <        s.release(4);
476 <        s.acquireUninterruptibly(4);
477 <        s.release(5);
478 <        s.acquireUninterruptibly(5);
479 <        assertEquals(1, s.availablePermits());
480 <    }
481 <
482 <    /**
483 <     * release(n) in one thread enables timed acquire(n) in another thread
484 <     */
485 <    public void testTimedAcquireReleaseNInSameThread_fair()
486 <        throws InterruptedException {
487 <        Semaphore s = new Semaphore(1, true);
488 <        s.release(1);
489 <        assertTrue(s.tryAcquire(1, SHORT_DELAY_MS, MILLISECONDS));
490 <        s.release(2);
491 <        assertTrue(s.tryAcquire(2, SHORT_DELAY_MS, MILLISECONDS));
492 <        s.release(3);
493 <        assertTrue(s.tryAcquire(3, SHORT_DELAY_MS, MILLISECONDS));
494 <        s.release(4);
495 <        assertTrue(s.tryAcquire(4, SHORT_DELAY_MS, MILLISECONDS));
496 <        s.release(5);
497 <        assertTrue(s.tryAcquire(5, SHORT_DELAY_MS, MILLISECONDS));
498 <        assertEquals(1, s.availablePermits());
499 <    }
500 <
501 <    /**
502 <     * release in one thread enables timed acquire in another thread
503 <     */
504 <    public void testTimedAcquireReleaseInSameThread_fair()
505 <        throws InterruptedException {
506 <        Semaphore s = new Semaphore(1, true);
507 <        assertTrue(s.tryAcquire(SHORT_DELAY_MS, MILLISECONDS));
508 <        s.release();
509 <        assertTrue(s.tryAcquire(SHORT_DELAY_MS, MILLISECONDS));
510 <        s.release();
511 <        assertTrue(s.tryAcquire(SHORT_DELAY_MS, MILLISECONDS));
512 <        s.release();
513 <        assertTrue(s.tryAcquire(SHORT_DELAY_MS, MILLISECONDS));
514 <        s.release();
515 <        assertTrue(s.tryAcquire(SHORT_DELAY_MS, MILLISECONDS));
516 <        s.release();
517 <        assertEquals(1, s.availablePermits());
515 >        assertFalse(s.tryAcquire(1));
516 >        assertFalse(s.tryAcquire(2));
517 >        assertEquals(0, s.availablePermits());
518      }
519  
520      /**
521 <     * A release in one thread enables an acquire in another thread
521 >     * acquire succeeds if permits available
522       */
523 <    public void testAcquireReleaseInDifferentThreads_fair()
524 <        throws InterruptedException {
525 <        final Semaphore s = new Semaphore(0, true);
526 <        Thread t = new Thread(new CheckedRunnable() {
527 <            public void realRun() throws InterruptedException {
528 <                s.acquire();
529 <                s.acquire();
530 <                s.acquire();
531 <                s.acquire();
532 <            }});
533 <
534 <        t.start();
535 <        Thread.sleep(SHORT_DELAY_MS);
536 <        s.release();
537 <        s.release();
538 <        s.release();
539 <        s.release();
540 <        s.release();
541 <        s.release();
542 <        t.join();
543 <        assertEquals(2, s.availablePermits());
523 >    public void testReleaseAcquireSameThread_acquire()       { testReleaseAcquireSameThread(false, AcquireMethod.acquire); }
524 >    public void testReleaseAcquireSameThread_acquire_fair()  { testReleaseAcquireSameThread(true, AcquireMethod.acquire); }
525 >    public void testReleaseAcquireSameThread_acquireN()      { testReleaseAcquireSameThread(false, AcquireMethod.acquireN); }
526 >    public void testReleaseAcquireSameThread_acquireN_fair() { testReleaseAcquireSameThread(true, AcquireMethod.acquireN); }
527 >    public void testReleaseAcquireSameThread_acquireUninterruptibly()       { testReleaseAcquireSameThread(false, AcquireMethod.acquireUninterruptibly); }
528 >    public void testReleaseAcquireSameThread_acquireUninterruptibly_fair()  { testReleaseAcquireSameThread(true, AcquireMethod.acquireUninterruptibly); }
529 >    public void testReleaseAcquireSameThread_acquireUninterruptiblyN()      { testReleaseAcquireSameThread(false, AcquireMethod.acquireUninterruptibly); }
530 >    public void testReleaseAcquireSameThread_acquireUninterruptiblyN_fair() { testReleaseAcquireSameThread(true, AcquireMethod.acquireUninterruptibly); }
531 >    public void testReleaseAcquireSameThread_tryAcquire()       { testReleaseAcquireSameThread(false, AcquireMethod.tryAcquire); }
532 >    public void testReleaseAcquireSameThread_tryAcquire_fair()  { testReleaseAcquireSameThread(true, AcquireMethod.tryAcquire); }
533 >    public void testReleaseAcquireSameThread_tryAcquireN()      { testReleaseAcquireSameThread(false, AcquireMethod.tryAcquireN); }
534 >    public void testReleaseAcquireSameThread_tryAcquireN_fair() { testReleaseAcquireSameThread(true, AcquireMethod.tryAcquireN); }
535 >    public void testReleaseAcquireSameThread_tryAcquireTimed()       { testReleaseAcquireSameThread(false, AcquireMethod.tryAcquireTimed); }
536 >    public void testReleaseAcquireSameThread_tryAcquireTimed_fair()  { testReleaseAcquireSameThread(true, AcquireMethod.tryAcquireTimed); }
537 >    public void testReleaseAcquireSameThread_tryAcquireTimedN()      { testReleaseAcquireSameThread(false, AcquireMethod.tryAcquireTimedN); }
538 >    public void testReleaseAcquireSameThread_tryAcquireTimedN_fair() { testReleaseAcquireSameThread(true, AcquireMethod.tryAcquireTimedN); }
539 >    public void testReleaseAcquireSameThread(boolean fair,
540 >                                             final AcquireMethod acquirer) {
541 >        Semaphore s = new Semaphore(1, fair);
542 >        for (int i = 1; i < 6; i++) {
543 >            s.release(i);
544 >            assertEquals(1 + i, s.availablePermits());
545 >            try {
546 >                acquirer.acquire(s, i);
547 >            } catch (InterruptedException e) { threadUnexpectedException(e); }
548 >            assertEquals(1, s.availablePermits());
549 >        }
550      }
551  
552      /**
553 <     * release(n) in one thread enables acquire(n) in another thread
553 >     * release in one thread enables acquire in another thread
554       */
555 <    public void testAcquireReleaseNInDifferentThreads_fair()
556 <        throws InterruptedException {
557 <        final Semaphore s = new Semaphore(0, true);
558 <        Thread t = new Thread(new CheckedRunnable() {
559 <            public void realRun() throws InterruptedException {
560 <                s.acquire();
561 <                s.release(2);
562 <                s.acquire();
563 <            }});
564 <
565 <        t.start();
566 <        Thread.sleep(SHORT_DELAY_MS);
567 <        s.release(2);
568 <        s.acquire(2);
569 <        s.release(1);
570 <        t.join();
555 >    public void testReleaseAcquireDifferentThreads_acquire()       { testReleaseAcquireDifferentThreads(false, AcquireMethod.acquire); }
556 >    public void testReleaseAcquireDifferentThreads_acquire_fair()  { testReleaseAcquireDifferentThreads(true, AcquireMethod.acquire); }
557 >    public void testReleaseAcquireDifferentThreads_acquireN()      { testReleaseAcquireDifferentThreads(false, AcquireMethod.acquireN); }
558 >    public void testReleaseAcquireDifferentThreads_acquireN_fair() { testReleaseAcquireDifferentThreads(true, AcquireMethod.acquireN); }
559 >    public void testReleaseAcquireDifferentThreads_acquireUninterruptibly()       { testReleaseAcquireDifferentThreads(false, AcquireMethod.acquireUninterruptibly); }
560 >    public void testReleaseAcquireDifferentThreads_acquireUninterruptibly_fair()  { testReleaseAcquireDifferentThreads(true, AcquireMethod.acquireUninterruptibly); }
561 >    public void testReleaseAcquireDifferentThreads_acquireUninterruptiblyN()      { testReleaseAcquireDifferentThreads(false, AcquireMethod.acquireUninterruptibly); }
562 >    public void testReleaseAcquireDifferentThreads_acquireUninterruptiblyN_fair() { testReleaseAcquireDifferentThreads(true, AcquireMethod.acquireUninterruptibly); }
563 >    public void testReleaseAcquireDifferentThreads_tryAcquireTimed()       { testReleaseAcquireDifferentThreads(false, AcquireMethod.tryAcquireTimed); }
564 >    public void testReleaseAcquireDifferentThreads_tryAcquireTimed_fair()  { testReleaseAcquireDifferentThreads(true, AcquireMethod.tryAcquireTimed); }
565 >    public void testReleaseAcquireDifferentThreads_tryAcquireTimedN()      { testReleaseAcquireDifferentThreads(false, AcquireMethod.tryAcquireTimedN); }
566 >    public void testReleaseAcquireDifferentThreads_tryAcquireTimedN_fair() { testReleaseAcquireDifferentThreads(true, AcquireMethod.tryAcquireTimedN); }
567 >    public void testReleaseAcquireDifferentThreads(boolean fair,
568 >                                                   final AcquireMethod acquirer) {
569 >        final Semaphore s = new Semaphore(0, fair);
570 >        final int rounds = 4;
571 >        long startTime = System.nanoTime();
572 >        Thread t = newStartedThread(new CheckedRunnable() {
573 >            public void realRun() throws InterruptedException {
574 >                for (int i = 0; i < rounds; i++) {
575 >                    assertFalse(s.hasQueuedThreads());
576 >                    if (i % 2 == 0)
577 >                        acquirer.acquire(s);
578 >                    else
579 >                        acquirer.acquire(s, 3);
580 >                }}});
581 >
582 >        for (int i = 0; i < rounds; i++) {
583 >            while (! (s.availablePermits() == 0 && s.hasQueuedThreads()))
584 >                Thread.yield();
585 >            assertTrue(t.isAlive());
586 >            if (i % 2 == 0)
587 >                s.release();
588 >            else
589 >                s.release(3);
590 >        }
591 >        awaitTermination(t);
592 >        assertEquals(0, s.availablePermits());
593 >        assertTrue(millisElapsedSince(startTime) < LONG_DELAY_MS);
594      }
595  
596      /**
597 <     * release(n) in one thread enables acquire(n) in another thread
597 >     * fair locks are strictly FIFO
598       */
599 <    public void testAcquireReleaseNInDifferentThreads_fair2()
600 <        throws InterruptedException {
601 <        final Semaphore s = new Semaphore(0, true);
602 <        Thread t = new Thread(new CheckedRunnable() {
599 >    public void testFairLocksFifo() {
600 >        final PublicSemaphore s = new PublicSemaphore(1, true);
601 >        final CountDownLatch pleaseRelease = new CountDownLatch(1);
602 >        Thread t1 = newStartedThread(new CheckedRunnable() {
603              public void realRun() throws InterruptedException {
604 <                s.acquire(2);
605 <                s.acquire(2);
577 <                s.release(4);
604 >                // Will block; permits are available, but not three
605 >                s.acquire(3);
606              }});
607  
608 <        t.start();
581 <        Thread.sleep(SHORT_DELAY_MS);
582 <        s.release(6);
583 <        s.acquire(2);
584 <        s.acquire(2);
585 <        s.release(2);
586 <        t.join();
587 <    }
608 >        waitForQueuedThread(s, t1);
609  
610 <
590 <    /**
591 <     * release in one thread enables timed acquire in another thread
592 <     */
593 <    public void testTimedAcquireReleaseInDifferentThreads_fair()
594 <        throws InterruptedException {
595 <        final Semaphore s = new Semaphore(1, true);
596 <        Thread t = new Thread(new CheckedRunnable() {
610 >        Thread t2 = newStartedThread(new CheckedRunnable() {
611              public void realRun() throws InterruptedException {
612 <                threadAssertTrue(s.tryAcquire(SHORT_DELAY_MS, MILLISECONDS));
613 <                threadAssertTrue(s.tryAcquire(SHORT_DELAY_MS, MILLISECONDS));
614 <                threadAssertTrue(s.tryAcquire(SHORT_DELAY_MS, MILLISECONDS));
601 <                threadAssertTrue(s.tryAcquire(SHORT_DELAY_MS, MILLISECONDS));
602 <                threadAssertTrue(s.tryAcquire(SHORT_DELAY_MS, MILLISECONDS));
603 <            }});
604 <
605 <        t.start();
606 <        s.release();
607 <        s.release();
608 <        s.release();
609 <        s.release();
610 <        s.release();
611 <        t.join();
612 <    }
612 >                // Will fail, even though 1 permit is available
613 >                assertFalse(s.tryAcquire(0L, MILLISECONDS));
614 >                assertFalse(s.tryAcquire(1, 0L, MILLISECONDS));
615  
616 <    /**
617 <     * release(n) in one thread enables timed acquire(n) in another thread
616 <     */
617 <    public void testTimedAcquireReleaseNInDifferentThreads_fair()
618 <        throws InterruptedException {
619 <        final Semaphore s = new Semaphore(2, true);
620 <        Thread t = new Thread(new CheckedRunnable() {
621 <            public void realRun() throws InterruptedException {
622 <                threadAssertTrue(s.tryAcquire(2, SHORT_DELAY_MS, MILLISECONDS));
616 >                // untimed tryAcquire will barge and succeed
617 >                assertTrue(s.tryAcquire());
618                  s.release(2);
619 <                threadAssertTrue(s.tryAcquire(2, SHORT_DELAY_MS, MILLISECONDS));
620 <                s.release(2);
626 <            }});
627 <
628 <        t.start();
629 <        assertTrue(s.tryAcquire(2, SHORT_DELAY_MS, MILLISECONDS));
630 <        s.release(2);
631 <        assertTrue(s.tryAcquire(2, SHORT_DELAY_MS, MILLISECONDS));
632 <        s.release(2);
633 <        t.join();
634 <    }
619 >                assertTrue(s.tryAcquire(2));
620 >                s.release();
621  
622 <    /**
623 <     * A waiting acquire blocks interruptibly
638 <     */
639 <    public void testAcquire_InterruptedException_fair()
640 <        throws InterruptedException {
641 <        final Semaphore s = new Semaphore(0, true);
642 <        Thread t = new Thread(new CheckedInterruptedRunnable() {
643 <            public void realRun() throws InterruptedException {
622 >                pleaseRelease.countDown();
623 >                // Will queue up behind t1, even though 1 permit is available
624                  s.acquire();
625              }});
626  
627 <        t.start();
628 <        Thread.sleep(SHORT_DELAY_MS);
629 <        t.interrupt();
630 <        t.join();
631 <    }
632 <
633 <    /**
654 <     * A waiting acquire(n) blocks interruptibly
655 <     */
656 <    public void testAcquireN_InterruptedException_fair()
657 <        throws InterruptedException {
658 <        final Semaphore s = new Semaphore(2, true);
659 <        Thread t = new Thread(new CheckedInterruptedRunnable() {
660 <            public void realRun() throws InterruptedException {
661 <                s.acquire(3);
662 <            }});
663 <
664 <        t.start();
665 <        Thread.sleep(SHORT_DELAY_MS);
666 <        t.interrupt();
667 <        t.join();
668 <    }
669 <
670 <    /**
671 <     *  A waiting tryAcquire blocks interruptibly
672 <     */
673 <    public void testTryAcquire_InterruptedException_fair()
674 <        throws InterruptedException {
675 <        final Semaphore s = new Semaphore(0, true);
676 <        Thread t = new Thread(new CheckedInterruptedRunnable() {
677 <            public void realRun() throws InterruptedException {
678 <                s.tryAcquire(MEDIUM_DELAY_MS, MILLISECONDS);
679 <            }});
680 <
681 <        t.start();
682 <        Thread.sleep(SHORT_DELAY_MS);
683 <        t.interrupt();
684 <        t.join();
685 <    }
686 <
687 <    /**
688 <     *  A waiting tryAcquire(n) blocks interruptibly
689 <     */
690 <    public void testTryAcquireN_InterruptedException_fair()
691 <        throws InterruptedException {
692 <        final Semaphore s = new Semaphore(1, true);
693 <        Thread t = new Thread(new CheckedInterruptedRunnable() {
694 <            public void realRun() throws InterruptedException {
695 <                s.tryAcquire(4, MEDIUM_DELAY_MS, MILLISECONDS);
696 <            }});
697 <
698 <        t.start();
699 <        Thread.sleep(SHORT_DELAY_MS);
700 <        t.interrupt();
701 <        t.join();
702 <    }
703 <
704 <    /**
705 <     * getQueueLength reports number of waiting threads
706 <     */
707 <    public void testGetQueueLength_fair() throws InterruptedException {
708 <        final Semaphore lock = new Semaphore(1, true);
709 <        Thread t1 = new Thread(new InterruptedLockRunnable(lock));
710 <        Thread t2 = new Thread(new InterruptibleLockRunnable(lock));
711 <        assertEquals(0, lock.getQueueLength());
712 <        lock.acquireUninterruptibly();
713 <        t1.start();
714 <        Thread.sleep(SHORT_DELAY_MS);
715 <        assertEquals(1, lock.getQueueLength());
716 <        t2.start();
717 <        Thread.sleep(SHORT_DELAY_MS);
718 <        assertEquals(2, lock.getQueueLength());
719 <        t1.interrupt();
720 <        Thread.sleep(SHORT_DELAY_MS);
721 <        assertEquals(1, lock.getQueueLength());
722 <        lock.release();
723 <        Thread.sleep(SHORT_DELAY_MS);
724 <        assertEquals(0, lock.getQueueLength());
725 <        t1.join();
726 <        t2.join();
727 <    }
728 <
729 <
730 <    /**
731 <     * a deserialized serialized semaphore has same number of permits
732 <     */
733 <    public void testSerialization_fair() throws Exception {
734 <        Semaphore l = new Semaphore(3, true);
735 <
736 <        l.acquire();
737 <        l.release();
738 <        ByteArrayOutputStream bout = new ByteArrayOutputStream(10000);
739 <        ObjectOutputStream out = new ObjectOutputStream(new BufferedOutputStream(bout));
740 <        out.writeObject(l);
741 <        out.close();
742 <
743 <        ByteArrayInputStream bin = new ByteArrayInputStream(bout.toByteArray());
744 <        ObjectInputStream in = new ObjectInputStream(new BufferedInputStream(bin));
745 <        Semaphore r = (Semaphore) in.readObject();
746 <        assertEquals(3, r.availablePermits());
747 <        assertTrue(r.isFair());
748 <        r.acquire();
749 <        r.release();
627 >        await(pleaseRelease);
628 >        waitForQueuedThread(s, t2);
629 >        s.release(2);
630 >        awaitTermination(t1);
631 >        assertTrue(t2.isAlive());
632 >        s.release();
633 >        awaitTermination(t2);
634      }
635  
636      /**
637       * toString indicates current number of permits
638       */
639 <    public void testToString() {
640 <        Semaphore s = new Semaphore(0);
641 <        String us = s.toString();
642 <        assertTrue(us.indexOf("Permits = 0") >= 0);
639 >    public void testToString()      { testToString(false); }
640 >    public void testToString_fair() { testToString(true); }
641 >    public void testToString(boolean fair) {
642 >        PublicSemaphore s = new PublicSemaphore(0, fair);
643 >        assertTrue(s.toString().contains("Permits = 0"));
644          s.release();
645 <        String s1 = s.toString();
646 <        assertTrue(s1.indexOf("Permits = 1") >= 0);
647 <        s.release();
648 <        String s2 = s.toString();
649 <        assertTrue(s2.indexOf("Permits = 2") >= 0);
645 >        assertTrue(s.toString().contains("Permits = 1"));
646 >        s.release(2);
647 >        assertTrue(s.toString().contains("Permits = 3"));
648 >        s.reducePermits(5);
649 >        assertTrue(s.toString().contains("Permits = -2"));
650      }
651  
652   }

Diff Legend

Removed lines
+ Added lines
< Changed lines
> Changed lines