From 059440f74e2a8f10d8a7705adfd8bbae2de3bded Mon Sep 17 00:00:00 2001
From: Olivia Kotsopoulos <okotsopo@broadinstitute.org>
Date: Tue, 6 Aug 2024 20:23:01 -0400
Subject: [PATCH] Downgrade swagger-codegen to version prior to breaking change

3.0.57 introduced a change in behavior which breaks many of our MVC unit tests.
See: https://github.com/swagger-api/swagger-codegen-generators/issues/1295

Runtime groovy dependency caused bootRun to fail on startup.  It was previously needed for logback, but we use XML now (logback hasn't supported groovy since 2021: https://logback.qos.ch/news.html#1.2.9)

Hadoop upgrades used in tests required the exclusion of slf4j-reload4j dependency for Spring Boot tests to initialize.
---
 build.gradle | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)

diff --git a/build.gradle b/build.gradle
index d01ed0243f..a623eefffa 100644
--- a/build.gradle
+++ b/build.gradle
@@ -20,7 +20,7 @@ buildscript {
         mavenCentral()
     }
     dependencies {
-        classpath('io.swagger.codegen.v3:swagger-codegen:3.0.60')
+        classpath('io.swagger.codegen.v3:swagger-codegen:3.0.56')
     }
 }
 
@@ -59,7 +59,7 @@ allprojects {
         }
         dependencies {
             dependency 'io.swagger.core.v3:swagger-annotations:2.2.22'
-            dependency 'io.swagger.codegen.v3:swagger-codegen-cli:3.0.60'
+            dependency 'io.swagger.codegen.v3:swagger-codegen-cli:3.0.56'
         }
     }
 }
@@ -252,18 +252,21 @@ dependencies {
         exclude group: 'com.sun.jersey', module: 'jersey-servlet'
         exclude group: 'com.sun.jersey', module: 'jersey-json'
         exclude group: 'com.sun.jersey', module: 'jersey-server'
+        exclude group: 'org.slf4j', module: 'slf4j-reload4j'
     }
     testImplementation ('org.apache.hadoop:hadoop-azure:3.4.0') {
         exclude group: 'com.sun.jersey', module: 'jersey-core'
         exclude group: 'com.sun.jersey', module: 'jersey-servlet'
         exclude group: 'com.sun.jersey', module: 'jersey-json'
         exclude group: 'com.sun.jersey', module: 'jersey-server'
+        exclude group: 'org.slf4j', module: 'slf4j-reload4j'
     }
     testImplementation('org.apache.hadoop:hadoop-mapreduce-client-core:3.4.0') {
         exclude group: 'com.sun.jersey', module: 'jersey-core'
         exclude group: 'com.sun.jersey', module: 'jersey-servlet'
         exclude group: 'com.sun.jersey', module: 'jersey-json'
         exclude group: 'com.sun.jersey', module: 'jersey-server'
+        exclude group: 'org.slf4j', module: 'slf4j-reload4j'
     }
 
     testImplementation 'au.com.dius.pact.provider:junit5:4.6.12'
@@ -271,10 +274,6 @@ dependencies {
 
     antlr 'org.antlr:antlr4:4.13.2'
 
-    // Need groovy on the class path for the logback config. Could use XML and skip this dependency,
-    // but the groovy config is... well... groovy.
-    runtimeOnly 'org.codehaus.groovy:groovy:3.0.22'
-
     liquibaseRuntime 'org.liquibase:liquibase-core'
     liquibaseRuntime 'org.postgresql:postgresql'
     liquibaseRuntime 'info.picocli:picocli:4.7.6'