Skip to content

Commit 91d020c

Browse files
committed
#1142 comments removed from tests
1 parent dddac7b commit 91d020c

File tree

169 files changed

+0
-1784
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

169 files changed

+0
-1784
lines changed

src/it/file-manager/src/test/java/org/takes/it/fm/AppITCase.java

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -42,11 +42,6 @@ public final class AppITCase {
4242
*/
4343
private static final String HOME = System.getProperty("takes.home");
4444

45-
/**
46-
* App can work.
47-
* @throws Exception If some problem inside
48-
* @checkstyle NonStaticMethodCheck (2 lines)
49-
*/
5045
@Test
5146
public void justWorks() throws Exception {
5247
Assertions.assertNotNull(AppITCase.HOME);

src/it/file-manager/src/test/java/org/takes/it/fm/AppTest.java

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -43,11 +43,6 @@
4343
*/
4444
public final class AppTest {
4545

46-
/**
47-
* App can work.
48-
* @param temp Temporal directory
49-
* @throws Exception If some problem inside
50-
*/
5146
@Test
5247
public void justWorks(@TempDir final Path temp) throws Exception {
5348
final File dir = temp.toFile();

src/test/java/org/takes/facets/auth/IdentityTest.java

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -33,9 +33,6 @@
3333
*/
3434
final class IdentityTest {
3535

36-
/**
37-
* Identity.ANONYMOUS can be equal to itself.
38-
*/
3936
@Test
4037
void equalsToItself() {
4138
MatcherAssert.assertThat(

src/test/java/org/takes/facets/auth/PsAllTest.java

Lines changed: 0 additions & 30 deletions
Original file line numberDiff line numberDiff line change
@@ -41,9 +41,6 @@
4141
*/
4242
final class PsAllTest {
4343

44-
/**
45-
* Fails if PsAll with 0 Passes is created.
46-
*/
4744
@Test
4845
void thereShouldBeAtLeastOnePass() {
4946
Assertions.assertThrows(
@@ -58,9 +55,6 @@ void thereShouldBeAtLeastOnePass() {
5855
);
5956
}
6057

61-
/**
62-
* Fails if index is less then 0.
63-
*/
6458
@Test
6559
void indexMustBeNonNegative() {
6660
Assertions.assertThrows(
@@ -75,9 +69,6 @@ void indexMustBeNonNegative() {
7569
);
7670
}
7771

78-
/**
79-
* Fails if index is greater or equal to the number of Passes to enter.
80-
*/
8172
@Test
8273
void indexMustBeSmallEnough() {
8374
Assertions.assertThrows(
@@ -95,10 +86,6 @@ void indexMustBeSmallEnough() {
9586
);
9687
}
9788

98-
/**
99-
* PsAll with a single Pass that can be entered should succeed.
100-
* @throws Exception If fails
101-
*/
10289
@Test
10390
void testOneSuccessfull() throws Exception {
10491
MatcherAssert.assertThat(
@@ -110,10 +97,6 @@ void testOneSuccessfull() throws Exception {
11097
);
11198
}
11299

113-
/**
114-
* Fail with only one failure test.
115-
* @throws Exception if exception
116-
*/
117100
@Test
118101
void testOneFail() throws Exception {
119102
MatcherAssert.assertThat(
@@ -125,11 +108,6 @@ void testOneFail() throws Exception {
125108
);
126109
}
127110

128-
/**
129-
* PsAll with multiple passes that all can be entered returns the
130-
* identity specified by an index.
131-
* @throws Exception If fails
132-
*/
133111
@Test
134112
void testSuccessfullIdx() throws Exception {
135113
final Pass resulting = new PsFixed(
@@ -150,10 +128,6 @@ void testSuccessfullIdx() throws Exception {
150128
);
151129
}
152130

153-
/**
154-
* Fail if one of Pass fails.
155-
* @throws Exception if exception
156-
*/
157131
@Test
158132
void testFail() throws Exception {
159133
MatcherAssert.assertThat(
@@ -170,10 +144,6 @@ void testFail() throws Exception {
170144
);
171145
}
172146

173-
/**
174-
* Exits.
175-
* @throws Exception If fails
176-
*/
177147
@Test
178148
void exits() throws Exception {
179149
final Response response = new RsEmpty();

src/test/java/org/takes/facets/auth/PsBasicDefaultTest.java

Lines changed: 0 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -34,9 +34,6 @@
3434
*/
3535
final class PsBasicDefaultTest {
3636

37-
/**
38-
* PsBasic.Default can accept a correct login/password pair.
39-
*/
4037
@Test
4138
void acceptsCorrectLoginPasswordPair() {
4239
MatcherAssert.assertThat(
@@ -56,10 +53,6 @@ void acceptsCorrectLoginPasswordPair() {
5653
);
5754
}
5855

59-
/**
60-
* PsBasic.Default can handle both <pre>%20</pre> and <pre>+</pre>
61-
* variants of encoding spaces in constructor parameters.
62-
*/
6356
@Test
6457
void supportsBothKindsOfSpace() {
6558
MatcherAssert.assertThat(
@@ -90,9 +83,6 @@ void supportsBothKindsOfSpace() {
9083
);
9184
}
9285

93-
/**
94-
* PsBasic.Default can be entered by a user with a space in his name.
95-
*/
9686
@Test
9787
void supportsUsersWithSpacesInTheirNames() {
9888
MatcherAssert.assertThat(
@@ -110,10 +100,6 @@ void supportsUsersWithSpacesInTheirNames() {
110100
);
111101
}
112102

113-
/**
114-
* PsBasic.Default can url-decode an urn from its parameter.
115-
* @throws Exception If fails
116-
*/
117103
@Test
118104
void supportsUrlencodedUrns() throws Exception {
119105
final String urn = "urn:a100%25:one-two+";
@@ -136,9 +122,6 @@ void supportsUrlencodedUrns() throws Exception {
136122
);
137123
}
138124

139-
/**
140-
* PsBasic.Default can reject incorrect password.
141-
*/
142125
@Test
143126
void rejectsIncorrectPassword() {
144127
MatcherAssert.assertThat(
@@ -154,9 +137,6 @@ void rejectsIncorrectPassword() {
154137
);
155138
}
156139

157-
/**
158-
* PsBasic.Default can reject a non-existing login.
159-
*/
160140
@Test
161141
void rejectsIncorrectLogin() {
162142
MatcherAssert.assertThat(

src/test/java/org/takes/facets/auth/PsBasicTest.java

Lines changed: 0 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -62,10 +62,6 @@ final class PsBasicTest {
6262
*/
6363
private static final String VALID_CODE = "?valid_code=%s";
6464

65-
/**
66-
* PsBasic can handle connection with valid credential.
67-
* @throws Exception if any error occurs
68-
*/
6965
@Test
7066
void handleConnectionWithValidCredential() throws Exception {
7167
final String user = "john";
@@ -91,11 +87,6 @@ void handleConnectionWithValidCredential() throws Exception {
9187
);
9288
}
9389

94-
/**
95-
* PsBasic can handle connection with valid credential when Entry is
96-
* a instance of Default.
97-
* @throws Exception if any error occurs
98-
*/
9990
@Test
10091
void handleConnectionWithValidCredentialDefaultEntry()
10192
throws Exception {
@@ -126,10 +117,6 @@ void handleConnectionWithValidCredentialDefaultEntry()
126117
);
127118
}
128119

129-
/**
130-
* PsBasic can handle connection with invalid credential.
131-
* @throws Exception If some problem inside
132-
*/
133120
@Test
134121
void handleConnectionWithInvalidCredential() throws Exception {
135122
RsForward forward = new RsForward();
@@ -163,10 +150,6 @@ void handleConnectionWithInvalidCredential() throws Exception {
163150
);
164151
}
165152

166-
/**
167-
* PsBasic can handle multiple headers with valid credential.
168-
* @throws Exception If some problem inside
169-
*/
170153
@Test
171154
void handleMultipleHeadersWithValidCredential() throws Exception {
172155
final String user = "bill";
@@ -197,9 +180,6 @@ void handleMultipleHeadersWithValidCredential() throws Exception {
197180
);
198181
}
199182

200-
/**
201-
* PsBasic can handle multiple headers with invalid content.
202-
*/
203183
@Test
204184
void handleMultipleHeadersWithInvalidContent() {
205185
Assertions.assertThrows(
@@ -230,10 +210,6 @@ void handleMultipleHeadersWithInvalidContent() {
230210
);
231211
}
232212

233-
/**
234-
* PsBasic can authenticate a user.
235-
* @throws Exception If some problem inside
236-
*/
237213
@Test
238214
void authenticatesUser() throws Exception {
239215
final Take take = new TkAuth(
@@ -259,10 +235,6 @@ void authenticatesUser() throws Exception {
259235
).affirm();
260236
}
261237

262-
/**
263-
* PsBasic can request authentication.
264-
* @throws Exception If some problem inside
265-
*/
266238
@Test
267239
void requestAuthentication() throws Exception {
268240
final Take take = new TkForward(

src/test/java/org/takes/facets/auth/PsByFlagTest.java

Lines changed: 0 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -46,10 +46,6 @@
4646
* @since 0.10
4747
*/
4848
final class PsByFlagTest {
49-
/**
50-
* PsByFlag can skip if nothing found.
51-
* @throws Exception If some problem inside
52-
*/
5349
@Test
5450
void skipsIfNothingFound() throws Exception {
5551
MatcherAssert.assertThat(
@@ -64,10 +60,6 @@ void skipsIfNothingFound() throws Exception {
6460
);
6561
}
6662

67-
/**
68-
* PsByFlag finds flag and authenticates user.
69-
* @throws Exception If some problem inside
70-
*/
7163
@Test
7264
void flagIsFoundUserAuthenticated() throws Exception {
7365
MatcherAssert.assertThat(
@@ -81,9 +73,6 @@ void flagIsFoundUserAuthenticated() throws Exception {
8173
);
8274
}
8375

84-
/**
85-
* PsByFlag wraps response with authenticated user.
86-
*/
8776
@Test
8877
@SuppressWarnings("unchecked")
8978
void exitTest() {
@@ -106,9 +95,6 @@ void exitTest() {
10695
);
10796
}
10897

109-
/**
110-
* Checks PsByFlag equals method.
111-
*/
11298
@Test
11399
void mustEvaluateTrueEqualityTest() {
114100
final Map<Pattern, Pass> passes = new HashMap<>(1);

src/test/java/org/takes/facets/auth/PsChainTest.java

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -36,10 +36,6 @@
3636
*/
3737
final class PsChainTest {
3838

39-
/**
40-
* PsChain returns proper identity.
41-
* @throws Exception if some problems inside
42-
*/
4339
@Test
4440
void chainExecutionTest() throws Exception {
4541
MatcherAssert.assertThat(
@@ -51,10 +47,6 @@ void chainExecutionTest() throws Exception {
5147
);
5248
}
5349

54-
/**
55-
* PsChain returns proper response.
56-
* @throws Exception if some problems inside
57-
*/
5850
@Test
5951
void exitChainTest() throws Exception {
6052
MatcherAssert.assertThat(

src/test/java/org/takes/facets/auth/PsCookieTest.java

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -37,10 +37,6 @@
3737
*/
3838
final class PsCookieTest {
3939

40-
/**
41-
* PsCookie can add a cookie.
42-
* @throws IOException If some problem inside
43-
*/
4440
@Test
4541
void addsCookieToResponse() throws IOException {
4642
new Assertion<>(

src/test/java/org/takes/facets/auth/RqAuthTest.java

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -37,10 +37,6 @@
3737
*/
3838
final class RqAuthTest {
3939

40-
/**
41-
* RqAuth can return identity.
42-
* @throws IOException If some problem inside
43-
*/
4440
@Test
4541
void returnsIdentity() throws IOException {
4642
final Identity.Simple identity = new Identity.Simple("urn:test:1");

src/test/java/org/takes/facets/auth/TkAuthTest.java

Lines changed: 0 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -50,10 +50,6 @@
5050
*/
5151
final class TkAuthTest {
5252

53-
/**
54-
* TkAuth can login a user.
55-
* @throws Exception If some problem inside
56-
*/
5753
@Test
5854
void logsUserIn() throws Exception {
5955
final Pass pass = new PsFixed(new Identity.Simple("urn:test:1"));
@@ -72,10 +68,6 @@ void logsUserIn() throws Exception {
7268
);
7369
}
7470

75-
/**
76-
* TkAuth can login a user via cookie.
77-
* @throws Exception If some problem inside
78-
*/
7971
@Test
8072
@SuppressWarnings("unchecked")
8173
void logsInUserViaCookie() throws Exception {
@@ -108,10 +100,6 @@ void logsInUserViaCookie() throws Exception {
108100
).affirm();
109101
}
110102

111-
/**
112-
* TkAuth can logout a user.
113-
* @throws Exception If some problem inside
114-
*/
115103
@Test
116104
void logsUserOut() throws Exception {
117105
final Pass pass = new PsLogout();
@@ -136,10 +124,6 @@ void logsUserOut() throws Exception {
136124
);
137125
}
138126

139-
/**
140-
* TkAuth can logout a user when a login cookie is present.
141-
* @throws Exception If some problem inside
142-
*/
143127
@Test
144128
void logsUserOutWithCookiePresent() throws Exception {
145129
new Assertion<>(

0 commit comments

Comments
 (0)