Page MenuHomePhorge
Differential D215

Add kazv-io-manager-test.cpp
Needs ReviewPublic
Actions

Authored by nannanko on Sat, May 24, 6:21 PM.

Details

Reviewers
tusooa
Group Reviewers
O1: the Kazv Project
Maniphest Tasks
T114: Add tests for kazv-io-manager and kazv-io-job
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.Sat, May 24, 6:21 PM
Owners added a reviewer: O1: the Kazv Project.Sat, May 24, 6:21 PM
Herald added a reviewer: servant. · View Herald TranscriptSat, May 24, 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)Sat, May 24, 6:22 PM
nannanko removed a reviewer: servant.
Harbormaster completed remote builds in B369: Diff 603.Sat, May 24, 6:40 PM
nannanko requested review of this revision.Sat, May 24, 6:40 PM
nannanko added a task: T114: Add tests for kazv-io-manager and kazv-io-job.Sun, May 25, 9:24 AM
tusooa requested changes to this revision.Sat, Jun 7, 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.Sat, Jun 7, 6:12 PM
nannanko updated this revision to Diff 621.Sat, Jun 7, 7:25 PM

Make const variables static global

nannanko marked an inline comment as done.Sat, Jun 7, 7:25 PM
Harbormaster completed remote builds in B383: Diff 621.Sat, Jun 7, 7:41 PM

Revision Contents
Changeset List

Diff 621

View Options

src/tests/CMakeLists.txt

Loading...
View Options

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

Loading...