Fix type mismatch in DAO tests from queryForInt to queryForObject migration

[Copilot]

PR #296 migrated from deprecated queryForInt() to queryForObject(sqlStr, Integer.class), but assigned results to primitive int variables, causing compilation errors.

Changes

Changed variable declarations from int to Integer for all queryForObject() calls returning Integer.class:

// Before (compilation error)
int count = jdbcTemplate.queryForObject(sqlStr, Integer.class);
assertTrue(count == descriptions.size());

// After
Integer count = jdbcTemplate.queryForObject(sqlStr, Integer.class);
assertTrue(count == descriptions.size());

Auto-unboxing preserves all existing comparison logic.

Files Modified

• AlgorithmDAOTest.java (2)
• AnalyzedDataDAOTest.java (2)
• MatrixDAOTest.java (2)
• PersonDAOTest.java (2)
• PhyloTreeDAOTest.java (9)
• RowSegmentDAOTest.java (3)
• StudyDAOTest.java (2)
• SubmissionDAOTest.java (3)
• UserDAOTest.java (3)

28 lines across 9 test files.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

• download.osgeo.org
  • Triggering command: /usr/lib/jvm/temurin-17-jdk-amd64/bin/java /usr/lib/jvm/temurin-17-jdk-amd64/bin/java --enable-native-access=ALL-UNNAMED -classpath /usr/share/apache-maven-3.9.11/boot/plexus-classworlds-2.9.0.jar -Dclassworlds.conf=/usr/share/apache-maven-3.9.11/bin/m2.conf -Dmaven.home=/usr/share/apache-maven-3.9.11 -Dlibrary.jansi.path=/usr/share/apache-maven-3.9.11/lib/jansi-native -Dmaven.multiModuleProjectDirectory=/home/REDACTED/work/treebase/treebase org.codehaus.plexus.classworlds.launcher.Launcher clean compile -DskipTests (dns block)
  • Triggering command: /usr/lib/jvm/temurin-17-jdk-amd64/bin/java /usr/lib/jvm/temurin-17-jdk-amd64/bin/java --enable-native-access=ALL-UNNAMED -classpath /usr/share/apache-maven-3.9.11/boot/plexus-classworlds-2.9.0.jar -Dclassworlds.conf=/usr/share/apache-maven-3.9.11/bin/m2.conf -Dmaven.home=/usr/share/apache-maven-3.9.11 -Dlibrary.jansi.path=/usr/share/apache-maven-3.9.11/lib/jansi-native -Dmaven.multiModuleProjectDirectory=/home/REDACTED/work/treebase/treebase org.codehaus.plexus.classworlds.launcher.Launcher test-compile -DskipTests (dns block)
• repository.gbif.org
  • Triggering command: /usr/lib/jvm/temurin-17-jdk-amd64/bin/java /usr/lib/jvm/temurin-17-jdk-amd64/bin/java --enable-native-access=ALL-UNNAMED -classpath /usr/share/apache-maven-3.9.11/boot/plexus-classworlds-2.9.0.jar -Dclassworlds.conf=/usr/share/apache-maven-3.9.11/bin/m2.conf -Dmaven.home=/usr/share/apache-maven-3.9.11 -Dlibrary.jansi.path=/usr/share/apache-maven-3.9.11/lib/jansi-native -Dmaven.multiModuleProjectDirectory=/home/REDACTED/work/treebase/treebase org.codehaus.plexus.classworlds.launcher.Launcher clean compile -DskipTests (dns block)
  • Triggering command: /usr/lib/jvm/temurin-17-jdk-amd64/bin/java /usr/lib/jvm/temurin-17-jdk-amd64/bin/java --enable-native-access=ALL-UNNAMED -classpath /usr/share/apache-maven-3.9.11/boot/plexus-classworlds-2.9.0.jar -Dclassworlds.conf=/usr/share/apache-maven-3.9.11/bin/m2.conf -Dmaven.home=/usr/share/apache-maven-3.9.11 -Dlibrary.jansi.path=/usr/share/apache-maven-3.9.11/lib/jansi-native -Dmaven.multiModuleProjectDirectory=/home/REDACTED/work/treebase/treebase org.codehaus.plexus.classworlds.launcher.Launcher test-compile -DskipTests (dns block)
• repository.jboss.org
  • Triggering command: /usr/lib/jvm/temurin-17-jdk-amd64/bin/java /usr/lib/jvm/temurin-17-jdk-amd64/bin/java --enable-native-access=ALL-UNNAMED -classpath /usr/share/apache-maven-3.9.11/boot/plexus-classworlds-2.9.0.jar -Dclassworlds.conf=/usr/share/apache-maven-3.9.11/bin/m2.conf -Dmaven.home=/usr/share/apache-maven-3.9.11 -Dlibrary.jansi.path=/usr/share/apache-maven-3.9.11/lib/jansi-native -Dmaven.multiModuleProjectDirectory=/home/REDACTED/work/treebase/treebase org.codehaus.plexus.classworlds.launcher.Launcher clean compile -DskipTests (dns block)
  • Triggering command: /usr/lib/jvm/temurin-17-jdk-amd64/bin/java /usr/lib/jvm/temurin-17-jdk-amd64/bin/java --enable-native-access=ALL-UNNAMED -classpath /usr/share/apache-maven-3.9.11/boot/plexus-classworlds-2.9.0.jar -Dclassworlds.conf=/usr/share/apache-maven-3.9.11/bin/m2.conf -Dmaven.home=/usr/share/apache-maven-3.9.11 -Dlibrary.jansi.path=/usr/share/apache-maven-3.9.11/lib/jansi-native -Dmaven.multiModuleProjectDirectory=/home/REDACTED/work/treebase/treebase org.codehaus.plexus.classworlds.launcher.Launcher test-compile -DskipTests (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to the custom allowlist in this repository's Copilot coding agent settings (admins only)

Original prompt

In PR 296 you made a change in a number of DAO classes to go from queryForInt() to queryForObject(). However, there is a problem with this: Compilation failure:
Error: /home/runner/work/treebase/treebase/treebase-core/src/test/java/org/cipres/treebase/dao/study/AlgorithmDAOTest.java:[60,56] incompatible types: java.lang.Object cannot be converted to int

This is pervasive throughout a number of DAO test classes that were changed. Fix this.

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.