Skip to content

[stable3] fix(dav): support SEARCH request result in resultToNode - #1779

Merged
ShGKme merged 2 commits into
stable3from
backport/1778/stable3
Oct 21, 2026
Merged

ShGKme merged 2 commits into
stable3from
backport/1778/stable3

Conversation

@backportbot

@backportbot backportbot Bot commented Oct 7, 2026

Copy link
Copy Markdown

Backport of PR #1778

@backportbot
backportbot Bot requested review from ShGKme and skjnldsv October 7, 2026 19:38
@backportbot backportbot Bot added type: bug 🐛 Something isn't working 3. to review 3️⃣ Waiting for reviews labels Oct 7, 2026
Comment thread __tests__/dav/dav.spec.ts Outdated
@codecov

codecov Bot commented Oct 8, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.30%. Comparing base (4281acb) to head (816b8c6).

Additional details and impacted files
@@             Coverage Diff             @@
##           stable3    #1779      +/-   ##
===========================================
+ Coverage    90.25%   90.30%   +0.04%     
===========================================
  Files           23       23              
  Lines          657      660       +3     
  Branches       178      179       +1     
===========================================
+ Hits           593      596       +3     
  Misses          53       53              
  Partials        11       11              

☔ 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.

@ShGKme

ShGKme commented Oct 8, 2026

Copy link
Copy Markdown
Contributor

@susnux I made the test fix a bit outside the fixed bug instead of adjusting expected result to make it more real-like. Is that OK for a backport?

On v4 test setup was different, so there was no problem.

@skjnldsv

skjnldsv commented Oct 9, 2026

Copy link
Copy Markdown
Contributor

Is that OK for a backport?

ok from me 👍

Signed-off-by: Grigorii K. Shartsev <me@shgk.me>
Signed-off-by: Grigorii K. Shartsev <me@shgk.me>
@ShGKme

ShGKme commented Oct 20, 2026

Copy link
Copy Markdown
Contributor

Squashed fixup! commit.

@ShGKme
ShGKme force-pushed the backport/1778/stable3 branch from 5229e93 to 816b8c6 Compare October 20, 2026 13:31
@ShGKme
ShGKme enabled auto-merge October 20, 2026 13:31
@ShGKme
ShGKme disabled auto-merge October 21, 2026 13:23
@ShGKme

ShGKme commented Oct 21, 2026

Copy link
Copy Markdown
Contributor

I'm merging with bypassed requirements.
node | Expected — Waiting for status to be reported is invalid. CI passed successfully, but there is name mismatch with the settings from the main branch.

@ShGKme
ShGKme merged commit 5deaa40 into stable3 Oct 21, 2026
11 checks passed
@ShGKme
ShGKme deleted the backport/1778/stable3 branch October 21, 2026 13:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review 3️⃣ Waiting for reviews type: bug 🐛 Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants