Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add not worth mentioning initial skeleton for testing using pytest #37

Open
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

yarikoptic
Copy link

@yarikoptic yarikoptic commented Mar 30, 2023

Would not test much if at all, but at least would try to install/import across all OSes and Python versions.

ref #1

edit: I also pushed as master to my clone so that action is executed... can be observed on https://github.com/yarikoptic/refuse/actions .

@yarikoptic yarikoptic mentioned this pull request Mar 30, 2023
@@ -32,6 +32,7 @@
# +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

import os
import sys
Copy link
Author

Choose a reason for hiding this comment

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

congratulate on the first bug fix contributed due to initiating the testing ;)

run: |
$url = 'https://github.com/winfsp/winfsp/releases/download/v2.0/winfsp-2.0.23075.msi'
Invoke-WebRequest -Uri $url -OutFile 'winfsp.msi'
Start-Process msiexec.exe -ArgumentList '/i', 'winfsp.msi', '/quiet', '/norestart' -Wait
Copy link
Author

Choose a reason for hiding this comment

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

well -- either low is not intended for Windows at all or smth else need to be done since we get ERROR tests/test_low.py - OSError: Unable to find libfuse . See https://github.com/yarikoptic/refuse/actions/runs/4559603821/jobs/8043726950 for more detail

if: startsWith(matrix.os, 'macos')
run: |
brew update
brew install osxfuse
Copy link
Author

Choose a reason for hiding this comment

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

attempt to run tests with this seems to just stall on macos-11 without any feedback: see https://github.com/yarikoptic/refuse/actions/runs/4559603821/jobs/8043727829 (still running ,will check if outputs anything tomorrow)

.github/workflows/test.yml Outdated Show resolved Hide resolved
@yarikoptic
Copy link
Author

both windows and OSX runs aren't happy so I will disable them for now since it annoys me to receive all the failing notifications emails ;-)

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