[PR #86] [MERGED] fix: update service reference user agent for integration tests #227

Closed
opened 2026-03-15 11:54:29 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/awslabs/iam-policy-autopilot/pull/86
Author: @karanjitsingh
Created: 12/17/2025
Status: Merged
Merged: 12/19/2025
Merged by: @karanjitsingh

Base: mainHead: main


📝 Commits (6)

📊 Changes

4 files changed (+16 additions, -2 deletions)

View changed files

📝 iam-policy-autopilot-cli/Cargo.toml (+1 -0)
📝 iam-policy-autopilot-mcp-server/Cargo.toml (+1 -0)
📝 iam-policy-autopilot-policy-generation/Cargo.toml (+2 -0)
📝 iam-policy-autopilot-policy-generation/src/enrichment/service_reference.rs (+12 -2)

📄 Description

Description of changes:

Use a separate user agent when calling for service reference during integration tests.

  • Creates and env variable in .cargo/cargo.toml - this env var is picked up when running in the workspace through cargo run / cargo test
  • Updates pr checks

Update: Change introduces a feature integ-test in iam-policy-autopilot-policy-generation for integration tests. All other crates containing integration tests use this feature in their dev-dependencies

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/86 **Author:** [@karanjitsingh](https://github.com/karanjitsingh) **Created:** 12/17/2025 **Status:** ✅ Merged **Merged:** 12/19/2025 **Merged by:** [@karanjitsingh](https://github.com/karanjitsingh) **Base:** `main` ← **Head:** `main` --- ### 📝 Commits (6) - [`7f8216a`](https://github.com/awslabs/iam-policy-autopilot/commit/7f8216a809b217a1ab4617128efac733f3353a49) fix: fix user agent for integration tests - [`b503c42`](https://github.com/awslabs/iam-policy-autopilot/commit/b503c42caae146993603591013e2e61dcd13eebc) fix: user agent format - [`da09b15`](https://github.com/awslabs/iam-policy-autopilot/commit/da09b156a944af77a96203ae359be40188a3aec3) fix: use cfg:feature - [`9f2f521`](https://github.com/awslabs/iam-policy-autopilot/commit/9f2f52134de80b646c6a0ea01e0db43918f9d0ce) fix: Remove stale env var - [`435b983`](https://github.com/awslabs/iam-policy-autopilot/commit/435b983a99494fc7d23f3fc558d15009fa91da79) fix: cargo fmt - [`8035690`](https://github.com/awslabs/iam-policy-autopilot/commit/80356908b4edc96d82c07ef2c901164ed4fc5e62) Merge branch 'main' into main ### 📊 Changes **4 files changed** (+16 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `iam-policy-autopilot-cli/Cargo.toml` (+1 -0) 📝 `iam-policy-autopilot-mcp-server/Cargo.toml` (+1 -0) 📝 `iam-policy-autopilot-policy-generation/Cargo.toml` (+2 -0) 📝 `iam-policy-autopilot-policy-generation/src/enrichment/service_reference.rs` (+12 -2) </details> ### 📄 Description *Description of changes:* ~Use a separate user agent when calling for service reference during integration tests.~ - ~Creates and env variable in `.cargo/cargo.toml` - this env var is picked up when running in the workspace through `cargo run` / `cargo test`~ - ~Updates pr checks~ Update: Change introduces a feature `integ-test` in `iam-policy-autopilot-policy-generation` for integration tests. All other crates containing integration tests use this feature in their `dev-dependencies` 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>
kerem 2026-03-15 11:54:29 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/iam-policy-autopilot#227
No description provided.