Skip to content

Commit 81773b8

Browse files
committed
For #42:
As per PR review: * Added missing javadoc tags * Renamed an instance variable to avoid name clashing
1 parent 432908c commit 81773b8

File tree

2 files changed

+5
-3
lines changed

2 files changed

+5
-3
lines changed

src/test/java/com/amihaiemil/docker/mock/AssertRequestTestCase.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -39,6 +39,8 @@
3939
* Tests for {@link AssertRequest}.
4040
*
4141
* @author George Aristy (george.aristy@gmail.com)
42+
* @version $Id$
43+
* @since 0.0.1
4244
*/
4345
public final class AssertRequestTestCase {
4446
/**

src/test/java/com/amihaiemil/docker/mock/Condition.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@ public final class Condition {
4444
/**
4545
* The test that the http request must satisfy.
4646
*/
47-
private final Predicate<HttpRequest> test;
47+
private final Predicate<HttpRequest> predicate;
4848

4949
/**
5050
* Ctor.
@@ -54,7 +54,7 @@ public final class Condition {
5454
*/
5555
public Condition(final String msg, final Predicate<HttpRequest> test) {
5656
this.msg = msg;
57-
this.test = test;
57+
this.predicate = test;
5858
}
5959

6060
/**
@@ -64,7 +64,7 @@ public Condition(final String msg, final Predicate<HttpRequest> test) {
6464
* @throws AssertionError if the request does not satisfy this condition
6565
*/
6666
public void test(final HttpRequest request) {
67-
if (!this.test.test(request)) {
67+
if (!this.predicate.test(request)) {
6868
Assert.fail(this.msg);
6969
}
7070
}

0 commit comments

Comments
 (0)