Skip to content

Add tests for environment helpers - #2935

Closed
abrarshivani wants to merge 1 commit into
mainfrom
devin/1789778916-sample-utils-tests
Closed

abrarshivani wants to merge 1 commit into
mainfrom
devin/1789778916-sample-utils-tests

Conversation

@abrarshivani

Copy link
Copy Markdown
Contributor

Description

Add focused unit coverage for path-list prepending and environment variable replacement. The cases verify ordering, no-op behavior, and exact key matching so similarly prefixed variables remain untouched.

Checklist

  • No secrets, sensitive information, or unrelated changes
  • Lint checks passing (make lint)
  • Generated assets in-sync (make validate-generated-assets)
  • Go mod artifacts in-sync (make validate-modules)
  • Test cases are added for new code paths

Testing

  • make fmt
  • Unit tests pending

Link to Devin session: https://nvidia-cloud.devinenterprise.com/sessions/202583d9612e47d987e4d776b7aa9225
Open in Devin Desktop: https://nvidia-cloud.devinenterprise.com/desktop/session/202583d9612e47d987e4d776b7aa9225?variant=devin
Requested by: @abrarshivani

Signed-off-by: Abrar Shivani <ashivani@nvidia.com>
Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
@devin-ai-integration

Copy link
Copy Markdown

I'll fix CI failures and address comments from users with write access. I'll skip comments containing "(aside)".

  • Disable automatic comment, CI, and merge conflict monitoring

@copy-pr-bot

copy-pr-bot Bot commented Sep 19, 2026

Copy link
Copy Markdown

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

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.

1 participant