davidhurka added a comment.
In D15580#606022 <https://phabricator.kde.org/D15580#606022>, @aacid wrote: > In D15580#605913 <https://phabricator.kde.org/D15580#605913>, @davidhurka wrote: > > > In D15580#605611 <https://phabricator.kde.org/D15580#605611>, @simgunz wrote: > > > > > aacid requested to write some autotest for the ToggleActionMenu before merging this. I'll merge master in this review and work on the autotests soon. > > > > > > It’s okay to me that you write the autotests. If you wish, I could do that instead. > > > > I am new to autotests, so I will have to learn how to do that. I am also unsure about what should be tested. The ToggleActionMenu itself, or its behaviour in Okular? > > > davidhurka (David Hurka) User > The functionality in Okular, so that if in the future we replace ToggleActionMenu with EvenBetterToggleActionMenu we can spot regressions I’m working on the autotests. But I had no backup, and so need to start over again. :( The functionality in Okular means for me that I trigger a mouse mode action, and then test the default action of the toolbar button. Is that what you mean? Can I add the ToggleActionMenu test to parttest, or do I need a new test? And someone needs to explain how to create merge requests on invent.kde.org. I tried three ways and failed, and unfortunately I can only find instructions for Phabricator. REPOSITORY R223 Okular REVISION DETAIL https://phabricator.kde.org/D15580 To: simgunz, #okular, #vdg, ngraham Cc: kde-doc-english, andreashurka, yurchor, trickyricky26, simgunz, ltoscano, cfeck, aacid, davidhurka, knambiar, ngraham, tobiasdeiminger, okular-devel, johnzh, andisa, siddharthmanthan, maguirre, gennad, fbampaloukas, joaonetto, kezik, tfella, skadinna, darcyshen