improve always manual device tests

Yubico · Jan 29, 2025 · fdc4f2f · fdc4f2f
1 parent 5a64d1d
commit fdc4f2f
Show file tree

Hide file tree

Showing 5 changed files with 10 additions and 5 deletions.
diff --git a/...android/src/androidTest/java/com/yubico/yubikit/testing/piv/PivBioMultiProtocolTests.java b/...android/src/androidTest/java/com/yubico/yubikit/testing/piv/PivBioMultiProtocolTests.java
@@ -16,12 +16,15 @@
 
 package com.yubico.yubikit.testing.piv;
 
+import com.yubico.yubikit.testing.AlwaysManualTest;
 import com.yubico.yubikit.testing.framework.PivInstrumentedTests;
 import org.junit.Test;
+import org.junit.experimental.categories.Category;
 
 public class PivBioMultiProtocolTests extends PivInstrumentedTests {
 
   @Test
+  @Category(AlwaysManualTest.class)
   public void testAuthenticate() throws Throwable {
     withPivSession(PivBioMultiProtocolDeviceTests::testAuthenticate);
   }

diff --git a/testing-android/src/androidTest/java/com/yubico/yubikit/testing/piv/PivTests.java b/testing-android/src/androidTest/java/com/yubico/yubikit/testing/piv/PivTests.java
@@ -30,7 +30,8 @@
   PivTests.NoScpTests.class,
   PivTests.Scp11bTests.class,
   PivJcaProviderTests.NoScpTests.class,
-  PivJcaProviderTests.Scp11bTests.class
+  PivJcaProviderTests.Scp11bTests.class,
+  PivBioMultiProtocolTests.class
 })
 public class PivTests {
   public static class NoScpTests extends PivInstrumentedTests {

diff --git a/...integrationTest/java/com/yubico/yubikit/testing/desktop/piv/PivBioMultiProtocolTests.java b/...integrationTest/java/com/yubico/yubikit/testing/desktop/piv/PivBioMultiProtocolTests.java
@@ -15,13 +15,16 @@
  */
 package com.yubico.yubikit.testing.desktop.piv;
 
+import com.yubico.yubikit.testing.desktop.AlwaysManualTest;
 import com.yubico.yubikit.testing.desktop.framework.PivInstrumentedTests;
 import com.yubico.yubikit.testing.piv.PivBioMultiProtocolDeviceTests;
 import org.junit.Test;
+import org.junit.experimental.categories.Category;
 
 public class PivBioMultiProtocolTests extends PivInstrumentedTests {
 
   @Test
+  @Category(AlwaysManualTest.class)
   public void testAuthenticate() throws Throwable {
     withPivSession(PivBioMultiProtocolDeviceTests::testAuthenticate);
   }

diff --git a/...ing-desktop/src/integrationTest/java/com/yubico/yubikit/testing/desktop/piv/PivTests.java b/...ing-desktop/src/integrationTest/java/com/yubico/yubikit/testing/desktop/piv/PivTests.java
@@ -32,7 +32,8 @@
   PivTests.NoScpTests.class,
   PivTests.Scp11bTests.class,
   PivJcaProviderTests.NoScpTests.class,
-  PivJcaProviderTests.Scp11bTests.class
+  PivJcaProviderTests.Scp11bTests.class,
+  PivBioMultiProtocolTests.class
 })
 public class PivTests {
   public static class NoScpTests extends PivInstrumentedTests {

diff --git a/testing/src/main/java/com/yubico/yubikit/testing/piv/PivTestState.java b/testing/src/main/java/com/yubico/yubikit/testing/piv/PivTestState.java
@@ -88,9 +88,6 @@ protected PivTestState(Builder builder) throws Throwable {
 
     DeviceInfo deviceInfo = getDeviceInfo();
 
-    // skip MPE devices
-    assumeFalse("Ignoring MPE device", isMpe(deviceInfo));
-
     boolean isPivFipsCapable = isFipsCapable(deviceInfo, Capability.PIV);
     boolean hasPinComplexity = deviceInfo != null && deviceInfo.getPinComplexity();