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 E2E test for unlink and unlinkat #3094

Open
t4lz opened this issue Feb 15, 2025 · 1 comment
Open

Add E2E test for unlink and unlinkat #3094

t4lz opened this issue Feb 15, 2025 · 1 comment
Assignees
Labels

Comments

@t4lz
Copy link
Member

t4lz commented Feb 15, 2025

There are hooks for unlink and unlinkat, as far as I can see those hooks are not tested anywhere.
We should add an E2E test (or add them to an existing test, doesn't matter), and test for each of the two:

  • calling on a remote path (i.e. a path that is configured to be written remotely)
  • calling on a relative path (should be bypassed)
  • calling on a local path (i.e. a path that is supposed to be bypassed by mirrord)
  • calling on a mapped path (i.e. a path that is configured to be mapped to a different path)
Copy link

linear bot commented Feb 15, 2025

@eyalb181 eyalb181 added the tests label Feb 17, 2025 — with Linear
@gememma gememma self-assigned this Feb 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants