Skip to content

Commit 6b31d51

Browse files
committed
freertos-integration-tests: Remove out-of-tree patch
* Remove upstreamed out of tree patch `0003-Introduce-user-defined-MQTT-keep-alive-value.patch`. * With the latest version of FreeRTOS Integration tests, OTA PAL test has been removed. Therefore, remove OTA PAL test specific changes from integration layer including the patch 0002-closefile-validsignature-test-Fix-test-bugs.patch. Signed-off-by: Devaraj Ranganna <[email protected]>
1 parent ce68c6d commit 6b31d51

File tree

11 files changed

+6
-248
lines changed

11 files changed

+6
-248
lines changed

applications/freertos_iot_libraries_tests/CMakeLists.txt

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -79,7 +79,6 @@ target_link_libraries(freertos-libraries-integration-tests-config
7979
coremqtt
8080
helpers-logging
8181
freertos-libraries-integration-tests-mqtt
82-
freertos-libraries-integration-tests-ota-pal
8382
freertos-libraries-integration-tests-qualification
8483
freertos-libraries-integration-tests-transport
8584
unity

applications/freertos_iot_libraries_tests/integration_tests_platform_function.c

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,6 @@
1919

2020
#include "transport_interface_test.h"
2121
#include "qualification_test.h"
22-
#include "ota_pal_test.h"
2322
#include "mqtt_test.h"
2423

2524
#include "demo_config.h"
@@ -128,11 +127,6 @@ void SetupMqttTestParam( MqttTestParam_t * pTestParam )
128127
}
129128
}
130129

