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
F73521062: D215.1776980837.diff
Wed, Apr 22, 2:47 PM
F73473870: D215.1776976216.diff
Wed, Apr 22, 1:30 PM
F73465248: D215.1776975370.diff
Wed, Apr 22, 1:16 PM
F73161306: D215.1776948258.diff
Wed, Apr 22, 5:44 AM
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 383
Build 749: GitLab CI for kazv
Build 748: 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
37–39

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 621

View Options

src/tests/CMakeLists.txt

Loading...
View Options

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

Loading...