Skip to content

Add winfsp-tests package for VirtIO-FS WinFsp coverage. - #34

Open
abrajput1 wants to merge 1 commit into
HCK-CI:mainfrom
abrajput1:winfsp-tests-package
Open

Add winfsp-tests package for VirtIO-FS WinFsp coverage.#34
abrajput1 wants to merge 1 commit into
HCK-CI:mainfrom
abrajput1:winfsp-tests-package

Conversation

@abrajput1

Copy link
Copy Markdown

Installs winfsp-tests-x64.exe into WinFsp\bin so functest winfsp_test can run the upstream WinFsp suite against the virtiofs mount.

Setup (install_args) on the client:

  1. Expand winfsp-tests-1.6.20027.zip into a temp directory
  2. Ensure C:\Program Files (x86)\WinFsp\bin exists
  3. Copy winfsp-tests-x64.exe into that WinFsp bin directory

Comment thread winfsp-tests/config.json Outdated
"download_url": "https://github.com/billziss-gh/winfsp/releases/download/v1.6/winfsp-tests-1.6.20027.zip",
"file_name": "winfsp-tests-1.6.20027.zip",
"install_cmd": "powershell",
"install_args": "-ExecutionPolicy Bypass -Command \"Expand-Archive -Force -Path '@sw_path@\\@file_name@' -DestinationPath '@temp@\\winfsp-tests'; New-Item -ItemType Directory -Force -Path 'C:\\Program Files (x86)\\WinFsp\\bin' | Out-Null; Copy-Item -Force '@temp@\\winfsp-tests\\winfsp-tests-x64.exe' 'C:\\Program Files (x86)\\WinFsp\\bin\\winfsp-tests-x64.exe'\"",

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Too many commands in one line

Please add a simple ps1 script

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks — moved the install steps into install.ps1

Installs winfsp-tests-x64.exe into WinFsp\bin so functest winfsp_test
can run the upstream WinFsp suite against the virtiofs mount.

Signed-off-by: Abhishek Rajput <abrajput@redhat.com>
@abrajput1
abrajput1 force-pushed the winfsp-tests-package branch from 355b9bc to 0dbc747 Compare August 5, 2026 11:16
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