test(node): add missing owner_did field to fails-closed seed update

[xdev00]

Summary

Adds the owner_did field missing from one ReceivedRefUpdate test initializer, unbreaking compilation of test targets on main (and CI on release PR #182).

Motivation & context

#113 and #145 raced: #145 added owner_did to ReceivedRefUpdate and updated its own call sites, but the seed initializer in list_repo_events_fails_closed_when_repo_lookup_errors (added by #113) predates it. Merged main fails clippy --all-targets, cargo test, and MSRV with E0063; build --release passes, which is why this slipped through. Blocks #182.

Kind of change

• Tests / CI

What changed

• gitlawb-node: set owner_did: None in the fails-closed test's seed update — None on purpose, the test exercises the legacy None-branch slug fallback.

How a reviewer can verify

cargo clippy --workspace --all-targets -- -D warnings   # fails on main, clean here
cargo test --workspace                                   # 985 passed locally w/ Postgres

Before you request review

- [x] Scope is one logical change; no unrelated churn
- [x] cargo test --workspace passes locally
- [x] New behavior is covered by tests (required for fixes)
- [x] cargo fmt --all and cargo clippy --workspace --all-targets -- -D warnings are clean
- [x] Commit titles use Conventional Commits (feat(...), fix(...), docs(...))
- [x] Docs / .env.example updated if behavior or config changed (or N/A)
- [x] Checked existing PRs so this isn't a duplicate

<!-- This is an auto-generated comment: release notes by coderabbit.ai -->

## Summary by CodeRabbit

* **Tests**
  * Improved test coverage for repository event handling when repository lookups fail.
  * Clarified test data by explicitly representing missing owner information.

<!-- end of auto-generated comment: release notes by coderabbit.ai -->

[github-actions]

Thanks for the contribution. A couple of things will help us review this faster:

• This changes Rust source but no tests changed. Tests are required for fixes and strongly encouraged for features.

See CONTRIBUTING.md. Update the PR and these notes will clear automatically.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: c90b3a4c-9296-494d-b8d7-2948233628f6

📥 Commits

Reviewing files that changed from the base of the PR and between 98d728c and a2e5392.

📒 Files selected for processing (1)

• crates/gitlawb-node/src/test_support.rs

---

📝 Walkthrough

Walkthrough

The repository event failure test now explicitly sets owner_did: None when constructing its ReceivedRefUpdate fixture.

Changes

Ref-update failure test

Layer / File(s)	Summary
Explicit owner field in failure test crates/gitlawb-node/src/test_support.rs	The ReceivedRefUpdate fixture in list_repo_events_fails_closed_when_repo_lookup_errors now sets owner_did to None.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

• Gitlawb/node#145: Adds and propagates the optional owner_did field on ref-update data.

Suggested labels: crate:node, kind:bug, subsystem:peers

Suggested reviewers: beardthelion

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title is concise and accurately describes the main change: adding the missing owner_did field in a test seed update.
Description check	✅ Passed	The description covers the required sections and includes context, change details, and verification steps, so it is mostly complete.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/owner-did-test-compile

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}