Fix github service tests

pom.xml

@@ -120,20 +120,67 @@
     </dependency>
 
     <dependency>
-        <groupId>org.springframework.restdocs</groupId>
-        <artifactId>spring-restdocs-mockmvc</artifactId>
-        <scope>test</scope>
+      <groupId>org.springframework.restdocs</groupId>
+      <artifactId>spring-restdocs-mockmvc</artifactId>
+      <scope>test</scope>
     </dependency>
 
     <dependency>
       <groupId>org.mockito</groupId>
       <artifactId>mockito-core</artifactId>
+      <version>4.6.1</version>
       <scope>test</scope>
+      <exclusions>
+        <exclusion>
+          <groupId>net.bytebuddy</groupId>
+          <artifactId>byte-buddy</artifactId>
+        </exclusion>
+        <exclusion>
+          <groupId>net.bytebuddy</groupId>
+          <artifactId>byte-buddy-agent</artifactId>
+        </exclusion>
+      </exclusions>
     </dependency>
 
     <dependency>
       <groupId>org.mockito</groupId>
       <artifactId>mockito-junit-jupiter</artifactId>
+      <version>4.6.1</version>
+      <scope>test</scope>
+    </dependency>
+
+    <dependency>
+      <groupId>net.bytebuddy</groupId>
+      <artifactId>byte-buddy</artifactId>
+      <version>1.12.13</version>
+      <scope>test</scope>
+    </dependency>
+
+    <dependency>
+      <groupId>net.bytebuddy</groupId>
+      <artifactId>byte-buddy-agent</artifactId>
+      <version>1.12.13</version>
+      <scope>test</scope>
+    </dependency>
+
+    <dependency>
+      <groupId>com.infradna.tool</groupId>
+      <artifactId>bridge-method-annotation</artifactId>
+      <version>1.23</version>
+      <scope>test</scope>
+    </dependency>
+
+    <dependency>
+      <groupId>com.google.code.findbugs</groupId>
+      <artifactId>jsr305</artifactId>
+      <version>3.0.2</version>
+      <scope>test</scope>
+    </dependency>
+
+    <dependency>
+      <groupId>com.google.code.findbugs</groupId>
+      <artifactId>annotations</artifactId>
+      <version>3.0.1</version>
       <scope>test</scope>
     </dependency>
 

src/main/java/edu/tamu/app/service/manager/GitHubMilestoneService.java

@@ -34,15 +34,15 @@ public List<Sprint> getActiveSprintsByScopeId(final String scopeId) throws Excep
         logger.info("Fetching active sprints for remote project with scope id " + scopeId);
         GHRepository repo = github.getRepositoryById(scopeId);
         String productName = repo.getName();
-        return repo.listProjects(ProjectStateFilter.OPEN).asList().stream()
+        return repo.listProjects(ProjectStateFilter.OPEN).toList().stream()
             .flatMap(project -> getActiveSprintsForProject(project, productName))
             .collect(Collectors.toList());
     }
 
     @Override
     public List<Sprint> getAdditionalActiveSprints() throws Exception {
         GHOrganization organization = github.getOrganization(ORGANIZATION);
-        return organization.listProjects(ProjectStateFilter.OPEN).asList().stream()
+        return organization.listProjects(ProjectStateFilter.OPEN).toList().stream()
             .flatMap(project -> getActiveSprintsForProject(project, toProductName(project)))
             .collect(Collectors.toList());
     }
