Fix temporary path in importer test unit
ClosedPublic

Authored by bdevries on Oct 22 2019, 8:33 PM.

Details

Summary

Add missing slash. Without this slash an additional directory is created out of the scope of the QTemporaryDir. This directory is hence not deleted at the end of the test.

Test Plan

Run importertest and verify that no leftover temporary directories are left in /tmp.

Diff Detail

Repository
R260 Gwenview
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
bdevries requested review of this revision.Oct 22 2019, 8:33 PM
bdevries created this revision.
ngraham accepted this revision.Oct 22 2019, 8:38 PM
ngraham added a subscriber: ngraham.

Thanks! Can you provide your email address so I can commit this patch with correct attribution?

This revision is now accepted and ready to land.Oct 22 2019, 8:38 PM

Sure, my "git" email address is devriesb@gmail.com .

As you might have seen, I've just submitted two more patches for the gwenview importer.
I'd like to also fix some pretty easy bugs and feature requests on the importer in the next few days.

Sounds lovely, please go ahead!

Since this is my first ever contribution to KDE: is there anything that I still need to do on my side? Or will this commit be applied without further intervention from me?

tommo added a subscriber: tommo.Oct 23 2019, 7:30 AM

Don't worry, Nate will apply it.

This revision was automatically updated to reflect the committed changes.