quicksilver / Quicksilver

Quicksilver Project Source
http://qsapp.com
Apache License 2.0
2.72k stars 283 forks source link

[Bug]: Tests fail due to using `~/Library/Application Support` #2953

Closed n8henrie closed 10 months ago

n8henrie commented 10 months ago

Before submitting your bug report, please confirm you have completed the following steps

Bug description

Continuation of some work from https://github.com/quicksilver/Quicksilver/issues/2583

Some tests -- specifically testAddingFileObjectToPasteboard in my case -- fail for my locally because they refer to QSObjects that have already been cached somewhere in ~/Library/Application Support. This error will likely not show up in CI because it will be using an empty directory there regardless.

When running tests, having the tests use a fake directory for App Support helps ensure reproducibility.

Steps to reproduce

Run the testAddingFileObjectToPasteboard while you already have an object for /Applications/Safari in your ~/Library/Application Support directory

Expected behavior

Tests should pass

MacOS Version

None

Quicksilver Version

No response

Relevant Plugins

No response

Crash Logs or Spindump

No response

Screenshots

No response

Additional info

No response