Skip to content

Crypto: Add sign/verify and encrypt/decrypt tests#120903

Merged
Repiteo merged 1 commit into
godotengine:masterfrom
Faless:crypto/more_module_tests
Jul 4, 2026
Merged

Crypto: Add sign/verify and encrypt/decrypt tests#120903
Repiteo merged 1 commit into
godotengine:masterfrom
Faless:crypto/more_module_tests

Conversation

@Faless

@Faless Faless commented Jul 3, 2026

Copy link
Copy Markdown
Collaborator

What problem(s) does this PR solve?

We had no tests for Crypto encrypt/decrypt/sign/verify

Additional information

I used these test to validate #120725 .

@Calinou Calinou left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Code looks good to me.

Comment thread modules/mbedtls/tests/test_crypto_mbedtls.h Outdated
@Faless Faless force-pushed the crypto/more_module_tests branch from 155b0b0 to 8b3827a Compare July 4, 2026 06:27

@akien-mga akien-mga left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks great!

@akien-mga akien-mga modified the milestones: 4.x, 4.8 Jul 4, 2026
@Repiteo Repiteo merged commit 6688271 into godotengine:master Jul 4, 2026
20 checks passed
@Repiteo

Repiteo commented Jul 4, 2026

Copy link
Copy Markdown
Contributor

Thanks!

@Faless Faless deleted the crypto/more_module_tests branch July 4, 2026 16:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants