Skip to content

Handle null path segments in CombinePath - #97

Merged
jochenwezel merged 1 commit into
mainfrom
codex/95-null-path-segments
Oct 11, 2026
Merged

jochenwezel merged 1 commit into
mainfrom
codex/95-null-path-segments

Conversation

@jochenwezel

Copy link
Copy Markdown
Member

Summary

  • ignore null path segments and a null parameter array in BaseDmsProvider.CombinePath
  • document the public null-segment behavior
  • cover the provider contract on .NET 8 and .NET Framework 4.8
  • cover DMS browser startup with an initial root and no selected child folder

Fixes #95

Verification

  • BaseDmsProviderPathTest.CombinePathIgnoresNullSegments on
    et8.0
  • BaseDmsProviderPathTest.CombinePathIgnoresNullSegments on
    et48
  • all seven DmsBrowserStartupTest.InitialTreeAndFilesShareOnlyTheirOwnOperationSnapshot cases on
    et8.0-windows, including selectedPath = Nothing
  • local Bierdeckel Management PROD FHW-AM diagnostic build prepared with the corrected assemblies for direct user verification

@github-actions

Copy link
Copy Markdown

Isolated Test Statistics (ubuntu-latest)

548 tests  +1   548 ✅ +1   14s ⏱️ ±0s
  1 suites ±0     0 💤 ±0 
  1 files   ±0     0 ❌ ±0 

Results for commit 07cac40. ± Comparison against base commit 6bb3257.

@github-actions

Copy link
Copy Markdown

Isolated Test Statistics (macos-latest)

548 tests  +1   548 ✅ +1   12s ⏱️ -2s
  1 suites ±0     0 💤 ±0 
  1 files   ±0     0 ❌ ±0 

Results for commit 07cac40. ± Comparison against base commit 6bb3257.

@github-actions

Copy link
Copy Markdown

Scopevisio Teamwork Test Statistics (ubuntu-latest)

20 tests  ±0   20 ✅ ±0   1m 59s ⏱️ ±0s
 1 suites ±0    0 💤 ±0 
 1 files   ±0    0 ❌ ±0 

Results for commit 07cac40. ± Comparison against base commit 6bb3257.

@github-actions

Copy link
Copy Markdown

WebDAV Test Statistics (ubuntu-latest)

16 tests  ±0   16 ✅ ±0   2m 2s ⏱️ +2s
 1 suites ±0    0 💤 ±0 
 1 files   ±0    0 ❌ ±0 

Results for commit 07cac40. ± Comparison against base commit 6bb3257.

@github-actions

Copy link
Copy Markdown

OwnCloud Test Statistics (macos-latest)

16 tests  ±0   16 ✅ ±0   2m 2s ⏱️ -1s
 1 suites ±0    0 💤 ±0 
 1 files   ±0    0 ❌ ±0 

Results for commit 07cac40. ± Comparison against base commit 6bb3257.

@github-actions

Copy link
Copy Markdown

Nextcloud Test Statistics (macos-latest)

16 tests  ±0   16 ✅ ±0   2m 3s ⏱️ ±0s
 1 suites ±0    0 💤 ±0 
 1 files   ±0    0 ❌ ±0 

Results for commit 07cac40. ± Comparison against base commit 6bb3257.

@github-actions

Copy link
Copy Markdown

Isolated Test Statistics (windows-latest)

548 tests  +1   548 ✅ +1   8s ⏱️ ±0s
  1 suites ±0     0 💤 ±0 
  1 files   ±0     0 ❌ ±0 

Results for commit 07cac40. ± Comparison against base commit 6bb3257.

@github-actions

Copy link
Copy Markdown

Scopevisio Teamwork Test Statistics (macos-latest)

20 tests  ±0   20 ✅ ±0   1m 59s ⏱️ ±0s
 1 suites ±0    0 💤 ±0 
 1 files   ±0    0 ❌ ±0 

Results for commit 07cac40. ± Comparison against base commit 6bb3257.

@github-actions

Copy link
Copy Markdown

Nextcloud Test Statistics (ubuntu-latest)

16 tests  ±0   16 ✅ ±0   2m 3s ⏱️ ±0s
 1 suites ±0    0 💤 ±0 
 1 files   ±0    0 ❌ ±0 

Results for commit 07cac40. ± Comparison against base commit 6bb3257.

@github-actions

Copy link
Copy Markdown

WebDAV Test Statistics (windows-latest)

16 tests  ±0   16 ✅ ±0   2m 0s ⏱️ -1s
 1 suites ±0    0 💤 ±0 
 1 files   ±0    0 ❌ ±0 

Results for commit 07cac40. ± Comparison against base commit 6bb3257.

@github-actions

Copy link
Copy Markdown

OwnCloud Test Statistics (windows-latest)

16 tests  ±0   16 ✅ ±0   2m 0s ⏱️ ±0s
 1 suites ±0    0 💤 ±0 
 1 files   ±0    0 ❌ ±0 

Results for commit 07cac40. ± Comparison against base commit 6bb3257.

@github-actions

Copy link
Copy Markdown

OwnCloud Test Statistics (ubuntu-latest)

16 tests  ±0   16 ✅ ±0   2m 0s ⏱️ -1s
 1 suites ±0    0 💤 ±0 
 1 files   ±0    0 ❌ ±0 

Results for commit 07cac40. ± Comparison against base commit 6bb3257.

@github-actions

Copy link
Copy Markdown

WebDAV Test Statistics (macos-latest)

16 tests  ±0   16 ✅ ±0   2m 1s ⏱️ -1s
 1 suites ±0    0 💤 ±0 
 1 files   ±0    0 ❌ ±0 

Results for commit 07cac40. ± Comparison against base commit 6bb3257.

@github-actions

Copy link
Copy Markdown

Nextcloud Test Statistics (windows-latest)

16 tests  ±0   16 ✅ ±0   2m 2s ⏱️ ±0s
 1 suites ±0    0 💤 ±0 
 1 files   ±0    0 ❌ ±0 

Results for commit 07cac40. ± Comparison against base commit 6bb3257.

@github-actions

Copy link
Copy Markdown

Scopevisio Teamwork Test Statistics (windows-latest)

20 tests  ±0   20 ✅ ±0   1m 59s ⏱️ ±0s
 1 suites ±0    0 💤 ±0 
 1 files   ±0    0 ❌ ±0 

Results for commit 07cac40. ± Comparison against base commit 6bb3257.

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.

Allow null path segments when opening a DMS collection root

1 participant