Skip to content

Exclude she keywrap interop test from non-posix targets#477

Merged
bigbrett merged 1 commit into
wolfSSL:mainfrom
AlexLanzano:she-keywrap-test-fix
Jul 22, 2026
Merged

Exclude she keywrap interop test from non-posix targets#477
bigbrett merged 1 commit into
wolfSSL:mainfrom
AlexLanzano:she-keywrap-test-fix

Conversation

@AlexLanzano

Copy link
Copy Markdown
Member

No description provided.

@AlexLanzano
AlexLanzano requested a review from Copilot July 22, 2026 15:55

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Note

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.

Updates SHE↔keywrap interop tests to avoid running/compiling on unsupported (non-POSIX) targets by tightening preprocessor guards.

Changes:

  • Narrowed SHE↔keywrap interop test compilation to POSIX builds with both client and server enabled.
  • Adjusted shared KEK ID definition/comment near the top of the file.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread test/wh_test_she.c
Comment thread test/wh_test_she.c Outdated
@bigbrett
bigbrett merged commit 1bfd791 into wolfSSL:main Jul 22, 2026
108 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants