Skip to content

Commit e391733

Browse files
committed
tests: refactor cert import tests
1 parent 27a0ead commit e391733

File tree

1 file changed

+15
-26
lines changed

1 file changed

+15
-26
lines changed

Yubico.YubiKey/tests/integration/Yubico/YubiKey/Piv/ImportTests.cs

Lines changed: 15 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -99,15 +99,10 @@ public void KeyAndCertImport(
9999
var testDevice = IntegrationTestDeviceEnumeration.GetTestDevice(testDeviceType);
100100
Assert.True(testDevice.EnabledUsbCapabilities.HasFlag(YubiKeyCapabilities.Piv));
101101

102-
using var pivSession = new PivSession(testDevice);
103-
var isValid = PivSupport.ResetPiv(pivSession);
104-
Assert.True(isValid);
105-
106-
var collectorObj = new Simple39KeyCollector();
107-
pivSession.KeyCollector = collectorObj.Simple39KeyCollectorDelegate;
102+
using var pivSession = GetSession(testDevice);
108103

109104
var testPrivateKey = TestKeys.GetTestPrivateKey(keyType);
110-
var testCert =TestKeys.GetTestCertificate(keyType);
105+
var testCert = TestKeys.GetTestCertificate(keyType);
111106
var privateKey = AsnPrivateKeyDecoder.CreatePrivateKey(testPrivateKey.EncodedKey);
112107

113108
pivSession.ImportPrivateKey(0x90, privateKey);
@@ -129,17 +124,14 @@ public void CertImport(
129124
var testDevice = IntegrationTestDeviceEnumeration.GetTestDevice(testDeviceType);
130125
Assert.True(testDevice.EnabledUsbCapabilities.HasFlag(YubiKeyCapabilities.Piv));
131126

132-
var isValid = SampleKeyPairs.GetMatchingKeyAndCert(keyType, out var cert, out var _);
133-
Assert.True(isValid);
134-
135-
using var pivSession = new PivSession(testDevice);
136-
var collectorObj = new Simple39KeyCollector();
137-
pivSession.KeyCollector = collectorObj.Simple39KeyCollectorDelegate;
127+
var testCertificate = TestKeys.GetTestCertificate(keyType);
128+
var testX509Certificate = testCertificate.AsX509Certificate2();
138129

139-
pivSession.ImportCertificate(0x90, cert!);
130+
using var pivSession = GetSession(testDevice);
131+
pivSession.ImportCertificate(0x90, testX509Certificate);
140132

141-
var getCert = pivSession.GetCertificate(0x90);
142-
Assert.True(getCert.Equals(cert));
133+
var resultCert = pivSession.GetCertificate(0x90);
134+
Assert.True(resultCert.Equals(testX509Certificate));
143135
}
144136

145137
[SkippableTheory(typeof(NotSupportedException), typeof(DeviceNotFoundException))]
@@ -155,19 +147,16 @@ public void ImportCompressedCert(KeyType keyType, StandardTestDevice testDeviceT
155147
var testDevice = IntegrationTestDeviceEnumeration.GetTestDevice(testDeviceType);
156148
Assert.True(testDevice.EnabledUsbCapabilities.HasFlag(YubiKeyCapabilities.Piv));
157149

158-
var isValid = SampleKeyPairs.GetMatchingKeyAndCert(keyType, out var cert, out var _);
159-
Assert.True(isValid);
160-
161-
using var pivSession = new PivSession(testDevice);
162-
var collectorObj = new Simple39KeyCollector();
163-
pivSession.KeyCollector = collectorObj.Simple39KeyCollectorDelegate;
150+
var testCertificate = TestKeys.GetTestCertificate(keyType);
151+
var testX509Certificate = testCertificate.AsX509Certificate2();
164152

165-
pivSession.ImportCertificate(0x91, cert!, true);
153+
using var pivSession = GetSession(testDevice);
154+
pivSession.ImportCertificate(0x91, testX509Certificate, true);
166155

167-
var getCert = pivSession.GetCertificate(0x91);
168-
Assert.True(getCert.Equals(cert));
156+
var resultCert = pivSession.GetCertificate(0x91);
157+
Assert.True(resultCert.Equals(testX509Certificate));
169158
}
170-
159+
171160
private static PivSession GetSession(
172161
IYubiKeyDevice testDevice)
173162
{

0 commit comments

Comments
 (0)