Skip to content

Commit

Permalink
Remove powermock based tests as they no longer work
Browse files Browse the repository at this point in the history
Newer JDKs deprecate certain access and powermock uses it. Powermock
seens unmaintained anyway, so we'd have to rewrite the whole test.

We're not really actively maintaining the driver anyway, so let's remove
these for now.

Signed-off-by: Dirkjan Bussink <[email protected]>
  • Loading branch information
dbussink committed Jan 21, 2025
1 parent 0df00ef commit b83a940
Show file tree
Hide file tree
Showing 8 changed files with 0 additions and 5,001 deletions.
19 changes: 0 additions & 19 deletions java/jdbc/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -57,25 +57,6 @@
<version>3.12.4</version>
<scope>test</scope>
</dependency>

<dependency>
<groupId>org.powermock</groupId>
<artifactId>powermock-api-mockito2</artifactId>
<version>2.0.9</version>
<scope>test</scope>
</dependency>
<dependency>
<groupId>org.powermock</groupId>
<artifactId>powermock-core</artifactId>
<version>2.0.9</version>
<scope>test</scope>
</dependency>
<dependency>
<groupId>org.powermock</groupId>
<artifactId>powermock-module-junit4</artifactId>
<version>2.0.9</version>
<scope>test</scope>
</dependency>
</dependencies>

<build>
Expand Down
637 changes: 0 additions & 637 deletions java/jdbc/src/test/java/io/vitess/jdbc/FieldWithMetadataTest.java

This file was deleted.

278 changes: 0 additions & 278 deletions java/jdbc/src/test/java/io/vitess/jdbc/VitessConnectionTest.java

This file was deleted.

Loading

0 comments on commit b83a940

Please sign in to comment.