fix for test case macro guards

[JacobBarthelmeh]

Fix for nightly-memusage Jenkins builds.

./configure --disable-shared --disable-rsa --disable-ecc --disable-dh --enable-curve25519 --enable-ed25519 --enable-tls13 && make

tests/api/test_tls_parse.c:1548:42: error: use of undeclared identifier 'ext'
1548 |             ExpectIntEQ(TLSX_Parse(sslp, ext, extLen, client_hello, suites),
    |                                          ^
tests/api/test_tls_parse.c:1548:47: error: use of undeclared identifier 'extLen'
1548 |             ExpectIntEQ(TLSX_Parse(sslp, ext, extLen, client_hello, suites),
    |                                               ^
CC       tests/api/unit_test-test_ossl_bn.o

[Copilot]

Warning

Copilot couldn't run its full agentic review because it didn't start before the timeout. Make sure your repository has a runner available, or add a copilot-code-review.yml file specifying one with the runs-on attribute. See the docs for more details.

Copilot review overview

Review effort: Lite
Findings: 1 · 1

Open (2)

• The ext/extLen declarations are guarded by a condition that includes an extra DH/Curve25519… · New
• The same “RSA or ECC available” predicate (!defined(NO_RSA) || defined(HAVE_ECC)) is now repeated… · New

What changed in this PR

Updates TLS parse unit tests’ preprocessor guards to avoid compilation failures in certain static-build feature combinations (e.g., nightly memusage Jenkins builds), primarily by tightening #if conditions and adjusting where shared test variables are declared.

Changes:

• Adds (!defined(NO_RSA) || defined(HAVE_ECC)) gating to several TLS 1.2/1.3 test blocks so they are excluded when neither RSA nor ECC is available.
• Moves ext / extLen declarations in test_TLSX_PointFormat_parse() behind a broader conditional so both TLS 1.2 and TLS 1.3 sections can reference them.
• Applies similar guard tightening to additional tests (e.g., supported versions parse, keyshare negotiate).

File	Description
tests/​api/​test_tls_parse.c	Adjusts compile-time guards and variable scoping in TLS parse tests to fix static-build configurations that previously failed to compile.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[github-actions]

MemBrowse Memory Report

No memory changes detected for:

• gcc-arm-cortex-m0plus
• gcc-arm-cortex-m3
• gcc-arm-cortex-m4
• gcc-arm-cortex-m4-baremetal
• gcc-arm-cortex-m4-crypto-only
• gcc-arm-cortex-m4-dtls13
• gcc-arm-cortex-m4-min-ecc
• gcc-arm-cortex-m4-openssl-compat
• gcc-arm-cortex-m4-pkcs7
• gcc-arm-cortex-m4-pq
• gcc-arm-cortex-m4-rsa-only
• gcc-arm-cortex-m4-sp-math
• gcc-arm-cortex-m4-tls12
• gcc-arm-cortex-m4-tls13
• gcc-arm-cortex-m7
• gcc-arm-cortex-m7-pq
• gcc-arm-cortex-m7-tls13
• linuxkm-pie
• linuxkm-standard
• stm32-sim-stm32h753