Skip to content

Adjust filter tests by using explicit file-local VFS object#407

Merged
eddelbuettel merged 1 commit intomasterfrom
de/sc-17654/refactor_filter_tests
May 10, 2022
Merged

Adjust filter tests by using explicit file-local VFS object#407
eddelbuettel merged 1 commit intomasterfrom
de/sc-17654/refactor_filter_tests

Conversation

@eddelbuettel
Copy link
Copy Markdown
Contributor

@eddelbuettel eddelbuettel commented May 9, 2022

This PR adjust the new Filter tests from #406 by using a file-local VFS object. This appears to help against sporadic failures that are seen with the package-environment-global VFS (using the same mechanism we use for Ctx and Config objects).

No code changes outside this adjustment to one test file.

@eddelbuettel eddelbuettel requested a review from ihnorton May 9, 2022 20:13
@shortcut-integration
Copy link
Copy Markdown

This pull request has been linked to Shortcut Story #17654: Refactor filter tests from PR 406.

@eddelbuettel eddelbuettel merged commit 47903fb into master May 10, 2022
@eddelbuettel eddelbuettel deleted the de/sc-17654/refactor_filter_tests branch May 11, 2022 16:37
eddelbuettel added a commit that referenced this pull request May 14, 2022
* pause a part of the filter test file for now

* renamed check to schema_check, set deprecation message

* remove temporary adjustment following #407 merge
@eddelbuettel eddelbuettel mentioned this pull request May 16, 2022
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