fsprojects / fsprojects/Paket

[repo-assist] scriptgen integration tests fail due to unpinned NuGet versions in scenario fixtures

Open
#4,399 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

automation bug needs investigation repo-assist
Dominant language
F#
Stars
2.1k
Forks
528
Avg merge
1d 12m
Merged PRs (30d)
54

Description

🤖 This is an automated response from Repo Assist.

The TestCategory=scriptgen integration tests in LoadingScriptGenerationTests.fs currently fail when run against a live NuGet feed, as seen in CI attempts for PR #4396.

Root cause: The scenario fixtures under integrationtests/scenarios/loading-scripts/*/before/paket.dependencies (e.g. simple-dependencies, mscorlib) use unpinned version ranges (nuget NUnit ~> 2, nuget log4net ~> 1, etc.) and ship no paket.lock. When paket install runs, it resolves whatever the latest matching package versions are on nuget.org today. Several of those packages have since changed their lib/ folder layout (NUnit, log4net, Microsoft.Rest.ClientRuntime.Azure), so the generated load-script logic can't find expected DLL paths (DirectoryNotFoundException), and the tests' hard-coded expected file lists no longer match reality.

This is pre-existing test fixture rot, unrelated to any recent Paket code change — the same failures reproduce on old commits with a fresh package resolution.

Suggested fix: pin exact package versions (or add a maintained paket.lock) for the affected loading-scripts scenarios, and refresh the expected script/file assertions to match current package contents.

I've disabled the newly-added linux-integration-scriptgen CI job in PR #4396's follow-up so CI is green again; re-enable it once these fixtures are repaired.
Related to #4396

Generated by 🌈 Repo Assist, see workflow run. Learn more.
Comment /repo-assist to run again

Add this agentic workflow to your repo

To install this agentic workflow, run

gh aw add githubnext/agentics/workflows/repo-assist.md@ae8d551f07c7ed7619f8c58c7bb4c3ac89395d38

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with the TestCategory=scriptgen tests in LoadingScriptGenerationTests.fs and inspect the paket.dependencies files under integrationtests/scenarios/loading-scripts/*/before, especially simple-dependencies and mscorlib. Run the integration tests against the live feed to identify affected package contents. Done means the fixtures use reproducible package versions, expected script/file assertions match them, and the disabled linux-integration-scriptgen CI job can be re-enabled.

Written by the indexing model from the issue text.

Assessment

Tech stack
fsharp
Domain
build-system, ci-cd, testing-qa
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Active
Clarity
Mostly clear
Newbie friendliness
65/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.