mirror of
https://github.com/Evolution-X/hardware_interfaces
synced 2026-02-01 16:50:18 +00:00
Merge "use getDescription in test failure message" into main am: 856abca8db am: 024387f2c9
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/2817881 Change-Id: Ica3ace0b1c8176771d0cb35a69f7d5183356caaf Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
This commit is contained in:
@@ -402,7 +402,7 @@ class CertificateRequestTestBase : public VtsRemotelyProvisionedComponentTests {
|
||||
for (auto& key : keysToSign_) {
|
||||
bytevec privateKeyBlob;
|
||||
auto status = provisionable_->generateEcdsaP256KeyPair(testMode, &key, &privateKeyBlob);
|
||||
ASSERT_TRUE(status.isOk()) << status.getMessage();
|
||||
ASSERT_TRUE(status.isOk()) << status.getDescription();
|
||||
|
||||
vector<uint8_t> payload_value;
|
||||
check_maced_pubkey(key, testMode, &payload_value);
|
||||
@@ -447,7 +447,7 @@ TEST_P(CertificateRequestTest, EmptyRequest_testMode) {
|
||||
auto status = provisionable_->generateCertificateRequest(
|
||||
testMode, {} /* keysToSign */, testEekChain_.chain, challenge_, &deviceInfo,
|
||||
&protectedData, &keysToSignMac);
|
||||
ASSERT_TRUE(status.isOk()) << status.getMessage();
|
||||
ASSERT_TRUE(status.isOk()) << status.getDescription();
|
||||
|
||||
auto result = verifyProductionProtectedData(
|
||||
deviceInfo, cppbor::Array(), keysToSignMac, protectedData, testEekChain_, eekId_,
|
||||
@@ -472,7 +472,7 @@ TEST_P(CertificateRequestTest, NewKeyPerCallInTestMode) {
|
||||
auto status = provisionable_->generateCertificateRequest(
|
||||
testMode, {} /* keysToSign */, testEekChain_.chain, challenge_, &deviceInfo,
|
||||
&protectedData, &keysToSignMac);
|
||||
ASSERT_TRUE(status.isOk()) << status.getMessage();
|
||||
ASSERT_TRUE(status.isOk()) << status.getDescription();
|
||||
|
||||
auto firstBcc = verifyProductionProtectedData(
|
||||
deviceInfo, /*keysToSign=*/cppbor::Array(), keysToSignMac, protectedData, testEekChain_,
|
||||
@@ -482,7 +482,7 @@ TEST_P(CertificateRequestTest, NewKeyPerCallInTestMode) {
|
||||
status = provisionable_->generateCertificateRequest(
|
||||
testMode, {} /* keysToSign */, testEekChain_.chain, challenge_, &deviceInfo,
|
||||
&protectedData, &keysToSignMac);
|
||||
ASSERT_TRUE(status.isOk()) << status.getMessage();
|
||||
ASSERT_TRUE(status.isOk()) << status.getDescription();
|
||||
|
||||
auto secondBcc = verifyProductionProtectedData(
|
||||
deviceInfo, /*keysToSign=*/cppbor::Array(), keysToSignMac, protectedData, testEekChain_,
|
||||
@@ -532,7 +532,7 @@ TEST_P(CertificateRequestTest, NonEmptyRequest_testMode) {
|
||||
auto status = provisionable_->generateCertificateRequest(
|
||||
testMode, keysToSign_, testEekChain_.chain, challenge_, &deviceInfo, &protectedData,
|
||||
&keysToSignMac);
|
||||
ASSERT_TRUE(status.isOk()) << status.getMessage();
|
||||
ASSERT_TRUE(status.isOk()) << status.getDescription();
|
||||
|
||||
auto result = verifyProductionProtectedData(
|
||||
deviceInfo, cborKeysToSign_, keysToSignMac, protectedData, testEekChain_, eekId_,
|
||||
@@ -576,7 +576,7 @@ TEST_P(CertificateRequestTest, NonEmptyRequestCorruptMac_testMode) {
|
||||
auto status = provisionable_->generateCertificateRequest(
|
||||
testMode, {keyWithCorruptMac}, testEekChain_.chain, challenge_, &deviceInfo,
|
||||
&protectedData, &keysToSignMac);
|
||||
ASSERT_FALSE(status.isOk()) << status.getMessage();
|
||||
ASSERT_FALSE(status.isOk()) << status.getDescription();
|
||||
EXPECT_EQ(status.getServiceSpecificError(), BnRemotelyProvisionedComponent::STATUS_INVALID_MAC);
|
||||
}
|
||||
|
||||
@@ -596,7 +596,7 @@ TEST_P(CertificateRequestTest, NonEmptyRequestCorruptMac_prodMode) {
|
||||
auto status = provisionable_->generateCertificateRequest(
|
||||
testMode, {keyWithCorruptMac}, getProdEekChain(rpcHardwareInfo.supportedEekCurve),
|
||||
challenge_, &deviceInfo, &protectedData, &keysToSignMac);
|
||||
ASSERT_FALSE(status.isOk()) << status.getMessage();
|
||||
ASSERT_FALSE(status.isOk()) << status.getDescription();
|
||||
EXPECT_EQ(status.getServiceSpecificError(), BnRemotelyProvisionedComponent::STATUS_INVALID_MAC);
|
||||
}
|
||||
|
||||
@@ -722,7 +722,7 @@ TEST_P(CertificateRequestV2Test, EmptyRequest) {
|
||||
auto challenge = randomBytes(size);
|
||||
auto status =
|
||||
provisionable_->generateCertificateRequestV2({} /* keysToSign */, challenge, &csr);
|
||||
ASSERT_TRUE(status.isOk()) << status.getMessage();
|
||||
ASSERT_TRUE(status.isOk()) << status.getDescription();
|
||||
|
||||
auto result = verifyProductionCsr(cppbor::Array(), csr, provisionable_.get(), challenge);
|
||||
ASSERT_TRUE(result) << result.message();
|
||||
@@ -743,7 +743,7 @@ TEST_P(CertificateRequestV2Test, NonEmptyRequest) {
|
||||
SCOPED_TRACE(testing::Message() << "challenge[" << size << "]");
|
||||
auto challenge = randomBytes(size);
|
||||
auto status = provisionable_->generateCertificateRequestV2(keysToSign_, challenge, &csr);
|
||||
ASSERT_TRUE(status.isOk()) << status.getMessage();
|
||||
ASSERT_TRUE(status.isOk()) << status.getDescription();
|
||||
|
||||
auto result = verifyProductionCsr(cborKeysToSign_, csr, provisionable_.get(), challenge);
|
||||
ASSERT_TRUE(result) << result.message();
|
||||
@@ -758,7 +758,7 @@ TEST_P(CertificateRequestV2Test, EmptyRequestWithInvalidChallengeFail) {
|
||||
|
||||
auto status = provisionable_->generateCertificateRequestV2(
|
||||
/* keysToSign */ {}, randomBytes(MAX_CHALLENGE_SIZE + 1), &csr);
|
||||
EXPECT_FALSE(status.isOk()) << status.getMessage();
|
||||
EXPECT_FALSE(status.isOk()) << status.getDescription();
|
||||
EXPECT_EQ(status.getServiceSpecificError(), BnRemotelyProvisionedComponent::STATUS_FAILED);
|
||||
}
|
||||
|
||||
@@ -773,13 +773,13 @@ TEST_P(CertificateRequestV2Test, NonEmptyRequestReproducible) {
|
||||
bytevec csr;
|
||||
|
||||
auto status = provisionable_->generateCertificateRequestV2(keysToSign_, challenge_, &csr);
|
||||
ASSERT_TRUE(status.isOk()) << status.getMessage();
|
||||
ASSERT_TRUE(status.isOk()) << status.getDescription();
|
||||
|
||||
auto firstCsr = verifyProductionCsr(cborKeysToSign_, csr, provisionable_.get(), challenge_);
|
||||
ASSERT_TRUE(firstCsr) << firstCsr.message();
|
||||
|
||||
status = provisionable_->generateCertificateRequestV2(keysToSign_, challenge_, &csr);
|
||||
ASSERT_TRUE(status.isOk()) << status.getMessage();
|
||||
ASSERT_TRUE(status.isOk()) << status.getDescription();
|
||||
|
||||
auto secondCsr = verifyProductionCsr(cborKeysToSign_, csr, provisionable_.get(), challenge_);
|
||||
ASSERT_TRUE(secondCsr) << secondCsr.message();
|
||||
@@ -797,7 +797,7 @@ TEST_P(CertificateRequestV2Test, NonEmptyRequestMultipleKeys) {
|
||||
bytevec csr;
|
||||
|
||||
auto status = provisionable_->generateCertificateRequestV2(keysToSign_, challenge_, &csr);
|
||||
ASSERT_TRUE(status.isOk()) << status.getMessage();
|
||||
ASSERT_TRUE(status.isOk()) << status.getDescription();
|
||||
|
||||
auto result = verifyProductionCsr(cborKeysToSign_, csr, provisionable_.get(), challenge_);
|
||||
ASSERT_TRUE(result) << result.message();
|
||||
@@ -815,7 +815,7 @@ TEST_P(CertificateRequestV2Test, NonEmptyRequestCorruptMac) {
|
||||
bytevec csr;
|
||||
auto status =
|
||||
provisionable_->generateCertificateRequestV2({keyWithCorruptMac}, challenge_, &csr);
|
||||
ASSERT_FALSE(status.isOk()) << status.getMessage();
|
||||
ASSERT_FALSE(status.isOk()) << status.getDescription();
|
||||
EXPECT_EQ(status.getServiceSpecificError(), BnRemotelyProvisionedComponent::STATUS_INVALID_MAC);
|
||||
}
|
||||
|
||||
@@ -829,7 +829,7 @@ TEST_P(CertificateRequestV2Test, CertificateRequestV1Removed_prodMode) {
|
||||
auto status = provisionable_->generateCertificateRequest(
|
||||
false /* testMode */, {} /* keysToSign */, {} /* EEK chain */, challenge_, &deviceInfo,
|
||||
&protectedData, &keysToSignMac);
|
||||
ASSERT_FALSE(status.isOk()) << status.getMessage();
|
||||
ASSERT_FALSE(status.isOk()) << status.getDescription();
|
||||
EXPECT_EQ(status.getServiceSpecificError(), BnRemotelyProvisionedComponent::STATUS_REMOVED);
|
||||
}
|
||||
|
||||
@@ -843,7 +843,7 @@ TEST_P(CertificateRequestV2Test, CertificateRequestV1Removed_testMode) {
|
||||
auto status = provisionable_->generateCertificateRequest(
|
||||
true /* testMode */, {} /* keysToSign */, {} /* EEK chain */, challenge_, &deviceInfo,
|
||||
&protectedData, &keysToSignMac);
|
||||
ASSERT_FALSE(status.isOk()) << status.getMessage();
|
||||
ASSERT_FALSE(status.isOk()) << status.getDescription();
|
||||
EXPECT_EQ(status.getServiceSpecificError(), BnRemotelyProvisionedComponent::STATUS_REMOVED);
|
||||
}
|
||||
|
||||
@@ -927,7 +927,7 @@ TEST_P(CertificateRequestV2Test, DeviceInfo) {
|
||||
bytevec csr;
|
||||
irpcStatus =
|
||||
provisionable_->generateCertificateRequestV2({} /* keysToSign */, challenge_, &csr);
|
||||
ASSERT_TRUE(irpcStatus.isOk()) << irpcStatus.getMessage();
|
||||
ASSERT_TRUE(irpcStatus.isOk()) << irpcStatus.getDescription();
|
||||
|
||||
auto result = verifyProductionCsr(cppbor::Array(), csr, provisionable_.get(), challenge_);
|
||||
ASSERT_TRUE(result) << result.message();
|
||||
|
||||
Reference in New Issue
Block a user