Skip to content

Commit

Permalink
GH-42208: [Java] Fix the Test in flight-sql-jdbc-driver Module (#42217)
Browse files Browse the repository at this point in the history
### Rationale for this change

Adjust assertion message order for JUnit 5 compatibility.

### What changes are included in this PR?

- Reordering the arguments of `assertEquals(expected, actual, message)`

### Are these changes tested?

Yes.

### Are there any user-facing changes?

No.

* GitHub Issue: #42208

Authored-by: Hyunseok Seo <hsseo0501@gmail.com>
Signed-off-by: David Li <li.davidm96@gmail.com>
  • Loading branch information
llama90 authored Jun 20, 2024
1 parent a01fe03 commit 27bbc3c
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 +70,7 @@ private static JarFile getJdbcJarFile() throws IOException {

assertNotNull(driverClassURL, "Driver jar was not detected in the classpath");
assertEquals(
"Driver jar was not detected in the classpath", "jar", driverClassURL.getProtocol());
"jar", driverClassURL.getProtocol(), "Driver jar was not detected in the classpath");

JarURLConnection connection = (JarURLConnection) driverClassURL.openConnection();
return connection.getJarFile();
Expand Down

0 comments on commit 27bbc3c

Please sign in to comment.