Skip to content

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

Open
yarikoptic wants to merge 11 commits into
pleiszenburg:masterfrom
yarikoptic:enh-pytest
Open

Add not worth mentioning initial skeleton for testing using pytest#37
yarikoptic wants to merge 11 commits into
pleiszenburg:masterfrom
yarikoptic:enh-pytest

Conversation

@yarikoptic

@yarikoptic yarikoptic commented Mar 30, 2023

Copy link
Copy Markdown

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
Comment thread src/refuse/_refactor.py
# +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

import os
import sys

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.

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
Copy Markdown
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
Copy Markdown
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)

Comment thread .github/workflows/test.yml Outdated
@yarikoptic

Copy link
Copy Markdown
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