Skip to content

Commit 35c0035

Browse files
committed
rollback my previous change to make this test pass, it was doing the right thing before!
1 parent d75a86d commit 35c0035

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

java/client/test/org/openqa/selenium/remote/ErrorHandlerTest.java

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -144,7 +144,7 @@ public void testShouldNotSetCauseIfResponseValueIsJustAString() {
144144
handler.throwIfResponseFailed(createResponse(ErrorCodes.UNHANDLED_ERROR, "boom"), 123);
145145
fail("Should have thrown!");
146146
} catch (WebDriverException expected) {
147-
assertEquals(InvalidArgumentException.class, expected.getClass());
147+
assertEquals(WebDriverException.class, expected.getClass());
148148
assertDoesNotHaveACause(expected);
149149
assertThat(expected.getMessage(), containsString("boom"));
150150
assertThat(expected.getMessage(), containsString(new WebDriverException().getMessage()));
@@ -282,7 +282,7 @@ public void testShouldDefaultToWebDriverExceptionIfClassIsNotSpecified()
282282

283283
Throwable cause = expected.getCause();
284284
assertNotNull(cause);
285-
assertEquals(InvalidArgumentException.class, cause.getClass());
285+
assertEquals(WebDriverException.class, cause.getClass());
286286
assertEquals(new WebDriverException(serverError.getMessage()).getMessage(),
287287
cause.getMessage());
288288
assertStackTracesEqual(serverError.getStackTrace(), cause.getStackTrace());
@@ -315,7 +315,7 @@ public void testShouldStillTryToBuildWebDriverExceptionIfClassIsNotProvidedAndSt
315315

316316
Throwable cause = expected.getCause();
317317
assertNotNull(cause);
318-
assertEquals(InvalidArgumentException.class, cause.getClass());
318+
assertEquals(WebDriverException.class, cause.getClass());
319319
assertEquals(helper.getMessage(),
320320
cause.getMessage());
321321

@@ -349,7 +349,7 @@ public void testToleratesNonNumericLineNumber() {
349349

350350
Throwable cause = expected.getCause();
351351
assertNotNull(cause);
352-
assertEquals(InvalidArgumentException.class, cause.getClass());
352+
assertEquals(WebDriverException.class, cause.getClass());
353353
assertEquals(helper.getMessage(),
354354
cause.getMessage());
355355

@@ -383,7 +383,7 @@ public void testToleratesNumericLineNumberAsString() {
383383

384384
Throwable cause = expected.getCause();
385385
assertNotNull(cause);
386-
assertEquals(InvalidArgumentException.class, cause.getClass());
386+
assertEquals(WebDriverException.class, cause.getClass());
387387
assertEquals(helper.getMessage(),
388388
cause.getMessage());
389389

0 commit comments

Comments
 (0)