Review Request 127346: RFC: Some infrastructure for Plasma testing

[Aleix Pol Gonzalez]

> On March 11, 2016, 11:23 p.m., Sebastian Kügler wrote:
> > shell/main.cpp, line 135
> > <https://git.reviewboard.kde.org/r/127346/diff/1/?file=449175#file449175line135>
> >
> >     We'll have multiple TestCorona's around if the Shell changes more than once. Not a big deal, since it's only in test mode, just some thing that struck me.

The idea is that we have `TestCorona` that observes how the ShellCorona gets set up and manages the test instances for the `plasmashell`.


> On March 11, 2016, 11:23 p.m., Sebastian Kügler wrote:
> > shell/tests/CMakeLists.txt, line 13
> > <https://git.reviewboard.kde.org/r/127346/diff/1/?file=449180#file449180line13>
> >
> >     I wonder if we should use QTemporaryDir here, but that would mean that we have to set the environment var from the binary. Something to consider, but not critical.

We are using QStandardPaths now, so probably can be changed.


- Aleix


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/127346/#review93425
-----------------------------------------------------------


On March 11, 2016, 9:26 p.m., Aleix Pol Gonzalez wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/127346/
> -----------------------------------------------------------
> 
> (Updated March 11, 2016, 9:26 p.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: plasma-workspace
> 
> 
> Description
> -------
> 
> Makes it possible to test how/if a plasmoid behaves correctly on the most similar environment possible to what the user would find on a production system.
> 
> * Loads a plasmoid on a panel and the desktop.
> * Lets it test his own stuff
> * Notifies, when done, if failed.
> 
> There's still more to look into:
> * what objects need to be passed, besides the `Plasma::AppletQuickItem`.
> * good things to test
> * how to test other shells than the desktop shell
> 
> Nevertheless, it looks to me that it's a step forward. Comments welcome.
> 
> 
> Diffs
> -----
> 
>   applets/analog-clock/contents/tests/testmain.qml PRE-CREATION 
>   shell/CMakeLists.txt e553133 
>   shell/main.cpp d6e3640 
>   shell/shellmanager.h 17acbc6 
>   shell/shellmanager.cpp 3c3638f 
>   shell/testcorona.h PRE-CREATION 
>   shell/testcorona.cpp PRE-CREATION 
>   shell/tests/CMakeLists.txt PRE-CREATION 
>   shell/tests/plasma/shells/org.kde.plasmashelltest/contents/layout.js.in PRE-CREATION 
>   shell/tests/plasma/shells/org.kde.plasmashelltest/contents/loader.qml PRE-CREATION 
>   shell/tests/plasma/shells/org.kde.plasmashelltest/metadata.desktop PRE-CREATION 
> 
> Diff: https://git.reviewboard.kde.org/r/127346/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Aleix Pol Gonzalez
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160312/3c294663/attachment.html>