Differential D24076

add a simple smoke test for slaves by using kioclient5
ClosedPublic
Actions

Authored by sitter on Sep 19 2019, 1:04 PM.

Details

Reviewers
dfaure
Commits
R320:6bdc413c720b: add a simple smoke test for slaves by using kioclient5
Summary

this is meant to give a fairly generic way to easily test core
functionality of any io slave

Test Plan

ran against sftp and smb and it's all lovely

Diff Detail

Repository
R320 KIO Extras
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
sitter created this revision.Sep 19 2019, 1:04 PM
Restricted Application added projects: Dolphin, Frameworks. · View Herald TranscriptSep 19 2019, 1:04 PM
Restricted Application added subscribers: kfm-devel, kde-frameworks-devel. · View Herald Transcript
sitter requested review of this revision.Sep 19 2019, 1:04 PM
feverfew added a subscriber: feverfew.Sep 19 2019, 3:08 PM

I won't review, but would like to note I think that with KIOFuse, in conjuction with fio we can do some really complex testing on KIO, correct me if I'm wrong.

If I'm right, then we can test KIO nicely, although if we do notice a bug via using fio, it can technically be in any of FUSE/KIOFuse/KIO, but in my experience KIO is the most buggy of the lot.

dfaure accepted this revision.Sep 20 2019, 4:31 PM
This revision is now accepted and ready to land.Sep 20 2019, 4:31 PM
sitter added a comment.Sep 23 2019, 10:47 AM
In D24076#534627, @feverfew wrote:

I won't review, but would like to note I think that with KIOFuse, in conjuction with fio we can do some really complex testing on KIO, correct me if I'm wrong.

If I'm right, then we can test KIO nicely, although if we do notice a bug via using fio, it can technically be in any of FUSE/KIOFuse/KIO, but in my experience KIO is the most buggy of the lot.

Great idea. Definitely worth looking into!

Closed by commit R320:6bdc413c720b: add a simple smoke test for slaves by using kioclient5 (authored by sitter). · Explain WhySep 23 2019, 11:34 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 66655

View Options

smoketest/README.md

Loading...
View Options

smoketest/file1

Loading...
View Options

smoketest/kio_smoke_test/.keep

Loading...
View Options

smoketest/smoke.sh

Loading...