Page MenuHomePhorge
Differential D215

Add kazv-io-manager-test.cpp
ClosedPublic
Actions

Authored by nannanko on May 24 2025, 6:21 PM.
Tags
None
Referenced Files
F5704159: D215.1755010455.diff
Mon, Aug 11, 7:54 AM
F5589846: D215.1754947966.diff
Sun, Aug 10, 2:32 PM
F5586825: D215.1754946655.diff
Sun, Aug 10, 2:10 PM
F5586789: D215.1754946641.diff
Sun, Aug 10, 2:10 PM
F5586281: D215.1754946184.diff
Sun, Aug 10, 2:03 PM
Subscribers
None

Details

Reviewers
tusooa
Group Reviewers
O1: the Kazv Project
Maniphest Tasks
T114: Add tests for kazv-io-manager and kazv-io-job
Commits
rK73a8eba421e6: Add kazv-io-manager-test.cpp
Summary

Add test for KazvIOManager

Type: skip

Test Plan

Verify unit tests pass.

Diff Detail

Repository
rK kazv
Branch
nannanko/kazv-io-manager-test
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 369
Build 721: GitLab CI for kazv
Build 720: arc lint + arc unit

Event Timeline

nannanko created this revision.May 24 2025, 6:21 PM
Owners added a reviewer: O1: the Kazv Project.May 24 2025, 6:21 PM
Herald added a reviewer: servant. · View Herald TranscriptMay 24 2025, 6:21 PM

Please add a changelog type to the summary, using the following format:

Type: (add|remove|skip|security|fix)

nannanko edited the summary of this revision. (Show Details)May 24 2025, 6:22 PM
nannanko removed a reviewer: servant.
Harbormaster completed remote builds in B369: Diff 603.May 24 2025, 6:40 PM
nannanko requested review of this revision.May 24 2025, 6:40 PM
nannanko added a task: T114: Add tests for kazv-io-manager and kazv-io-job.May 25 2025, 9:24 AM
tusooa requested changes to this revision.Jun 7 2025, 6:12 PM
tusooa added inline comments.
src/tests/kazv-io-manager-test.cpp
36–38

they should be static globals

This revision now requires changes to proceed.Jun 7 2025, 6:12 PM
nannanko updated this revision to Diff 621.Jun 7 2025, 7:25 PM

Make const variables static global

nannanko marked an inline comment as done.Jun 7 2025, 7:25 PM
Harbormaster completed remote builds in B383: Diff 621.Jun 7 2025, 7:41 PM
tusooa accepted this revision.Jun 18 2025, 4:28 PM
This revision is now accepted and ready to land.Jun 18 2025, 4:28 PM
Closed by commit rK73a8eba421e6: Add kazv-io-manager-test.cpp. · Explain WhyJun 21 2025, 7:08 PM
This revision was automatically updated to reflect the committed changes.
nannanko added a commit: rK73a8eba421e6: Add kazv-io-manager-test.cpp.

Revision Contents
Changeset List

Diff 603

View Options

src/tests/CMakeLists.txt

Loading...
View Options

src/tests/kazv-io-manager-test.cpp

Loading...