Skip to content

Fix ls method on non folder and method calls at root - #12

Merged
lmignon merged 4 commits into
mainfrom
master-lmi-fixes
Jul 2, 2026
Merged

Fix ls method on non folder and method calls at root#12
lmignon merged 4 commits into
mainfrom
master-lmi-fixes

Conversation

@lmignon

@lmignon lmignon commented Jul 1, 2026

Copy link
Copy Markdown
Member

No description provided.

lmignon and others added 2 commits July 1, 2026 15:55
if i try to leave the fs_storage.directory_path empty, those lines are in error
@codecov-commenter

codecov-commenter commented Jul 1, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 10.00000% with 18 lines in your changes missing coverage. Please review.
✅ Project coverage is 33.09%. Comparing base (e4a3926) to head (754205c).

Files with missing lines Patch % Lines
src/msgraphfs/core.py 10.00% 18 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #12      +/-   ##
==========================================
- Coverage   33.22%   33.09%   -0.14%     
==========================================
  Files           2        2              
  Lines         963      970       +7     
  Branches      167      168       +1     
==========================================
+ Hits          320      321       +1     
- Misses        623      629       +6     
  Partials       20       20              

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

…vel path handling

- Implemented split_parent_child function to handle path splitting.
- Added regression tests for creating files and directories at the root level.
- Included tests for copying and moving files to top-level paths without slashes.

These changes complements #11
@lmignon
lmignon force-pushed the master-lmi-fixes branch from 8ad5279 to 754205c Compare July 2, 2026 06:56
@lmignon
lmignon merged commit a8e61f3 into main Jul 2, 2026
8 checks passed
@lmignon
lmignon deleted the master-lmi-fixes branch July 2, 2026 07:23
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.

3 participants