[PR #75] [MERGED] test: add tests for #70 #215

New issue

Closed

opened 2026-03-15 11:53:56 +03:00 by kerem · 0 comments

kerem commented

2026-03-15 11:53:56 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/awslabs/iam-policy-autopilot/pull/75
Author: @mschlaipfer
Created: 12/11/2025
Status: ✅ Merged
Merged: 12/11/2025
Merged by: @mschlaipfer

Base: main ← Head: fixes-66-tests

📝 Commits (1)

c53c80f test: add tests for #70

📊 Changes

1 file changed (+107 additions, -43 deletions)

View changed files

📝 iam-policy-autopilot-policy-generation/src/enrichment/resource_matcher.rs (+107 -43)

📄 Description

Issue #, if available: https://github.com/awslabs/iam-policy-autopilot/issues/66

Description of changes: I somehow dropped the unit tests I'd added when I opened https://github.com/awslabs/iam-policy-autopilot/pull/70. Adding them here, and cleans up some duplication in the unit tests of resource_matcher.rs

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/awslabs/iam-policy-autopilot/pull/75 **Author:** [@mschlaipfer](https://github.com/mschlaipfer) **Created:** 12/11/2025 **Status:** ✅ Merged **Merged:** 12/11/2025 **Merged by:** [@mschlaipfer](https://github.com/mschlaipfer) **Base:** `main` ← **Head:** `fixes-66-tests` --- ### 📝 Commits (1) - [`c53c80f`](https://github.com/awslabs/iam-policy-autopilot/commit/c53c80ff3e191daa3af2154ba6a4bf944ffd920d) test: add tests for #70 ### 📊 Changes **1 file changed** (+107 additions, -43 deletions) <details> <summary>View changed files</summary> 📝 `iam-policy-autopilot-policy-generation/src/enrichment/resource_matcher.rs` (+107 -43) </details> ### 📄 Description *Issue #, if available:* https://github.com/awslabs/iam-policy-autopilot/issues/66 *Description of changes:* I somehow dropped the unit tests I'd added when I opened https://github.com/awslabs/iam-policy-autopilot/pull/70. Adding them here, and cleans up some duplication in the unit tests of `resource_matcher.rs` By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>