Remove deprecated tables + v133 compatibility

.github/workflows/build.yaml

@@ -16,7 +16,7 @@ jobs:
       - name: Install tools
         run: apt update && apt install -y git wget
       - name: Checkout code
-        uses: actions/checkout@v2
+        uses: actions/checkout@v4
       - name: Gradle build
         env:
           API_VERSION: ${{ steps.get_version.outputs.API_VERSION }}

src/inttest/java/com/faforever/api/config/MainDbTestContainers.java

@@ -21,7 +21,7 @@
 @Configuration
 public class MainDbTestContainers {
   private static final MariaDBContainer<?> fafDBContainer = new MariaDBContainer<>("mariadb:10.6");
-  private static final GenericContainer<?> flywayMigrationsContainer = new GenericContainer<>("faforever/faf-db-migrations:v129");
+  private static final GenericContainer<?> flywayMigrationsContainer = new GenericContainer<>("faforever/faf-db-migrations:v133");
   private static final Network sharedNetwork = Network.newNetwork();
 
   @Bean

src/inttest/resources/sql/prepDefaultData.sql

@@ -1,8 +1,3 @@
-INSERT INTO oauth_clients (id, name, client_secret, client_type, redirect_uris, default_redirect_uri, default_scope)
-VALUES ('test', 'test', '{noop}test', 'public', 'http://localhost https://www.getpostman.com/oauth2/callback ',
-        'http://localhost',
-        'read_events read_achievements upload_map upload_mod upload_avatar write_account_data vote read_sensible_userdata');
-
 INSERT INTO login (id, login, email, password, ip)
 VALUES (1, 'USER', '[email protected]', '92b7b421992ef490f3b75898ec0e511f1a5c02422819d89719b20362b023ee4f',
         '127.0.0.1'),

src/inttest/resources/sql/prepMapData.sql

@@ -6,9 +6,6 @@ INSERT INTO map_version (id, description, max_players, width, height, version, f
   (1, 'SCMP 001', 8, 5, 5, 1, 'maps/scmp_001.v0001.zip', 0, 1),
   (2, 'SCMP 002', 8, 5, 5, 1, 'maps/scmp_002.v0001.zip', 0, 2);
 
-INSERT INTO ladder_map (id, idmap) VALUES
-  (1, 1);
-
 INSERT INTO map_pool (id, name) VALUES
   (1, 'Ladder 1v1 <300'),
   (2, 'Ladder 1v1 300-800'),

src/inttest/resources/sql/truncateTables.sql

@@ -17,19 +17,8 @@ DELETE FROM `game_player_stats`;
 DELETE FROM `game_review`;
 DELETE FROM `game_reviews_summary`;
 DELETE FROM `game_stats`;
--- DELETE FROM `game_validity`;
-DELETE FROM `global_rating`;
-DELETE FROM `global_rating_distribution`;
-DELETE FROM `global_rating_rank_view`;
 -- DELETE FROM `group_permission`;
 DELETE FROM `group_permission_assignment`;
-DELETE FROM `jwt_users`;
-DELETE FROM `ladder1v1_rating`;
-DELETE FROM `ladder1v1_rating_distribution`;
-DELETE FROM `ladder1v1_rating_rank_view`;
-DELETE FROM `ladder_division`;
-DELETE FROM `ladder_division_score`;
-DELETE FROM `ladder_map`;
 -- DELETE FROM `leaderboard`;
 DELETE FROM `leaderboard_rating`;
 DELETE FROM `leaderboard_rating_journal`;
@@ -53,8 +42,6 @@ DELETE FROM `mod_version_review`;
 DELETE FROM `mod_version_reviews_summary`;
 DELETE FROM `moderation_report`;
 DELETE FROM `name_history`;
-DELETE FROM `oauth_clients`;
-DELETE FROM `oauth_tokens`;
 DELETE FROM `patchs_table`;
 DELETE FROM `player_achievements`;
 DELETE FROM `player_events`;

src/main/java/com/faforever/api/data/domain/Player.java

@@ -11,7 +11,6 @@
 import org.hibernate.annotations.BatchSize;
 
 import jakarta.persistence.Entity;
-import jakarta.persistence.FetchType;
 import jakarta.persistence.ManyToMany;
 import jakarta.persistence.OneToMany;
 import jakarta.persistence.OneToOne;
@@ -27,26 +26,12 @@
 public class Player extends Login {
 
   public static final String TYPE_NAME = "player";
-  private Ladder1v1Rating ladder1v1Rating;
-  private GlobalRating globalRating;
   private ClanMembership clanMembership;
   private Set<NameRecord> names;
   private Set<AvatarAssignment> avatarAssignments;
   private Set<ModerationReport> reporterOnModerationReports;
   private Set<ModerationReport> reportedOnModerationReports;
 
-  @OneToOne(mappedBy = "player", fetch = FetchType.LAZY)
-  @Deprecated
-  public Ladder1v1Rating getLadder1v1Rating() {
-    return ladder1v1Rating;
-  }
-
-  @OneToOne(mappedBy = "player", fetch = FetchType.LAZY)
-  @Deprecated
-  public GlobalRating getGlobalRating() {
-    return globalRating;
-  }
-
   // Permission is managed by ClanMembership class
   @UpdatePermission(expression = Prefab.ALL)
   @OneToOne(mappedBy = "player")