From f0eb4077ba3884729b6125e7145b68fc6e5c7946 Mon Sep 17 00:00:00 2001
From: Adam Rauch <adam@labkey.com>
Date: Thu, 2 Oct 2025 13:50:38 -0700
Subject: [PATCH] Return Set<Class<?>> from Module "get test" methods

---
 .../src/org/labkey/oconnorexperiments/view/history.jsp      | 4 +---
 .../src/org/labkey/genotyping/GenotypingAnalysesView.java   | 4 ++--
 genotyping/src/org/labkey/genotyping/GenotypingModule.java  | 6 ++----
 .../src/org/labkey/genotyping/GenotypingRunsView.java       | 4 ++--
 genotyping/src/org/labkey/genotyping/GenotypingWebPart.java | 2 +-
 5 files changed, 8 insertions(+), 12 deletions(-)

diff --git a/OConnorExperiments/src/org/labkey/oconnorexperiments/view/history.jsp b/OConnorExperiments/src/org/labkey/oconnorexperiments/view/history.jsp
index 796b1e3a..417d36ed 100644
--- a/OConnorExperiments/src/org/labkey/oconnorexperiments/view/history.jsp
+++ b/OConnorExperiments/src/org/labkey/oconnorexperiments/view/history.jsp
@@ -24,10 +24,8 @@
 <%@ page import="org.labkey.api.query.QueryView" %>
 <%@ page import="org.labkey.api.query.UserSchema" %>
 <%@ page import="org.labkey.api.security.User" %>
-<%@ page import="org.labkey.api.services.ServiceRegistry" %>
 <%@ page import="org.labkey.api.view.ViewContext" %>
 <%@ page import="org.labkey.api.view.WebPartView" %>
-<%@ page import="org.labkey.api.view.template.ClientDependencies" %>
 <%@ page import="org.labkey.api.wiki.WikiService" %>
 <%@ page import="org.labkey.oconnorexperiments.query.OConnorExperimentsUserSchema" %>
 <%@ page extends="org.labkey.api.jsp.JspBase" %>
@@ -86,7 +84,7 @@
 <h3>Notes History</h3>
 <%
     {
-        WebPartView view = WikiService.get().getHistoryView(c, "default");
+        WebPartView<?> view = WikiService.get().getHistoryView(c, "default");
         if (view != null)
             include(view, out);
     }
diff --git a/genotyping/src/org/labkey/genotyping/GenotypingAnalysesView.java b/genotyping/src/org/labkey/genotyping/GenotypingAnalysesView.java
index 0a4e0c9a..85b32e97 100644
--- a/genotyping/src/org/labkey/genotyping/GenotypingAnalysesView.java
+++ b/genotyping/src/org/labkey/genotyping/GenotypingAnalysesView.java
@@ -41,9 +41,9 @@ public class GenotypingAnalysesView extends QueryView
     public static final WebPartFactory FACTORY = new BaseWebPartFactory("Genotyping Analyses")
     {
         @Override
-        public WebPartView getWebPartView(@NotNull ViewContext ctx, @NotNull Portal.WebPart webPart)
+        public WebPartView<?> getWebPartView(@NotNull ViewContext ctx, @NotNull Portal.WebPart webPart)
         {
-            WebPartView view = new GenotypingAnalysesView(ctx, null, "GenotypingAnalyses");
+            WebPartView<?> view = new GenotypingAnalysesView(ctx, null, "GenotypingAnalyses");
             view.setTitle("Genotyping Analyses");
             view.setTitleHref(GenotypingController.getAnalysesURL(ctx.getContainer()));
             return view;
diff --git a/genotyping/src/org/labkey/genotyping/GenotypingModule.java b/genotyping/src/org/labkey/genotyping/GenotypingModule.java
index 903a186f..7bb0dbe9 100644
--- a/genotyping/src/org/labkey/genotyping/GenotypingModule.java
+++ b/genotyping/src/org/labkey/genotyping/GenotypingModule.java
@@ -140,16 +140,14 @@ public Set<String> getSchemaNames()
         return Collections.singleton(GenotypingSchema.get().getSchemaName());
     }
 
-    @NotNull
     @Override
-    public Set<Class> getUnitTests()
+    public @NotNull Set<Class<?>> getUnitTests()
     {
         return PageFlowUtil.set(HaplotypeDataHandler.TestCase.class, IlluminaFastqParser.DupeTestCase.class);
     }
 
-    @NotNull
     @Override
-    public Set<Class> getIntegrationTests()
+    public @NotNull Set<Class<?>> getIntegrationTests()
     {
         return PageFlowUtil.set(IlluminaFastqParser.HeaderTestCase.class, ImportAnalysisJob.TestCase.class);
     }
diff --git a/genotyping/src/org/labkey/genotyping/GenotypingRunsView.java b/genotyping/src/org/labkey/genotyping/GenotypingRunsView.java
index e13ff8f5..f67503bb 100644
--- a/genotyping/src/org/labkey/genotyping/GenotypingRunsView.java
+++ b/genotyping/src/org/labkey/genotyping/GenotypingRunsView.java
@@ -42,9 +42,9 @@ public class GenotypingRunsView extends QueryView
     public static final WebPartFactory FACTORY = new BaseWebPartFactory("Sequencing Runs")
     {
         @Override
-        public WebPartView getWebPartView(@NotNull ViewContext ctx, @NotNull Portal.WebPart webPart)
+        public WebPartView<?> getWebPartView(@NotNull ViewContext ctx, @NotNull Portal.WebPart webPart)
         {
-            WebPartView view = new GenotypingRunsView(ctx, null, "SequencingRuns", false);
+            WebPartView<?> view = new GenotypingRunsView(ctx, null, "SequencingRuns", false);
             view.setTitle("Sequencing Runs");
             view.setTitleHref(GenotypingController.getRunsURL(ctx.getContainer()));
             return view;
diff --git a/genotyping/src/org/labkey/genotyping/GenotypingWebPart.java b/genotyping/src/org/labkey/genotyping/GenotypingWebPart.java
index 2acd6343..0b90d61a 100644
--- a/genotyping/src/org/labkey/genotyping/GenotypingWebPart.java
+++ b/genotyping/src/org/labkey/genotyping/GenotypingWebPart.java
@@ -36,7 +36,7 @@ public class GenotypingWebPart extends JspView
     public static final WebPartFactory FACTORY = new BaseWebPartFactory("Genotyping Overview")
     {
         @Override
-        public WebPartView getWebPartView(@NotNull ViewContext portalCtx, @NotNull Portal.WebPart webPart)
+        public WebPartView<?> getWebPartView(@NotNull ViewContext portalCtx, @NotNull Portal.WebPart webPart)
         {
             return new GenotypingWebPart();
         }