Skip to content

Migrate IAM SSH keys tests#13772

Open
pinzon wants to merge 1 commit intoiam/moto-migrationfrom
cris/unc-225
Open

Migrate IAM SSH keys tests#13772
pinzon wants to merge 1 commit intoiam/moto-migrationfrom
cris/unc-225

Conversation

@pinzon
Copy link
Member

@pinzon pinzon commented Feb 13, 2026

Motivation

This PR migrates the tests related to IAM SSH Keys tests from Moto into LocalStack

Changes

Source

File: moto-repo/tests/test_iam/test_iam.py

Destination

File: localstack/tests/aws/services/iam/test_iam_ssh_public_keys.py

Test Mapping

# Moto Test Line LocalStack Test Description
1 test_upload_ssh_public_key 2336 test_upload_ssh_public_key Tests uploading an SSH public key
2 test_get_ssh_public_key 2355 test_get_ssh_public_key Tests retrieving an SSH public key
3 test_get_ssh_public_key 2355 test_get_ssh_public_key_not_found Tests error for non-existent key
4 test_list_ssh_public_keys 2377 test_list_ssh_public_keys Tests listing keys (empty and populated)
5 test_update_ssh_public_key 2396 test_update_ssh_public_key Tests updating key status
6 test_update_ssh_public_key 2396 test_update_ssh_public_key_not_found Tests error for non-existent key
7 test_delete_ssh_public_key 2423 test_delete_ssh_public_key Tests deleting a key
8 test_delete_ssh_public_key 2423 test_delete_ssh_public_key_not_found Tests error for non-existent key

@pinzon pinzon added semver: patch Non-breaking changes which can be included in patch releases docs: skip Pull request does not require documentation changes notes: skip Pull request does not have to be mentioned in the release notes labels Feb 13, 2026
@pinzon pinzon marked this pull request as ready for review February 13, 2026 19:47
@pinzon pinzon requested a review from dfangl as a code owner February 13, 2026 19:47
@github-actions
Copy link

LocalStack Community integration with Pro

  2 files  ±    0    2 suites  ±0   1m 2s ⏱️ - 2h 3m 9s
425 tests  - 5 009  223 ✅  - 4 661  202 💤  - 348  0 ❌ ±0 
427 runs   - 5 009  223 ✅  - 4 661  204 💤  - 348  0 ❌ ±0 

Results for commit cbfa8c4. ± Comparison against base commit cdcd4df.

@github-actions
Copy link

Test Results - Preflight, Unit

23 099 tests   - 12   21 237 ✅  - 12   6m 35s ⏱️ +20s
     1 suites ± 0    1 862 💤 ± 0 
     1 files   ± 0        0 ❌ ± 0 

Results for commit cbfa8c4. ± Comparison against base commit cdcd4df.

@github-actions
Copy link

Test Results (amd64) - Acceptance

7 tests  ±0   5 ✅ ±0   2m 46s ⏱️ -15s
1 suites ±0   2 💤 ±0 
1 files   ±0   0 ❌ ±0 

Results for commit cbfa8c4. ± Comparison against base commit cdcd4df.

@github-actions
Copy link

Test Results (amd64) - Integration, Bootstrap

  5 files  ±    0    5 suites  ±0   9m 40s ⏱️ - 2h 29m 37s
449 tests  - 5 390  247 ✅  - 4 886  202 💤  - 504  0 ❌ ±0 
455 runs   - 5 390  247 ✅  - 4 886  208 💤  - 504  0 ❌ ±0 

Results for commit cbfa8c4. ± Comparison against base commit cdcd4df.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs: skip Pull request does not require documentation changes notes: skip Pull request does not have to be mentioned in the release notes semver: patch Non-breaking changes which can be included in patch releases

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant