Skip to content

Commit fa1f5a9

Browse files
cardilcardil
committed
Testing exceptions contract methods.
1 parent 5bca6d9 commit fa1f5a9

10 files changed

Lines changed: 704 additions & 79 deletions

‎src/main/java/pl/wavesoftware/eid/exceptions/EidIllegalArgumentException.java‎

Lines changed: 15 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,9 @@
2121
import javax.annotation.Nullable;
2222

2323
/**
24-
* This exception is Eid version of Java's {@link IllegalArgumentException}.
24+
* This exception is Eid version of Java's {@link IllegalArgumentException} that
25+
* holds an {@link Eid} object. It can be used to process it in application
26+
* central error handler.
2527
* <p>
2628
* <strong>Caution!</strong> This class shouldn't be used in any public API or
2729
* library. It is designed to be used for in-house development of end user
@@ -94,7 +96,9 @@ public EidIllegalArgumentException(EidMessage message) {
9496
* {@link #getCause()} method). (A <tt>null</tt> value is
9597
* permitted, and indicates that the cause is nonexistent or
9698
*/
97-
public EidIllegalArgumentException(CharSequence eid, String message, @Nullable Throwable cause) {
99+
public EidIllegalArgumentException(
100+
CharSequence eid, String message, @Nullable Throwable cause
101+
) {
98102
super(eid, message, cause);
99103
}
100104

@@ -113,7 +117,9 @@ public EidIllegalArgumentException(CharSequence eid, String message, @Nullable T
113117
* {@link #getCause()} method). (A <tt>null</tt> value is
114118
* permitted, and indicates that the cause is nonexistent or
115119
*/
116-
public EidIllegalArgumentException(EidMessage message, @Nullable Throwable cause) {
120+
public EidIllegalArgumentException(
121+
EidMessage message, @Nullable Throwable cause
122+
) {
117123
super(message, cause);
118124
}
119125

@@ -133,7 +139,9 @@ public EidIllegalArgumentException(EidMessage message, @Nullable Throwable cause
133139
* {@link #getCause()} method). (A <tt>null</tt> value is
134140
* permitted, and indicates that the cause is nonexistent or
135141
*/
136-
public EidIllegalArgumentException(CharSequence eid, @Nullable Throwable cause) {
142+
public EidIllegalArgumentException(
143+
CharSequence eid, @Nullable Throwable cause
144+
) {
137145
super(eid, cause);
138146
}
139147

@@ -181,7 +189,9 @@ public EidIllegalArgumentException(Eid id, String message) {
181189
* {@link #getCause()} method). (A <tt>null</tt> value is
182190
* permitted, and indicates that the cause is nonexistent or
183191
*/
184-
public EidIllegalArgumentException(Eid id, String message, @Nullable Throwable cause) {
192+
public EidIllegalArgumentException(
193+
Eid id, String message, @Nullable Throwable cause
194+
) {
185195
super(id, message, cause);
186196
}
187197

‎src/main/java/pl/wavesoftware/eid/exceptions/EidIllegalStateException.java‎

Lines changed: 7 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,9 @@
2121
import javax.annotation.Nullable;
2222

2323
/**
24-
* This exception is Eid version of Java's {@link IllegalStateException}.
24+
* This exception is Eid version of Java's {@link IllegalStateException} that
25+
* holds an {@link Eid} object. It can be used to process it in application
26+
* central error handler.
2527
* <p>
2628
* <strong>Caution!</strong> This class shouldn't be used in any public API or
2729
* library. It is designed to be used for in-house development of end user
@@ -95,9 +97,7 @@ public EidIllegalStateException(EidMessage message) {
9597
* permitted, and indicates that the cause is nonexistent or
9698
*/
9799
public EidIllegalStateException(
98-
CharSequence eid,
99-
String message,
100-
@Nullable Throwable cause
100+
CharSequence eid, String message, @Nullable Throwable cause
101101
) {
102102
super(eid, message, cause);
103103
}
@@ -118,8 +118,7 @@ public EidIllegalStateException(
118118
* permitted, and indicates that the cause is nonexistent or
119119
*/
120120
public EidIllegalStateException(
121-
EidMessage message,
122-
@Nullable Throwable cause
121+
EidMessage message, @Nullable Throwable cause
123122
) {
124123
super(message, cause);
125124
}
@@ -141,8 +140,7 @@ public EidIllegalStateException(
141140
* permitted, and indicates that the cause is nonexistent or
142141
*/
143142
public EidIllegalStateException(
144-
CharSequence eid,
145-
@Nullable Throwable cause
143+
CharSequence eid, @Nullable Throwable cause
146144
) {
147145
super(eid, cause);
148146
}
@@ -192,9 +190,7 @@ public EidIllegalStateException(Eid id, String message) {
192190
* permitted, and indicates that the cause is nonexistent or
193191
*/
194192
public EidIllegalStateException(
195-
Eid id,
196-
String message,
197-
@Nullable Throwable cause
193+
Eid id, String message, @Nullable Throwable cause
198194
) {
199195
super(id, message, cause);
200196
}

‎src/main/java/pl/wavesoftware/eid/exceptions/EidIndexOutOfBoundsException.java‎

Lines changed: 8 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,9 @@
2121
import javax.annotation.Nullable;
2222

2323
/**
24-
* This exception is Eid version of Java's {@link IndexOutOfBoundsException}.
24+
* This exception is Eid version of Java's {@link IndexOutOfBoundsException} that
25+
* holds an {@link Eid} object. It can be used to process it in application
26+
* central error handler.
2527
* <p>
2628
* <strong>Caution!</strong> This class shouldn't be used in any public API or
2729
* library. It is designed to be used for in-house development of end user
@@ -95,9 +97,7 @@ public EidIndexOutOfBoundsException(EidMessage message) {
9597
* permitted, and indicates that the cause is nonexistent or
9698
*/
9799
public EidIndexOutOfBoundsException(
98-
CharSequence eid,
99-
String message,
100-
@Nullable Throwable cause
100+
CharSequence eid, String message, @Nullable Throwable cause
101101
) {
102102
super(eid, message, cause);
103103
}
@@ -118,8 +118,7 @@ public EidIndexOutOfBoundsException(
118118
* permitted, and indicates that the cause is nonexistent or
119119
*/
120120
public EidIndexOutOfBoundsException(
121-
EidMessage message,
122-
@Nullable Throwable cause
121+
EidMessage message, @Nullable Throwable cause
123122
) {
124123
super(message, cause);
125124
}
@@ -141,8 +140,7 @@ public EidIndexOutOfBoundsException(
141140
* permitted, and indicates that the cause is nonexistent or
142141
*/
143142
public EidIndexOutOfBoundsException(
144-
CharSequence eid,
145-
@Nullable Throwable cause
143+
CharSequence eid, @Nullable Throwable cause
146144
) {
147145
super(eid, cause);
148146
}
@@ -192,9 +190,7 @@ public EidIndexOutOfBoundsException(Eid id, String message) {
192190
* permitted, and indicates that the cause is nonexistent or
193191
*/
194192
public EidIndexOutOfBoundsException(
195-
Eid id,
196-
String message,
197-
@Nullable Throwable cause
193+
Eid id, String message, @Nullable Throwable cause
198194
) {
199195
super(id, message, cause);
200196
}
@@ -216,8 +212,7 @@ public EidIndexOutOfBoundsException(
216212
* permitted, and indicates that the cause is nonexistent or
217213
*/
218214
public EidIndexOutOfBoundsException(
219-
Eid id,
220-
@Nullable Throwable cause
215+
Eid id, @Nullable Throwable cause
221216
) {
222217
super(id, cause);
223218
}

‎src/main/java/pl/wavesoftware/eid/exceptions/EidNullPointerException.java‎

Lines changed: 15 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,9 @@
2121
import javax.annotation.Nullable;
2222

2323
/**
24-
* This exception is Eid version of Java's {@link NullPointerException}.
24+
* This exception is Eid version of Java's {@link NullPointerException} that
25+
* holds an {@link Eid} object. It can be used to process it in application
26+
* central error handler.
2527
* <p>
2628
* <strong>Caution!</strong> This class shouldn't be used in any public API or
2729
* library. It is designed to be used for in-house development of end user
@@ -94,7 +96,9 @@ public EidNullPointerException(EidMessage message) {
9496
* {@link #getCause()} method). (A <tt>null</tt> value is
9597
* permitted, and indicates that the cause is nonexistent or
9698
*/
97-
public EidNullPointerException(CharSequence eid, String message, @Nullable Throwable cause) {
99+
public EidNullPointerException(
100+
CharSequence eid, String message, @Nullable Throwable cause
101+
) {
98102
super(eid, message, cause);
99103
}
100104

@@ -113,7 +117,9 @@ public EidNullPointerException(CharSequence eid, String message, @Nullable Throw
113117
* {@link #getCause()} method). (A <tt>null</tt> value is
114118
* permitted, and indicates that the cause is nonexistent or
115119
*/
116-
public EidNullPointerException(EidMessage message, @Nullable Throwable cause) {
120+
public EidNullPointerException(
121+
EidMessage message, @Nullable Throwable cause
122+
) {
117123
super(message, cause);
118124
}
119125

@@ -133,7 +139,9 @@ public EidNullPointerException(EidMessage message, @Nullable Throwable cause) {
133139
* {@link #getCause()} method). (A <tt>null</tt> value is
134140
* permitted, and indicates that the cause is nonexistent or
135141
*/
136-
public EidNullPointerException(CharSequence eid, @Nullable Throwable cause) {
142+
public EidNullPointerException(
143+
CharSequence eid, @Nullable Throwable cause
144+
) {
137145
super(eid, cause);
138146
}
139147

@@ -181,7 +189,9 @@ public EidNullPointerException(Eid id, String message) {
181189
* {@link #getCause()} method). (A <tt>null</tt> value is
182190
* permitted, and indicates that the cause is nonexistent or
183191
*/
184-
public EidNullPointerException(Eid id, String message, @Nullable Throwable cause) {
192+
public EidNullPointerException(
193+
Eid id, String message, @Nullable Throwable cause
194+
) {
185195
super(id, message, cause);
186196
}
187197

‎src/main/java/pl/wavesoftware/eid/exceptions/EidRuntimeException.java‎

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,7 @@
1919
import pl.wavesoftware.eid.EidContainer;
2020
import pl.wavesoftware.eid.EidMessage;
2121

22+
import javax.annotation.Nonnull;
2223
import javax.annotation.Nullable;
2324

2425
/**
@@ -110,7 +111,9 @@ public EidRuntimeException(EidMessage message) {
110111
* permitted, and indicates that the cause is nonexistent or
111112
* unknown.)
112113
*/
113-
public EidRuntimeException(CharSequence eid, String message, @Nullable Throwable cause) {
114+
public EidRuntimeException(
115+
CharSequence eid, String message, @Nullable Throwable cause
116+
) {
114117
this(new Eid(eid), message, cause);
115118
}
116119

@@ -131,7 +134,7 @@ public EidRuntimeException(CharSequence eid, String message, @Nullable Throwable
131134
* unknown.)
132135
*/
133136
public EidRuntimeException(EidMessage message, @Nullable Throwable cause) {
134-
this(message.getEid(), message.toString(), cause);
137+
this(message.getEid(), message.getFormattedMessage().toString(), cause);
135138
}
136139

137140
/**
@@ -241,11 +244,14 @@ private static String messageOf(Eid eid, @Nullable Throwable cause) {
241244
return eid.toString();
242245
}
243246

247+
@Nonnull
244248
private static String messageOf(Throwable cause) {
245-
return coalesce(
249+
String value = coalesce(
246250
coalesce(cause.getLocalizedMessage(), cause.getMessage()),
247251
cause.toString()
248252
);
253+
assert value != null : "20181214:234625";
254+
return value;
249255
}
250256

251257
@Nullable

‎src/test/java/pl/wavesoftware/eid/exceptions/A.java‎

Lines changed: 28 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,28 @@
1+
/*
2+
* Copyright (c) 2018 Wave Software
3+
*
4+
* Licensed under the Apache License, Version 2.0 (the "License");
5+
* you may not use this file except in compliance with the License.
6+
* You may obtain a copy of the License at
7+
*
8+
* http://www.apache.org/licenses/LICENSE-2.0
9+
*
10+
* Unless required by applicable law or agreed to in writing, software
11+
* distributed under the License is distributed on an "AS IS" BASIS,
12+
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
13+
* See the License for the specific language governing permissions and
14+
* limitations under the License.
15+
*/
16+
17+
package pl.wavesoftware.eid.exceptions;
18+
19+
/**
20+
* @author <a href="mailto:krzysztof.suszynski@wavesoftware.pl">Krzysztof Suszynski</a>
21+
* @since 2018-12-16
22+
*/
23+
class A {
24+
@Override
25+
public String toString() {
26+
return "a";
27+
}
28+
}

‎src/test/java/pl/wavesoftware/eid/exceptions/ConstantUniqueIdRule.java‎

Lines changed: 67 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,67 @@
1+
/*
2+
* Copyright (c) 2018 Wave Software
3+
*
4+
* Licensed under the Apache License, Version 2.0 (the "License");
5+
* you may not use this file except in compliance with the License.
6+
* You may obtain a copy of the License at
7+
*
8+
* http://www.apache.org/licenses/LICENSE-2.0
9+
*
10+
* Unless required by applicable law or agreed to in writing, software
11+
* distributed under the License is distributed on an "AS IS" BASIS,
12+
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
13+
* See the License for the specific language governing permissions and
14+
* limitations under the License.
15+
*/
16+
17+
package pl.wavesoftware.eid.exceptions;
18+
19+
import org.junit.rules.TestRule;
20+
import org.junit.runner.Description;
21+
import org.junit.runners.model.Statement;
22+
import pl.wavesoftware.eid.Eid;
23+
import pl.wavesoftware.eid.configuration.ConfigurationBuilder;
24+
import pl.wavesoftware.eid.configuration.Configurator;
25+
import pl.wavesoftware.eid.configuration.UniqueIdGenerator;
26+
27+
/**
28+
* @author <a href="mailto:krzysztof.suszynski@wavesoftware.pl">Krzysztof Suszynski</a>
29+
* @since 2.0.0
30+
*/
31+
final class ConstantUniqueIdRule implements TestRule {
32+
33+
private final String constantId;
34+
35+
ConstantUniqueIdRule(String constantId) {
36+
this.constantId = constantId;
37+
}
38+
39+
@Override
40+
public Statement apply(final Statement base, Description description) {
41+
return new Statement() {
42+
@Override
43+
public void evaluate() throws Throwable {
44+
Configurator configurator = configureConstantId();
45+
try {
46+
base.evaluate();
47+
} finally {
48+
Eid.getBinding().getConfigurationSystem().configure(configurator);
49+
}
50+
}
51+
};
52+
}
53+
54+
private Configurator configureConstantId() {
55+
return Eid.getBinding().getConfigurationSystem().configure(new Configurator() {
56+
@Override
57+
public void configure(ConfigurationBuilder configuration) {
58+
configuration.uniqueIdGenerator(new UniqueIdGenerator() {
59+
@Override
60+
public String generateUniqId() {
61+
return constantId;
62+
}
63+
});
64+
}
65+
});
66+
}
67+
}

0 commit comments

Comments
 (0)