@@ -55,8 +55,8 @@ private Stream<Sprint> getActiveSprintsForProject(GHProject project, String prod
     }
 
     private Map<String, List<Card>> getCards(GHProject project) throws IOException {
-        return project.listColumns().asList().stream()
-            .flatMap(column -> exceptionHandlerWrapper(column, c -> c.listCards().asList().stream()))
+        return project.listColumns().toList().stream()
+            .flatMap(column -> exceptionHandlerWrapper(column, c -> c.listCards().toList().stream()))
             .map(card -> Pair.of(card, exceptionHandlerWrapper(card, c -> c.getContent())))
             // Card without contents is a note
             .filter(p -> Objects.nonNull(p.getValue()))

src/main/java/edu/tamu/app/service/manager/GitHubProjectService.java

@@ -28,7 +28,7 @@ public List<Sprint> getActiveSprintsByScopeId(final String scopeId) throws Excep
         logger.info("Fetching active sprints for remote project with scope id " + scopeId);
         GHRepository repo = github.getRepositoryById(scopeId);
         String productName = repo.getName();
-        return repo.listProjects(ProjectStateFilter.OPEN).asList().stream()
+        return repo.listProjects(ProjectStateFilter.OPEN).toList().stream()
             .filter(this::isSprintProject)
             .map(p -> toSprint(p, productName))
             .collect(Collectors.toList());
@@ -37,7 +37,7 @@ public List<Sprint> getActiveSprintsByScopeId(final String scopeId) throws Excep
     @Override
     public List<Sprint> getAdditionalActiveSprints() throws Exception {
         return github.getOrganization(ORGANIZATION)
-            .listProjects(ProjectStateFilter.OPEN).asList().stream()
+            .listProjects(ProjectStateFilter.OPEN).toList().stream()
             .filter(this::isSprintProject)
             .map(p -> toSprint(p, ORGANIZATION))
             .collect(Collectors.toList());
@@ -58,8 +58,8 @@ private Sprint toSprint(GHProject project, String productName) {
     }
 
     private List<Card> getCards(GHProject project) {
-        return exceptionHandlerWrapper(project, i -> i.listColumns().asList().stream())
-            .flatMap(c -> exceptionHandlerWrapper(c, i -> i.listCards().asList().stream()))
+        return exceptionHandlerWrapper(project, i -> i.listColumns().toList().stream())
+            .flatMap(c -> exceptionHandlerWrapper(c, i -> i.listCards().toList().stream()))
             .map(c -> Pair.of(c, exceptionHandlerWrapper(c, i -> i.getContent())))
             // Card without contents is a note
             .filter(e -> Objects.nonNull(e.getValue()))

src/test/java/edu/tamu/app/cache/controller/integration/ActiveSprintsCacheControllerIntegrationTest.java

@@ -15,7 +15,6 @@
 import org.junit.jupiter.api.AfterEach;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
-import org.junit.jupiter.api.extension.ExtendWith;
 import org.mockito.MockitoAnnotations;
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.boot.test.autoconfigure.restdocs.AutoConfigureRestDocs;
@@ -25,7 +24,6 @@
 import org.springframework.boot.test.mock.mockito.MockBean;
 import org.springframework.http.MediaType;
 import org.springframework.security.test.context.support.WithMockUser;
-import org.springframework.test.context.junit.jupiter.SpringExtension;
 import org.springframework.test.web.servlet.MockMvc;
 
 import edu.tamu.app.ProductApplication;
@@ -41,7 +39,6 @@
 @SpringBootTest(classes = { ProductApplication.class }, webEnvironment=WebEnvironment.RANDOM_PORT)
 @AutoConfigureMockMvc
 @AutoConfigureRestDocs(outputDir = "target/generated-snippets")
-@ExtendWith(SpringExtension.class)
 public class ActiveSprintsCacheControllerIntegrationTest extends AbstractRepoTest {
 
     private static final String TEST_MEMBER_ID = "Test Member ID";

src/test/java/edu/tamu/app/controller/integration/InternalRequestControllerIntegrationTest.java

@@ -27,13 +27,9 @@
 import java.util.List;
 import java.util.Optional;
 
-import com.fasterxml.jackson.core.JsonProcessingException;
-import com.fasterxml.jackson.databind.JsonNode;
-
 import org.junit.jupiter.api.AfterEach;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
-import org.junit.jupiter.api.extension.ExtendWith;
 import org.kohsuke.github.GitHubBuilder;
 import org.mockito.MockitoAnnotations;
 import org.springframework.beans.factory.annotation.Autowired;
@@ -44,10 +40,12 @@
 import org.springframework.boot.test.mock.mockito.MockBean;
 import org.springframework.http.MediaType;
 import org.springframework.security.test.context.support.WithMockUser;
-import org.springframework.test.context.junit.jupiter.SpringExtension;
 import org.springframework.test.web.servlet.MockMvc;
 import org.springframework.test.web.servlet.ResultActions;
 
+import com.fasterxml.jackson.core.JsonProcessingException;
+import com.fasterxml.jackson.databind.JsonNode;
+
 import edu.tamu.app.ProductApplication;
 import edu.tamu.app.cache.service.ActiveSprintsScheduledCacheService;
 import edu.tamu.app.cache.service.ProductsStatsScheduledCacheService;
@@ -69,7 +67,6 @@
 @SpringBootTest(classes = { ProductApplication.class }, webEnvironment=WebEnvironment.RANDOM_PORT)
 @AutoConfigureMockMvc
 @AutoConfigureRestDocs(outputDir = "target/generated-snippets")
-@ExtendWith(SpringExtension.class)
 public class InternalRequestControllerIntegrationTest extends AbstractRepoTest {
 
     private static long currentId = 0L;

src/test/java/edu/tamu/app/controller/integration/RemoteProjectManagerControllerIntegrationTest.java

@@ -17,7 +17,6 @@
 import org.junit.jupiter.api.AfterEach;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
-import org.junit.jupiter.api.extension.ExtendWith;
 import org.kohsuke.github.GitHubBuilder;
 import org.mockito.MockitoAnnotations;
 import org.springframework.beans.factory.annotation.Autowired;
@@ -28,7 +27,6 @@
 import org.springframework.boot.test.mock.mockito.MockBean;
 import org.springframework.http.MediaType;
 import org.springframework.security.test.context.support.WithMockUser;
-import org.springframework.test.context.junit.jupiter.SpringExtension;
 import org.springframework.test.web.servlet.MockMvc;
 import org.springframework.test.web.servlet.ResultActions;
 
@@ -40,8 +38,8 @@
 import edu.tamu.app.cache.service.RemoteProjectsScheduledCacheService;
 import edu.tamu.app.model.RemoteProjectManager;
 import edu.tamu.app.model.ServiceType;
-import edu.tamu.app.model.repo.RemoteProjectManagerRepo;
 import edu.tamu.app.model.repo.AbstractRepoTest;
+import edu.tamu.app.model.repo.RemoteProjectManagerRepo;
 import edu.tamu.app.service.manager.GitHubProjectService;
 import edu.tamu.app.service.manager.VersionOneService;
 import edu.tamu.weaver.response.ApiResponse;
@@ -50,7 +48,6 @@
 @SpringBootTest(classes = { ProductApplication.class }, webEnvironment=WebEnvironment.RANDOM_PORT)
 @AutoConfigureMockMvc
 @AutoConfigureRestDocs(outputDir = "target/generated-snippets")
-@ExtendWith(SpringExtension.class)
 public class RemoteProjectManagerControllerIntegrationTest extends AbstractRepoTest {
 
     private static long currentId = 0L;

src/test/java/edu/tamu/app/mapping/EstimateMappingServiceTest.java

@@ -8,17 +8,14 @@
 import org.junit.jupiter.api.AfterEach;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
-import org.junit.jupiter.api.extension.ExtendWith;
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.boot.test.context.SpringBootTest;
 import org.springframework.boot.test.context.SpringBootTest.WebEnvironment;
-import org.springframework.test.context.junit.jupiter.SpringExtension;
 
 import edu.tamu.app.ProductApplication;
 import edu.tamu.app.model.Estimate;
 import edu.tamu.app.model.repo.EstimateRepo;
 
-@ExtendWith(SpringExtension.class)
 @SpringBootTest(classes = { ProductApplication.class }, webEnvironment = WebEnvironment.RANDOM_PORT)
 public class EstimateMappingServiceTest {
 

src/test/java/edu/tamu/app/model/ProductTest.java

@@ -8,14 +8,12 @@
 
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
-import org.junit.jupiter.api.extension.ExtendWith;
 import org.kohsuke.github.GitHub;
 import org.kohsuke.github.GitHubBuilder;
 import org.mockito.MockitoAnnotations;
 import org.springframework.boot.test.context.SpringBootTest;
 import org.springframework.boot.test.context.SpringBootTest.WebEnvironment;
 import org.springframework.boot.test.mock.mockito.MockBean;
-import org.springframework.test.context.junit.jupiter.SpringExtension;
 
 import edu.tamu.app.ProductApplication;
 import edu.tamu.app.cache.service.ActiveSprintsScheduledCacheService;
@@ -25,7 +23,6 @@
 import edu.tamu.app.service.manager.VersionOneService;
 import edu.tamu.app.service.ticketing.SugarService;
 
-@ExtendWith(SpringExtension.class)
 @SpringBootTest(classes = { ProductApplication.class }, webEnvironment = WebEnvironment.RANDOM_PORT)
 public class ProductTest extends AbstractModelTest {
 

src/test/java/edu/tamu/app/model/repo/EstimateRepoTest.java

@@ -12,15 +12,13 @@
 import org.junit.jupiter.api.AfterEach;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
-import org.junit.jupiter.api.extension.ExtendWith;
 import org.kohsuke.github.GitHub;
 import org.kohsuke.github.GitHubBuilder;
 import org.mockito.MockitoAnnotations;
 import org.springframework.boot.test.context.SpringBootTest;
 import org.springframework.boot.test.context.SpringBootTest.WebEnvironment;
 import org.springframework.boot.test.mock.mockito.MockBean;
 import org.springframework.dao.DataIntegrityViolationException;
-import org.springframework.test.context.junit.jupiter.SpringExtension;
 
 import edu.tamu.app.ProductApplication;
 import edu.tamu.app.cache.service.ActiveSprintsScheduledCacheService;
@@ -31,7 +29,6 @@
 import edu.tamu.app.service.manager.VersionOneService;
 import edu.tamu.app.service.ticketing.SugarService;
 
-@ExtendWith(SpringExtension.class)
 @SpringBootTest(classes = { ProductApplication.class }, webEnvironment = WebEnvironment.RANDOM_PORT)
 public class EstimateRepoTest extends AbstractRepoTest {
 

src/test/java/edu/tamu/app/model/repo/InternalRequestRepoTest.java

@@ -9,14 +9,12 @@
 import org.junit.jupiter.api.AfterEach;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
-import org.junit.jupiter.api.extension.ExtendWith;
 import org.kohsuke.github.GitHub;
 import org.kohsuke.github.GitHubBuilder;
 import org.mockito.MockitoAnnotations;
 import org.springframework.boot.test.context.SpringBootTest;
 import org.springframework.boot.test.context.SpringBootTest.WebEnvironment;
 import org.springframework.boot.test.mock.mockito.MockBean;
-import org.springframework.test.context.junit.jupiter.SpringExtension;
 
 import edu.tamu.app.ProductApplication;
 import edu.tamu.app.cache.service.ActiveSprintsScheduledCacheService;
@@ -31,7 +29,6 @@
 import edu.tamu.app.service.manager.VersionOneService;
 import edu.tamu.app.service.ticketing.SugarService;
 
-@ExtendWith(SpringExtension.class)
 @SpringBootTest(classes = { ProductApplication.class }, webEnvironment = WebEnvironment.RANDOM_PORT)
 public class InternalRequestRepoTest extends AbstractRepoTest {
 

src/test/java/edu/tamu/app/model/repo/ProductRepoTest.java

@@ -12,7 +12,6 @@
 import org.junit.jupiter.api.AfterEach;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
-import org.junit.jupiter.api.extension.ExtendWith;
 import org.kohsuke.github.GitHub;
 import org.kohsuke.github.GitHubBuilder;
 import org.mockito.Mock;
@@ -22,7 +21,6 @@
 import org.springframework.boot.test.mock.mockito.MockBean;
 import org.springframework.dao.DataIntegrityViolationException;
 import org.springframework.messaging.simp.SimpMessagingTemplate;
-import org.springframework.test.context.junit.jupiter.SpringExtension;
 
 import edu.tamu.app.ProductApplication;
 import edu.tamu.app.cache.service.ActiveSprintsScheduledCacheService;
@@ -37,10 +35,9 @@
 import edu.tamu.app.service.registry.ManagementBeanRegistry;
 import edu.tamu.app.service.ticketing.SugarService;
 
-@ExtendWith(SpringExtension.class)
 @SpringBootTest(classes = { ProductApplication.class }, webEnvironment = WebEnvironment.RANDOM_PORT)
 public class ProductRepoTest extends AbstractRepoTest {
-    
+
     private static final RemoteProjectManager TEST_REMOTE_PROJECT_MANAGER1 = new RemoteProjectManager(TEST_REMOTE_PROJECT_MANAGER_NAME1, ServiceType.VERSION_ONE, TEST_PROJECT_URL1, TEST_PROJECT_TOKEN1);
 
     private static final RemoteProjectInfo TEST_REMOTE_PROJECT_INFO1 = new RemoteProjectInfo(TEST_PROJECT_SCOPE1, TEST_REMOTE_PROJECT_MANAGER1);

src/test/java/edu/tamu/app/model/request/AbstractRequetTest.java

@@ -9,7 +9,6 @@
 import org.mockito.Mockito;
 import org.springframework.test.context.junit.jupiter.SpringExtension;
 
-
 @ExtendWith(SpringExtension.class)
 public class AbstractRequetTest {