131-
void SetupOtaPalTestParam( OtaPalTestParam_t * pTestParam )
132-
{
133-
pTestParam->pageSize = 4096;
134-
}
135-
136130
typedef struct Task_t
137131
{
138132
TaskHandle_t handle;

applications/freertos_iot_libraries_tests/test_execution_config.h

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -55,11 +55,6 @@
5555
*/
5656
#define TRANSPORT_INTERFACE_TEST_ENABLED ( 1 )
5757

58-
/**
59-
* @brief Configuration to enable the OTA PAL test.
60-
*/
61-
#define OTA_PAL_TEST_ENABLED ( 1 )
62-
6358
/**
6459
* @brief Configuration to enable the OTA End-to-end test.
6560
*/

applications/freertos_iot_libraries_tests/test_param_config.h

Lines changed: 0 additions & 29 deletions
Original file line numberDiff line numberDiff line change
@@ -186,35 +186,6 @@
186186
* #define TEST_START_DELAY_MS 5000
187187
*/
188188

189-
#define OTA_RSA_SHA1 1
190-
#define OTA_RSA_SHA256 2
191-
#define OTA_ECDSA_SHA256 3
192-
193-
/**
194-
* @brief Certificate type for OTA PAL test.
195-
* Valid options are: OTA_RSA_SHA1, OTA_RSA_SHA256, OTA_ECDSA_SHA256.
196-
*/
197-
#define OTA_PAL_TEST_CERT_TYPE OTA_RSA_SHA256
198-
199-
/**
200-
* @brief Path to cert for OTA test PAL. Used to verify signature.
201-
* If applicable, the device must be pre-provisioned with this certificate. Please see
202-
* test/common/ota/test_files for the set of certificates.
203-
*/
204-
#define OTA_PAL_CERTIFICATE_FILE "ecdsa-sha256-signer.crt.pem"
205-
206-
/**
207-
* @brief Some devices have a hard-coded name for the firmware image to boot.
208-
*/
209-
#define OTA_PAL_FIRMWARE_FILE "non_secure image"
210-
211-
/**
212-
* @brief Some boards OTA PAL layers will use the file names passed into it for the
213-
* image and the certificates because their non-volatile memory is abstracted by a
214-
* file system. Set this to 1 if that is the case for your device.
215-
*/
216-
#define OTA_PAL_USE_FILE_SYSTEM 0
217-
218189
/**
219190
* @brief The PKCS #11 supports RSA key function.
220191
*

applications/helpers/provisioning/CMakeLists.txt

Lines changed: 4 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -23,17 +23,10 @@ else()
2323
fri-bsp
2424
)
2525

26-
if(FREERTOS_LIBRARIES_INTEGRATION_TESTS EQUAL 1)
27-
set(
28-
CODE_SIGNING_PUBLIC_KEY_PEM_PATH
29-
${freertos_libraries_integration_tests_SOURCE_DIR}/src/ota/test_files/tfm-rsa-sha256-signer.crt.pem
30-
)
31-
else()
32-
set(
33-
CODE_SIGNING_PUBLIC_KEY_PEM_PATH
34-
${AWS_OTA_SIGNATURE_PUBLIC_KEY_PATH}
35-
)
36-
endif()
26+
set(
27+
CODE_SIGNING_PUBLIC_KEY_PEM_PATH
28+
${AWS_OTA_SIGNATURE_PUBLIC_KEY_PATH}
29+
)
3730

3831
add_custom_command(OUTPUT ${CMAKE_CURRENT_BINARY_DIR}/aws_clientcredential_keys.h
3932
DEPENDS ${AWS_CLIENT_PRIVATE_KEY_PEM_PATH}

components/tools/freertos_libraries_integration_tests/CMakeLists.txt

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,8 +13,6 @@ include(ApplyPatches)
1313
set(PATCH_FILES_DIRECTORY "${CMAKE_CURRENT_LIST_DIR}/integration/patches")
1414
set(PATCH_FILES
1515
"${PATCH_FILES_DIRECTORY}/0001-Reduce-the-TRANSPORT_TEST_BUFFER_WRITABLE_LENGTH.patch"
16-
"${PATCH_FILES_DIRECTORY}/0002-closefile-validsignature-test-Fix-test-bugs.patch"
17-
"${PATCH_FILES_DIRECTORY}/0003-Introduce-user-defined-MQTT-keep-alive-value.patch"
1816
)
1917
iot_reference_arm_corstone3xx_apply_patches("${freertos_libraries_integration_tests_SOURCE_DIR}" "${PATCH_FILES}")
2018

components/tools/freertos_libraries_integration_tests/integration/CMakeLists.txt

Lines changed: 0 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,6 @@ add_library(freertos-libraries-integration-tests-config INTERFACE)
77
include(${freertos_libraries_integration_tests_SOURCE_DIR}/qualification_test.cmake)
88
include(${freertos_libraries_integration_tests_SOURCE_DIR}/src/mqtt_test.cmake)
99
include(${freertos_libraries_integration_tests_SOURCE_DIR}/src/pkcs11_test.cmake)
10-
include(${freertos_libraries_integration_tests_SOURCE_DIR}/src/ota_pal_test.cmake)
1110
include(${freertos_libraries_integration_tests_SOURCE_DIR}/src/transport_interface_test.cmake)
1211

1312
add_library(freertos-libraries-integration-tests-mqtt
@@ -31,20 +30,6 @@ target_compile_definitions(freertos-libraries-integration-tests-mqtt
3130
PRIVATE
3231
MQTT_KEEP_ALIVE_INTERVAL_SECONDS=60
3332
)
34-
add_library(freertos-libraries-integration-tests-ota-pal
35-
${OTA_PAL_TEST_SOURCES}
36-
)
37-
target_include_directories(freertos-libraries-integration-tests-ota-pal
38-
PUBLIC
39-
${OTA_PAL_TEST_INCLUDE_DIRS}
40-
)
41-
target_link_libraries(freertos-libraries-integration-tests-ota-pal
42-
PRIVATE
43-
freertos-libraries-integration-tests-config
44-
freertos-ota-pal-psa
45-
ota-for-aws-iot-embedded-sdk
46-
unity
47-
)
4833

4934
add_library(freertos-libraries-integration-tests-pkcs11
5035
${PKCS11_TEST_SOURCES}
@@ -82,7 +67,6 @@ target_link_libraries(freertos-libraries-integration-tests-qualification
8267
coremqtt
8368
freertos-libraries-integration-tests-config
8469
freertos-libraries-integration-tests-mqtt
85-
freertos-libraries-integration-tests-ota-pal
8670
freertos-libraries-integration-tests-pkcs11
8771
)
8872

components/tools/freertos_libraries_integration_tests/integration/patches/0002-closefile-validsignature-test-Fix-test-bugs.patch

Lines changed: 0 additions & 140 deletions
This file was deleted.

components/tools/freertos_libraries_integration_tests/integration/patches/0003-Introduce-user-defined-MQTT-keep-alive-value.patch

Lines changed: 0 additions & 36 deletions
This file was deleted.

0 commit comments

Comments
 (0)