Page MenuHomePhorge
Differential D176

Add tests for KazvFile and KazvSaveFile
ClosedPublic
Actions

Authored by nannanko on Aug 23 2024, 10:39 PM.

Details

Reviewers
tusooa
Group Reviewers
O1: the Kazv Project
Commits
rKd9db047e9339: Add tests for KazvFile and KazvSaveFile
Summary

Add test for KazvFile and KazvSaveFile.

Type: add

Test Plan

Verify unit tests pass.

Diff Detail

Repository
rK kazv
Branch
nannanko/kazv-file-test
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 282
Build 552: GitLab CI for kazv
Build 551: arc lint + arc unit

Event Timeline

nannanko created this revision.Aug 23 2024, 10:39 PM
Owners added a reviewer: O1: the Kazv Project.Aug 23 2024, 10:39 PM
Herald added a reviewer: servant. · View Herald TranscriptAug 23 2024, 10:39 PM

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

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

Harbormaster returned this revision to the author for changes because remote builds failed.Aug 23 2024, 10:54 PM
Harbormaster failed remote builds in B279: Diff 466!
nannanko requested review of this revision.Aug 23 2024, 11:16 PM
nannanko edited the summary of this revision. (Show Details)
nannanko removed a reviewer: servant.
Harbormaster completed remote builds in B279: Diff 466.Aug 23 2024, 11:32 PM
tusooa added inline comments.Aug 24 2024, 5:54 PM
src/tests/kazv-file-test.cpp
32–35

don't write anything with side effects into the assertion macros. your code should still work if you remove all of the macros entirely.

36

use Qt's random generator

nannanko planned changes to this revision.Aug 24 2024, 6:04 PM
nannanko updated this revision to Diff 469.Aug 24 2024, 6:51 PM

Add tests for KazvFile and KazvSaveFile

nannanko marked 2 inline comments as done.Aug 24 2024, 6:56 PM
Harbormaster completed remote builds in B282: Diff 469.Aug 24 2024, 7:07 PM
tusooa requested changes to this revision.Aug 27 2024, 7:04 PM

test that if it is cancelled, the file will not be found on disk.

This revision now requires changes to proceed.Aug 27 2024, 7:04 PM
nannanko added a comment.EditedAug 31 2024, 6:28 AM
In D176#3265, @tusooa wrote:

test that if it is cancelled, the file will not be found on disk.

This seems not necessary to test, or should be placed in the test of KazvIOJob, because the relevant functions are implemented by Qt. If you think it is necessary, I will write relevant tests for KazvIOBaseJob::cancel().

nannanko requested review of this revision.Aug 31 2024, 6:28 AM
tusooa accepted this revision.Aug 31 2024, 7:03 PM
This revision is now accepted and ready to land.Aug 31 2024, 7:03 PM
Closed by commit rKd9db047e9339: Add tests for KazvFile and KazvSaveFile. · Explain WhySep 4 2024, 12:46 AM
This revision was automatically updated to reflect the committed changes.
nannanko added a commit: rKd9db047e9339: Add tests for KazvFile and KazvSaveFile.

Revision Contents
Changeset List

Diff 469

View Options

src/tests/CMakeLists.txt

Loading...
View Options

src/tests/kazv-file-test.cpp

Loading...