Skip to content

api-tests: crypto: add multipart hash capacity test c081 - #422

Open
arthurcassegrain wants to merge 1 commit into
ARM-software:mainfrom
arthurcassegrain:features/add-parallel-hash-test
Open

api-tests: crypto: add multipart hash capacity test c081#422
arthurcassegrain wants to merge 1 commit into
ARM-software:mainfrom
arthurcassegrain:features/add-parallel-hash-test

Conversation

@arthurcassegrain

Copy link
Copy Markdown

Add crypto test c081 to verify multipart hash operation capacity.

The new test opens the configured number of parallel hash operations, checks that one additional setup fails, then aborts the active operations and verifies that a new operation can be created again.

Register the new test as c081 in testsuite.db.

Add crypto test c081 to verify multipart hash operation capacity.

The new test opens the configured number of parallel hash operations,
checks that one additional setup fails, then aborts the active
operations and verifies that a new operation can be created again.

Register the new test as c081 in testsuite.db.

Signed-off-by: Arthur Cassegrain <arthur.cassegrain@arm.com>
@avinaw01-arm avinaw01-arm self-assigned this Jul 3, 2026
@avinaw01-arm

Copy link
Copy Markdown
Member

Hi @arthurcassegrain

I noticed that a similar scenario is already covered in test_c011 (psa_hash_setup_test). The compliance result you're aiming to achieve with the proposed changes could be addressed by adding a new data-set or helper function within the existing test-case. Given that, I don't see a strong need to introduce a separate dedicated test-case for this.

Additionally, I reviewed the Specifications and couldn't find any requirement for an explicit test-case for this feature. Based on that, I would recommend extending the existing test-case, provided it aligns well with the intended coverage. Thanks.

Best Regards,
Avi.

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.

2 participants