Page MenuHomePhorge
Differential D210

Add kazv-io-job-test
Changes PlannedPublic
Actions

Authored by nannanko on Mar 14 2025, 6:04 AM.
Tags
None
Referenced Files
F308057: D210.1745364400.diff
Mon, Apr 21, 4:26 PM
F308056: D210.1745364399.diff
Mon, Apr 21, 4:26 PM
F308055: D210.1745364398.diff
Mon, Apr 21, 4:26 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
Summary

Add tests for kazv-io-job.cpp

Type: skip

Test Plan

Verify unit tests pass.

Diff Detail

Repository
rK kazv
Branch
nannanko/kazv-io-test
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 340
Build 663: GitLab CI for kazv
Build 662: arc lint + arc unit

Event Timeline

nannanko created this revision.Mar 14 2025, 6:04 AM
Owners added a reviewer: O1: the Kazv Project.Mar 14 2025, 6:04 AM
nannanko added a task: T114: Add tests for kazv-io-manager and kazv-io-job.Mar 14 2025, 6:11 AM
Harbormaster returned this revision to the author for changes because remote builds failed.Mar 14 2025, 6:17 AM
Harbormaster failed remote builds in B337: Diff 557!
nannanko updated this revision to Diff 558.Mar 14 2025, 10:31 PM

Fix code for libhttpserver 0.18.2

Harbormaster completed remote builds in B338: Diff 558.Mar 14 2025, 10:45 PM
nannanko requested review of this revision.Mar 14 2025, 10:45 PM
nannanko planned changes to this revision.Mar 14 2025, 11:22 PM
tusooa requested changes to this revision.Mar 15 2025, 5:56 PM
tusooa added inline comments.
CMakeLists.txt
81
src/tests/CMakeLists.txt
36

Use the PkgConfig::libhttpserver target, because it also includes the include dirs of that target. Also, use ecm_add_test to single out this one test that requires extra link libraries.

src/tests/kazv-io-job-test.cpp
62–66

This suffers from time-of-check, time-of-use problem. By the time the port is closed, it may no longer be free.

nannanko updated this revision to Diff 559.Mar 21 2025, 6:55 PM

Use PkgConfig correctly, fixed time-of-check, time-of-use problem for port.

nannanko planned changes to this revision.Mar 21 2025, 6:55 PM
nannanko marked 3 inline comments as done.
Harbormaster completed remote builds in B339: Diff 559.Mar 21 2025, 7:09 PM
nannanko updated this revision to Diff 560.Sat, Mar 29, 8:06 PM

Use QHttpServer instead of libhttpserver, and add upload test.

nannanko planned changes to this revision.Sat, Mar 29, 8:06 PM
nannanko updated this revision to Diff 561.Wed, Apr 2, 4:45 AM

Add test about pause among downloading.

nannanko planned changes to this revision.Wed, Apr 2, 4:45 AM
nannanko updated this revision to Diff 562.Wed, Apr 2, 5:28 AM

Add test for cancel download.

nannanko updated this revision to Diff 564.Wed, Apr 2, 5:30 AM

Upgrade the required Qt version to 6.8

nannanko planned changes to this revision.Wed, Apr 2, 5:30 AM
nannanko updated this revision to Diff 565.Wed, Apr 2, 5:44 AM

Add test for KazvIODownloadJob::fileName and KazvIOUploadJob::fileName

nannanko planned changes to this revision.Wed, Apr 2, 5:44 AM
nannanko updated this revision to Diff 566.Sat, Apr 5, 7:50 PM

verify KazvIOBaseJob::isResulted

nannanko planned changes to this revision.Sat, Apr 5, 7:50 PM
Harbormaster failed remote builds in B345: Diff 566!Sat, Apr 5, 7:54 PM
nannanko updated this revision to Diff 567.Sat, Apr 19, 7:18 AM

Split test

nannanko planned changes to this revision.Sat, Apr 19, 7:19 AM
Harbormaster failed remote builds in B346: Diff 567!Sat, Apr 19, 7:23 AM
nannanko updated this revision to Diff 568.Sat, Apr 19, 7:24 AM

Fixed wrong indentation

nannanko planned changes to this revision.Sat, Apr 19, 7:24 AM
Harbormaster failed remote builds in B347: Diff 568!Sat, Apr 19, 7:28 AM
nannanko updated this revision to Diff 569.Sat, Apr 19, 6:33 PM

Add test for HashError

nannanko planned changes to this revision.Sat, Apr 19, 6:33 PM
Harbormaster failed remote builds in B348: Diff 569!Sat, Apr 19, 6:37 PM

Revision Contents
Changeset List

Diff 560

View Options

CMakeLists.txt

Loading...
View Options

src/tests/CMakeLists.txt

Loading...
View Options

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

Loading...