* Re: [PATCH] UEFI-SCT: SctPkg: Replaced the incorrect references RANDOM_NAME_PROTOCOL to RANDOM_NUMBER_PROTOCOL
2023-01-31 17:24 ` [PATCH] UEFI-SCT: SctPkg: Replaced the incorrect references RANDOM_NAME_PROTOCOL to RANDOM_NUMBER_PROTOCOL Sunny Wang
@ 2023-01-31 17:56 ` G Edhaya Chandran
0 siblings, 0 replies; 2+ messages in thread
From: G Edhaya Chandran @ 2023-01-31 17:56 UTC (permalink / raw)
To: Sunny Wang, devel@edk2.groups.io
Cc: Barton Gao, Carolyn Gjertsen, Samer El-Haj-Mahmoud, Sam Kaynor,
Rebecca Cran
Thank you Sunny.
The solution is upstreamed through the commit: https://github.com/tianocore/edk2-test/commit/56093929e99cadab5cd95f560d67fe61663d67d4
With Warm Regards,
Edhay
> -----Original Message-----
> From: Sunny Wang <Sunny.Wang@arm.com>
> Sent: 31 January 2023 22:55
> To: G Edhaya Chandran <Edhaya.Chandran@arm.com>;
> devel@edk2.groups.io
> Cc: G Edhaya Chandran <Edhaya.Chandran@arm.com>; Barton Gao
> <gaojie@byosoft.com.cn>; Carolyn Gjertsen <Carolyn.Gjertsen@amd.com>;
> Samer El-Haj-Mahmoud <Samer.El-Haj-Mahmoud@arm.com>; Sam Kaynor
> <Sam.Kaynor@arm.com>; Sunny Wang <Sunny.Wang@arm.com>
> Subject: RE: [PATCH] UEFI-SCT: SctPkg: Replaced the incorrect references
> RANDOM_NAME_PROTOCOL to RANDOM_NUMBER_PROTOCOL
>
> Looks good to me. Thank for addressing my comments in another email
> thread, Edhaya.
> Reviewed-by: Sunny Wang <sunny.wang@arm.com> Cc Sam for his
> awareness.
>
> -----Original Message-----
> From: G Edhaya Chandran <edhaya.chandran@arm.com>
> Sent: 31 January 2023 17:04
> To: devel@edk2.groups.io
> Cc: G Edhaya Chandran <Edhaya.Chandran@arm.com>; Barton Gao
> <gaojie@byosoft.com.cn>; Carolyn Gjertsen <Carolyn.Gjertsen@amd.com>;
> Samer El-Haj-Mahmoud <Samer.El-Haj-Mahmoud@arm.com>; Sunny Wang
> <Sunny.Wang@arm.com>
> Subject: [PATCH] UEFI-SCT: SctPkg: Replaced the incorrect references
> RANDOM_NAME_PROTOCOL to RANDOM_NUMBER_PROTOCOL
>
> From: edhay <edhaya.chandran@arm.com>
>
> Replaced the incorrect references RANDOM_NAME_PROTOCOL to
> RANDOM_NUMBER_PROTOCOL
> 9 instances were replaced
>
> Please find more details in the ticket:
> https://bugzilla.tianocore.org/show_bug.cgi?id=4328
>
> Cc: Barton Gao <gaojie@byosoft.com.cn>
> Cc: Carolyn Gjertsen <Carolyn.Gjertsen@amd.com>
> Cc: Samer El-Haj-Mahmoud <samer.el-haj-mahmoud@arm.com>
> Cc: Sunny Wang <sunny.wang@arm.com>
>
> Signed-off-by: G Edhaya Chandran<edhaya.chandran@arm.com>
> ---
> .../BlackBoxTest/RandomNumberBBTestConformance.c | 10 +++++-----
> .../BlackBoxTest/RandomNumberBBTestFunction.c | 8 ++++----
> 2 files changed, 9 insertions(+), 9 deletions(-)
>
> diff --git a/uefi-
> sct/SctPkg/TestCase/UEFI/EFI/Protocol/RandomNumber/BlackBoxTest/Rand
> omNumberBBTestConformance.c b/uefi-
> sct/SctPkg/TestCase/UEFI/EFI/Protocol/RandomNumber/BlackBoxTest/Rand
> omNumberBBTestConformance.c
> index 47e7800f..5e55f639 100644
> --- a/uefi-
> sct/SctPkg/TestCase/UEFI/EFI/Protocol/RandomNumber/BlackBoxTest/Rand
> omNumberBBTestConformance.c
> +++ b/uefi-
> sct/SctPkg/TestCase/UEFI/EFI/Protocol/RandomNumber/BlackBoxTe
> +++ st/RandomNumberBBTestConformance.c
> @@ -223,7 +223,7 @@ BBTestGetRNGConformanceTestCheckpoint1 (
> StandardLib,
> EFI_TEST_ASSERTION_FAILED,
> gTestGenericFailureGuid,
> - L"RANDOM_NAME_PROTOCOL.GetInfo - GetInfo() is not
> supported or EFI_DEVICE_ERROR",
> + L"RANDOM_NUMBER_PROTOCOL.GetInfo - GetInfo() is not
> + supported or EFI_DEVICE_ERROR",
> L"%a:%d: Status - %r",
> __FILE__,
> (UINTN)__LINE__,
> @@ -303,7 +303,7 @@ BBTestGetRNGConformanceTestCheckpoint1 (
> StandardLib,
> AssertionType,
> gConformanceTestAssertionGuid002,
> - L"RANDOM_NAME_PROTOCOL.GetRNG - GetRNG() returns
> EFI_INVALID_PARAMETER when RNGValueLength is 0",
> + L"RANDOM_NUMBER_PROTOCOL.GetRNG - GetRNG() returns
> + EFI_INVALID_PARAMETER when RNGValueLength is 0",
> L"%a:%d: Status - %r",
> __FILE__,
> (UINTN)__LINE__,
> @@ -326,7 +326,7 @@ BBTestGetRNGConformanceTestCheckpoint1 (
> StandardLib,
> AssertionType,
> gConformanceTestAssertionGuid003,
> - L"RANDOM_NAME_PROTOCOL.GetRNG - GetRNG() returns
> EFI_INVALID_PARAMETER when RNGValue is NULL",
> + L"RANDOM_NUMBER_PROTOCOL.GetRNG - GetRNG() returns
> + EFI_INVALID_PARAMETER when RNGValue is NULL",
> L"%a:%d: Status - %r",
> __FILE__,
> (UINTN)__LINE__,
> @@ -352,7 +352,7 @@ BBTestGetRNGConformanceTestCheckpoint1 (
> StandardLib,
> AssertionType,
> gConformanceTestAssertionGuid004,
> - L"RANDOM_NAME_PROTOCOL.GetRNG - GetRNG() returns
> EFI_INVALID_PARAMETER when RNGValueLength is 0",
> + L"RANDOM_NUMBER_PROTOCOL.GetRNG - GetRNG() returns
> + EFI_INVALID_PARAMETER when RNGValueLength is 0",
> L"%a:%d: Status - %r",
> __FILE__,
> (UINTN)__LINE__,
> @@ -375,7 +375,7 @@ BBTestGetRNGConformanceTestCheckpoint1 (
> StandardLib,
> AssertionType,
> gConformanceTestAssertionGuid005,
> - L"RANDOM_NAME_PROTOCOL.GetRNG - GetRNG() returns
> EFI_INVALID_PARAMETER when RNGValue is NULL",
> + L"RANDOM_NUMBER_PROTOCOL.GetRNG - GetRNG() returns
> + EFI_INVALID_PARAMETER when RNGValue is NULL",
> L"%a:%d: Status - %r",
> __FILE__,
> (UINTN)__LINE__,
> diff --git a/uefi-
> sct/SctPkg/TestCase/UEFI/EFI/Protocol/RandomNumber/BlackBoxTest/Rand
> omNumberBBTestFunction.c b/uefi-
> sct/SctPkg/TestCase/UEFI/EFI/Protocol/RandomNumber/BlackBoxTest/Rand
> omNumberBBTestFunction.c
> index df879e4c..23c8e730 100644
> --- a/uefi-
> sct/SctPkg/TestCase/UEFI/EFI/Protocol/RandomNumber/BlackBoxTest/Rand
> omNumberBBTestFunction.c
> +++ b/uefi-
> sct/SctPkg/TestCase/UEFI/EFI/Protocol/RandomNumber/BlackBoxTe
> +++ st/RandomNumberBBTestFunction.c
> @@ -248,7 +248,7 @@ BBTestGetInfoFunctionTestCheckpoint1 (
> StandardLib,
> AssertionType,
> gFunctionTestAssertionGuid002,
> - L"RANDOM_NAME_PROTOCOL.GetInfo - GetInfo() returns
> EFI_SUCCESS with valid parameters",
> + L"RANDOM_NUMBER_PROTOCOL.GetInfo - GetInfo() returns
> + EFI_SUCCESS with valid parameters",
> L"%a:%d: Status - %r, RNGAlgorithmListSize - %d",
> __FILE__,
> (UINTN)__LINE__,
> @@ -271,7 +271,7 @@ BBTestGetInfoFunctionTestCheckpoint1 (
> StandardLib,
> AssertionType,
> gFunctionTestAssertionGuid003,
> - L"RANDOM_NAME_PROTOCOL.GetInfo - GetInfo() should returns
> valid algo",
> + L"RANDOM_NUMBER_PROTOCOL.GetInfo - GetInfo() should
> + returns valid algo",
> L"%a:%d:",
> __FILE__,
> (UINTN)__LINE__
> @@ -316,7 +316,7 @@ BBTestGetRNGFunctionTestCheckpoint1 (
> StandardLib,
> EFI_TEST_ASSERTION_FAILED,
> gTestGenericFailureGuid,
> - L"RANDOM_NAME_PROTOCOL.GetInfo - GetInfo() is not
> supported or EFI_DEVICE_ERROR",
> + L"RANDOM_NUMBER_PROTOCOL.GetInfo - GetInfo() is not
> + supported or EFI_DEVICE_ERROR",
> L"%a:%d: Status - %r",
> __FILE__,
> (UINTN)__LINE__,
> @@ -400,7 +400,7 @@ BBTestGetRNGFunctionTestCheckpoint1 (
> StandardLib,
> AssertionType,
> gFunctionTestAssertionGuid004,
> - L"RANDOM_NAME_PROTOCOL.GetRNG - GetRNG() should
> returns EFI_SUCCESS with valid parameters",
> + L"RANDOM_NUMBER_PROTOCOL.GetRNG - GetRNG() should
> + returns EFI_SUCCESS with valid parameters",
> L"%a:%d: Status - %r",
> __FILE__,
> (UINTN)__LINE__,
> --
> 2.17.1
IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.
^ permalink raw reply [flat|nested] 2+ messages in thread