PSA TESTS: Include mbedtls/config.h before evaluating MBEDTLS_PSA_CRYPTO_C

[urutva]

Description

Include mbedtls/config.h before evaluating MBEDTLS_PSA_CRYPTO_C
Signed-off-by: Devaraj Ranganna [email protected]

Fixes #10888 (comment)

Pull request type

[x] Fix
[ ] Refactor
[ ] Target update
[ ] Functionality change
[ ] Docs update
[ ] Test update
[ ] Breaking change

Reviewers

@Patater @jeromecoutant

Release Notes

[ciarmcom]

@Devran01, thank you for your changes.
@Patater @ARMmbed/mbed-os-crypto @ARMmbed/mbed-os-test @ARMmbed/mbed-os-maintainers please review.

[gilles-peskine-arm]

Why these files but not other similar files?

$ git grep -F -l 'crypto.h' |xargs grep -L -F 'mbedtls/config.h'
psa/crypto_access_control/COMPONENT_PSA_SRV_IPC/test_partition_proxy.h
psa/crypto_access_control/COMPONENT_SPE/test_partition.c
psa/entropy_inject/main.cpp

And do it this way rather than include psa/crypto.h?

[urutva]

@gilles-peskine-arm These tests were skipped because of the change I did with #10888. Basically removing MBEDTLS_PSA_CRYPTO_C from PSA targets. These tests evaluate MBEDTLS_PSA_CRYPTO_C without including mbedtls/config.h.

I wasn't aware that psa/crypto.h includes mbedtls/config.h. Including psa/crypto.h is a cleaner solution. I've updated the tests.

[artokin]

CI started

[mbed-ci]

Test run: FAILED

Summary: 1 of 4 test jobs failed
Build number : 1
Build artifacts

Failed test jobs:

• jenkins-ci/mbed-os-ci_greentea-test

[0xc0170]

Restarted CI test (looks like CI deamon fialure)

[0xc0170]

Restarted again after deamon error

cc @ARMmbed/mbed-os-test

[0xc0170]

Failed again with the same error, @ARMmbed/mbed-os-test notified

[0xc0170]

Still valid issue in CI, will restart once fixed

[0xc0170]

CI restarted

[mbed-ci]

Test run: SUCCESS

Summary: 4 of 4 test jobs passed
Build number : 2
Build artifacts