Skip to content

Scope ServiceVerbTests to a fixture-local GVFS service - #2059

Merged
tyrielv merged 1 commit into
masterfrom
tyrielv/test-registry-scoping
Jul 15, 2026
Merged

Scope ServiceVerbTests to a fixture-local GVFS service#2059
tyrielv merged 1 commit into
masterfrom
tyrielv/test-registry-scoping

Conversation

@tyrielv

@tyrielv tyrielv commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

Purpose

Tests for gvfs service --mount-all and gvfs service --unmount-all currently can be flaky or cause flakiness in other tests because the service is system-scope. This PR changes the service scope to be a single test fixture while running functional tests.

Summary

  • Run ServiceVerbTests against a dedicated service name per fixture instance instead of the shared test service
  • Thread optional service-name overrides through GVFSHelpers, GVFSProcess, and GVFSFunctionalTestEnlistment so clone/mount/unmount/register/service verbs all hit the same fixture-local registry
  • Add overloads in GVFSServiceProcess to install/start/stop/uninstall a named test service while keeping existing defaults intact

Validation

  • dotnet build GVFS/GVFS.FunctionalTests/GVFS.FunctionalTests.csproj -c Debug
  • attempted targeted functional test run of ServiceVerbTests.ServiceCommandsWithNoRepos (blocked locally by existing ProgramData access denial while cleaning Test.GVFS.Service logs before tests start)

@tyrielv
tyrielv marked this pull request as ready for review July 10, 2026 18:15
@tyrielv
tyrielv enabled auto-merge July 10, 2026 18:15
Run ServiceVerbTests against a dedicated GVFS service name so --mount-all/--unmount-all only touch repos created in that fixture.

This wires service-name overrides through functional-test helpers and adds fixture-local service lifecycle setup/teardown for ServiceVerbTests.

Assisted-by: GPT-5.3-Codex
Signed-off-by: Tyrie Vella <tyrielv@gmail.com>
@tyrielv
tyrielv force-pushed the tyrielv/test-registry-scoping branch from 47683e8 to f034ec7 Compare July 10, 2026 18:17
Comment thread GVFS/GVFS.FunctionalTests/Tools/GVFSServiceProcess.cs
@tyrielv
tyrielv merged commit 2e50a46 into master Jul 15, 2026
39 checks passed
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.

2 participants