Skip to content

Add tests and simplify permission handling logic #337

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.

Sign up for GitHub

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

Jump to bottom
Open
udirctera wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Add tests and simplify permission handling logic #337

udirctera wants to merge 2 commits into from

Conversation

@udirctera
Copy link
Collaborator

@udirctera udirctera commented Feb 11, 2026

Description

This pull request focuses on adding unit tests for strict permission errors in file and directory operations. It also simplifies the permission error handling logic in the following ways:

  • Adds tests for handling completed_with_warnings in file operations.
  • Removes the strict permission argument from directory creation commands in _parents_generator.
  • Adds a test for strict permission directory creation at root paths.

@udirctera
SDK-348
c9bd7a0 
- Add unit test for handling `completed_with_warnings` in file move operations
- Simplify `_parents_generator` logic, removing strict permission argument from directory creation commands
- Add unit test for strict permission directory creation at root paths
@udirctera
SDK-348
32e76a0 
Add tests for handling strict permission errors during file operations and simplify permission error handling logic
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@saimonation saimonation Awaiting requested review from saimonation

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@udirctera