Merge lp:~dandrader/unity8/unifyLightDMMocks into lp:unity8
| Status: | Merged |
|---|---|
| Approved by: | Michael Terry on 2015-01-23 |
| Approved revision: | 1568 |
| Merged at revision: | 1586 |
| Proposed branch: | lp:~dandrader/unity8/unifyLightDMMocks |
| Merge into: | lp:unity8 |
| Diff against target: |
3381 lines (+1695/-953) 64 files modified
plugins/LightDM/CMakeLists.txt (+3/-3) plugins/LightDM/Greeter.cpp (+1/-19) plugins/LightDM/Greeter.h (+1/-1) plugins/LightDM/GreeterPrivate.h (+39/-0) plugins/LightDM/liblightdm/CMakeLists.txt (+37/-0) plugins/LightDM/liblightdm/Greeter.cpp (+184/-0) plugins/LightDM/liblightdm/Greeter.h (+112/-0) plugins/LightDM/liblightdm/GreeterPrivate.cpp (+281/-0) plugins/LightDM/liblightdm/GreeterPrivate.h (+51/-0) plugins/LightDM/liblightdm/UsersModel.cpp (+123/-0) plugins/LightDM/liblightdm/UsersModel.h (+66/-0) plugins/LightDM/liblightdm/UsersModelPrivate.cpp (+41/-0) plugins/LightDM/liblightdm/UsersModelPrivate.h (+59/-0) plugins/Lights/CMakeLists.txt (+0/-2) plugins/Powerd/CMakeLists.txt (+0/-2) plugins/Unity/Launcher/CMakeLists.txt (+0/-1) plugins/Unity/Session/CMakeLists.txt (+0/-2) plugins/Utils/CMakeLists.txt (+0/-3) run.sh (+3/-26) tests/autopilot/unity8/shell/__init__.py (+0/-13) tests/autopilot/unity8/shell/tests/__init__.py (+8/-11) tests/autopilot/unity8/shell/tests/test_edges_demo.py (+0/-1) tests/autopilot/unity8/shell/tests/test_lock_screen.py (+6/-5) tests/mocks/LightDM/CMakeLists.txt (+8/-9) tests/mocks/LightDM/MockGreeter.cpp (+35/-0) tests/mocks/LightDM/MockGreeter.h (+37/-0) tests/mocks/LightDM/MockUsersModel.cpp (+39/-0) tests/mocks/LightDM/MockUsersModel.h (+37/-0) tests/mocks/LightDM/QLightDM/Greeter (+17/-0) tests/mocks/LightDM/QLightDM/UsersModel (+17/-0) tests/mocks/LightDM/demo/CMakeLists.txt (+0/-37) tests/mocks/LightDM/demo/GreeterPrivate.cpp (+0/-281) tests/mocks/LightDM/demo/UsersModelPrivate.cpp (+0/-41) tests/mocks/LightDM/full/CMakeLists.txt (+0/-18) tests/mocks/LightDM/full/GreeterPrivate.cpp (+0/-98) tests/mocks/LightDM/full/UsersModelPrivate.cpp (+0/-52) tests/mocks/LightDM/liblightdm/CMakeLists.txt (+18/-0) tests/mocks/LightDM/liblightdm/Greeter.cpp (+18/-3) tests/mocks/LightDM/liblightdm/Greeter.h (+7/-0) tests/mocks/LightDM/liblightdm/GreeterPrivate.cpp (+132/-0) tests/mocks/LightDM/liblightdm/GreeterPrivate.h (+5/-3) tests/mocks/LightDM/liblightdm/UsersModel.cpp (+17/-0) tests/mocks/LightDM/liblightdm/UsersModel.h (+10/-3) tests/mocks/LightDM/liblightdm/UsersModelPrivate.cpp (+104/-0) tests/mocks/LightDM/liblightdm/UsersModelPrivate.h (+7/-3) tests/mocks/LightDM/plugin.cpp (+68/-0) tests/mocks/LightDM/plugin.h (+32/-0) tests/mocks/LightDM/single-passphrase/CMakeLists.txt (+0/-19) tests/mocks/LightDM/single-passphrase/GreeterPrivate.cpp (+0/-46) tests/mocks/LightDM/single-passphrase/UsersModelPrivate.cpp (+0/-33) tests/mocks/LightDM/single-pin/CMakeLists.txt (+0/-19) tests/mocks/LightDM/single-pin/GreeterPrivate.cpp (+0/-46) tests/mocks/LightDM/single-pin/UsersModelPrivate.cpp (+0/-33) tests/mocks/LightDM/single/CMakeLists.txt (+0/-18) tests/mocks/LightDM/single/GreeterPrivate.cpp (+0/-46) tests/mocks/LightDM/single/UsersModelPrivate.cpp (+0/-33) tests/plugins/LightDM/CMakeLists.txt (+3/-3) tests/plugins/LightDM/greeter.qml (+5/-0) tests/qmltests/CMakeLists.txt (+19/-19) tests/qmltests/Greeter/tst_Infographics.qml (+10/-0) tests/qmltests/Greeter/tst_MultiGreeter.qml (+11/-0) tests/qmltests/tst_Shell.qml (+8/-1) tests/qmltests/tst_ShellWithPin.qml (+8/-0) tests/qmltests/tst_TabletShell.qml (+8/-0) |
| To merge this branch: | bzr merge lp:~dandrader/unity8/unifyLightDMMocks |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Michael Terry | 2014-12-12 | Approve on 2015-01-23 | |
| PS Jenkins bot | continuous-integration | Needs Fixing on 2015-01-22 | |
|
Review via email:
|
|||
Commit Message
Unify all liblightdm mocks
Benefits:
- Cleaner tests/qmltests/
- Ability change LightDM fake behavior at runtime, which
makes it possible to merge several tests, reducing code duplication,
such as tst_Shell, tst_TabletShell and tst_ShellWithPin.
- Ability to make building of tests optional, as the demo version
of lightdm mock has been moved into plugins/LightDM, removing
the dependency between producition code and test code
Description of the Change
* Are there any related MPs required for this MP to build/function as expected? Please list.
No.
* Did you perform an exploratory manual test run of your code change and any related functionality?
Yes.
* Did you make sure that your branch does not contain spurious tags?
Yes.
* If you changed the packaging (debian), did you subscribe the ubuntu-unity team to this MP?
Not applicable.
* If you changed the UI, has there been a design review?
Not applicable.
| Michael Terry (mterry) wrote : | # |
- 1503. By Launchpad Translations on behalf of unity-team on 2014-12-13
-
Launchpad automatic translations update.
- 1504. By Launchpad Translations on behalf of unity-team on 2014-12-14
-
Launchpad automatic translations update.
- 1505. By Launchpad Translations on behalf of unity-team on 2014-12-15
-
Launchpad automatic translations update.
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1501
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
| Daniel d'Andrada (dandrader) wrote : | # |
> The demo plugin shares a lot of code with the liblightdm mock. Any chance
> they could actually share the file? Like have the mock version subclass the
> demo version or something?
I spent two miserable days trying to do it. It creates a horrible entanglement. Could we please avoid this?
- 1506. By Albert Astals Cid on 2014-12-15
-
Rework how we set the ranges so we get some more asynchronousity from item views Fixes: #1384374
Approved by: Andrea Cimitan - 1507. By Daniel d'Andrada on 2014-12-15
-
Refactor Dialogs.qml to use proper in-scene dialogs
Proper in-scene dialogs keep its Loader as parent and
therefore carry on all its transformations, following its
rotation, etc.It needs Dialog from ubuntu-ui-toolkit to comply with the
reparentToRootItem property.
Approved by: Michał Sawicz, Ying-Chun Liu - 1508. By Daniel d'Andrada on 2014-12-15
-
Make Greeter nicely handle being resized
- Introduced support for bidirectional DragHandles
- Refactored greeter to use a bidirectional DragHandle
- Added code to ensure Greeter retains its relative positioning when Shell is resized
Approved by: Michael Terry - 1509. By Albert Astals Cid on 2014-12-15
-
clickscope is back to being unfavoritable Fixes: #1400762
Approved by: Michał Sawicz - 1510. By Albert Astals Cid on 2014-12-15
-
Fix clicking on the manage dash list after having moved the current scope
Comes with a test for that and for a other manage dash scope move situation
And removes unused code from the test
Approved by: Michael Zanetti - 1511. By Albert Astals Cid on 2014-12-15
-
ScopesList: Use the default scope style for the header
- 1512. By Albert Astals Cid on 2014-12-15
-
Make the drag range be a multiple of the drag "rate" value. Workarounds https:/
/bugs.launchpad .net/mir/ +bug/1399690 Also small fixlet to make _get_scope_loader faster
Approved by: Michał Sawicz - 1513. By Gerry Boland on 2014-12-15
-
MockAppMan: emit SIGSTOP so AP test apps are run Fixes: #1394208
- 1514. By CI Train Bot Account on 2014-12-15
-
Releasing 8.02+15.
04.20141215. 3-0ubuntu1 - 1515. By Launchpad Translations on behalf of unity-team on 2014-12-16
-
Launchpad automatic translations update.
- 1516. By Albert Astals Cid on 2014-12-16
-
Need to wait until loaded is true to use the count property
Approved by: Michael Zanetti - 1517. By Albert Astals Cid on 2014-12-16
-
Manage Dash: Ellide properly
Approved by: Pawel Stolowski
- 1518. By CI Train Bot Account on 2014-12-16
-
Releasing 8.02+15.
04.20141216. 1-0ubuntu1
| Michael Terry (mterry) wrote : | # |
> I spent two miserable days trying to do it. It creates a horrible entanglement. Could we please avoid this?
OK, but can you at least add giant warnings at the top of the file that if you change one, you ought to change the other?
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1502
http://
Executed test runs:
UNSTABLE: http://
UNSTABLE: http://
SUCCESS: http://
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
| Daniel d'Andrada (dandrader) wrote : | # |
On 16/12/14 12:18, Michael Terry wrote:
>> I spent two miserable days trying to do it. It creates a horrible entanglement. Could we please avoid this?
> OK, but can you at least add giant warnings at the top of the file that if you change one, you ought to change the other?
Sure. done.
| Michael Terry (mterry) wrote : | # |
Running ./run.sh -f gives:
file://
import LightDM 0.1 as LightDM
^
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1503
http://
Executed test runs:
UNSTABLE: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
- 1519. By Launchpad Translations on behalf of unity-team on 2014-12-17
-
Launchpad automatic translations update.
- 1520. By Launchpad Translations on behalf of unity-team on 2014-12-21
-
Launchpad automatic translations update.
- 1521. By Launchpad Translations on behalf of unity-team on 2014-12-22
-
Launchpad automatic translations update.
- 1522. By Launchpad Translations on behalf of unity-team on 2014-12-27
-
Launchpad automatic translations update.
- 1523. By Launchpad Translations on behalf of unity-team on 2014-12-28
-
Launchpad automatic translations update.
- 1524. By Launchpad Translations on behalf of unity-team on 2015-01-08
-
Launchpad automatic translations update.
- 1525. By Albert Astals Cid on 2015-01-09
-
Fix going to scopes when the Manage Dash is open Fixes: #1403464
Approved by: Michael Zanetti, PS Jenkins bot - 1526. By Albert Astals Cid on 2015-01-09
-
QSortFilterProx
yModelQML -> UnitySortFilter ProxyModelQML Someone copied the class to the sdk (but with less features...) so we better rename, class and qml name collisions are not cool Fixes: #1403758
Approved by: Michael Zanetti, PS Jenkins bot - 1527. By Albert Astals Cid on 2015-01-09
-
Clip the Scopes List header
Now that we have a background since we use a default ScopeStyle we need to clip the "upper/search" part of the header Fixes: #1407620
Approved by: Andrea Cimitan, PS Jenkins bot - 1528. By Albert Astals Cid on 2015-01-09
-
Fix ScopesList not being under finger
Do not take into the account the initial distance that took the movement to be accepted to calculate the position Fixes: #1403045
Approved by: Andrea Cimitan, PS Jenkins bot - 1529. By Martin Pitt on 2015-01-09
-
tests: Fix Notify.
Notification. add_action invocation to work also with unpatched libnotify API. Fixes: #1223401
Approved by: Michał Sawicz, PS Jenkins bot - 1530. By Albert Astals Cid on 2015-01-09
-
Make waitForRendering with no item fail instead of crash
Approved by: Michał Sawicz, PS Jenkins bot - 1531. By Albert Astals Cid on 2015-01-09
-
Disable Dash horizontal scroll while Navigation InverseMouseArea is pressed Fixes: #1403048
Approved by: Andrea Cimitan - 1532. By Michał Sawicz on 2015-01-09
-
We don't need the SIGSTOP in main() any more.
Approved by: Michael Zanetti - 1533. By dobey on 2015-01-09
-
Depend on :native version of g++ to allow cross-compiling to work. Fixes: #1353855
Approved by: Michał Sawicz, PS Jenkins bot - 1534. By Nick Dedekind on 2015-01-09
-
Unhook Lights interface from indicator widgets Fixes: #1385331
Approved by: Albert Astals Cid - 1535. By Michael Terry on 2015-01-09
-
Don't block handling power events on loading the greeter's qml and the background image.
During power button press, we call greeter.showNow(), which was taking so long to return that we couldn't handle the power button release for a few seconds. Which meant that the shutdown dialog thought it would be a good time to appear.
The easiest fix is to delay calling showNow until after we finish handling the event. Fixes: #1383277
Approved by: Daniel d'Andrada, PS Jenkins bot - 1536. By Michael Terry on 2015-01-09
-
Show OSK above the wizard. Fixes: #1401213
Approved by: Andrea Cimitan, PS Jenkins bot - 1537. By Leo Arias on 2015-01-09
-
Added an autopilot test for the edges demo.
Approved by: PS Jenkins bot, Albert Astals Cid - 1538. By Michael Terry on 2015-01-09
-
Unify the name of the Greeter DBus test, make it use our standard binary test macro (which also nicely exports xml results), and make the test a little more robust against timing issues.
Approved by: PS Jenkins bot, Albert Astals Cid - 1539. By Michael Terry on 2015-01-09
-
Add a test to make sure the shell always starts disabled until it is enabled by a complete PAM interaction.
Approved by: Michał Sawicz - 1540. By Albert Astals Cid on 2015-01-09
-
Test: Make sure the mouse area is enabled before clicking into it
Despite the name swipeToCloseEnabled controls the enabled property of the whole mouse area
of the spread delegate so it needs to be true (animations & movement finished) before we can click on itApproved by: Michał Sawicz
- 1541. By Albert Astals Cid on 2015-01-09
-
Test: We actually need to click on the customBackButton and not on backButton
Approved by: Michał Sawicz
- 1542. By Albert Astals Cid on 2015-01-09
-
Test: By default mouseX act on the middle
Saves us lots of typing, tests are easier to read and workarounds problems that sometimes happen if we end up with 0, 0 as mouse coordinates because the precission of some geometry goes crazy and since we're clicking on 0x0 after lots transformations of pos into pos into pos we end up clicking outside the item by a very small number
Approved by: Michał Sawicz
- 1543. By CI Train Bot Account on 2015-01-09
-
Releasing 8.02+15.
04.20150109. 2-0ubuntu1 - 1544. By Launchpad Translations on behalf of unity-team on 2015-01-10
-
Launchpad automatic translations update.
| Albert Astals Cid (aacid) wrote : | # |
Text conflict in tests/mocks/
Text conflict in tests/plugins/
2 conflicts encountered.
- 1545. By Leo Arias on 2015-01-13
-
Added an autopilot test for the edges demo.
Approved by: PS Jenkins bot, Albert Astals Cid - 1546. By Gerry Boland on 2015-01-13
-
DashCommunicator: replace QDBusInterface with a non-blocking simplification which does not introspect the service on creation Fixes: #1403508
Approved by: Michael Zanetti - 1547. By Albert Astals Cid on 2015-01-13
-
Don't show the manage dash pull up arrow on temp scopes Fixes: #1401869
Approved by: Michael Zanetti - 1548. By Albert Astals Cid on 2015-01-13
-
Remove hack, newer Qt already support keyClick(char)
Approved by: Michael Zanetti, PS Jenkins bot
- 1549. By Albert Astals Cid on 2015-01-13
-
Tests: Add Qt 5.5 removal TODOs
Approved by: Michał Sawicz
- 1550. By Michael Zanetti on 2015-01-13
-
patch debian/control file before using it to make it work with mk-build-deps
Approved by: Michał Sawicz - 1551. By Albert Astals Cid on 2015-01-13
-
Make sure we use fPIC when compiling files for the static library too
Fixes compilation in ARM + Qt 5.4 Fixes: #1409235
Approved by: Michał Sawicz, Dmitry Shachnev - 1552. By Albert Astals Cid on 2015-01-13
-
Make sure changing a scope doesn't trigger creation/
destruction of delegates until it's finished
Fixes: #1410122
Approved by: Michał Sawicz - 1553. By Andrea Cimitan on 2015-01-13
-
support background on horizontal cards with summary Fixes: #1393008
Approved by: Michał Sawicz - 1554. By CI Train Bot Account on 2015-01-13
-
Releasing 8.02+15.
04.20150113. 1-0ubuntu1 - 1555. By CI Train Bot Account on 2015-01-14
-
Resync trunk
- 1556. By Launchpad Translations on behalf of unity-team on 2015-01-15
-
Launchpad automatic translations update.
- 1557. By Launchpad Translations on behalf of unity-team on 2015-01-16
-
Launchpad automatic translations update.
- 1558. By Launchpad Translations on behalf of unity-team on 2015-01-17
-
Launchpad automatic translations update.
- 1559. By Launchpad Translations on behalf of unity-team on 2015-01-18
-
Launchpad automatic translations update.
- 1560. By Launchpad Translations on behalf of unity-team on 2015-01-19
-
Launchpad automatic translations update.
- 1561. By Daniel d'Andrada on 2015-01-20
-
Unify all liblightdm mocks
Benefits:
- Cleaner tests/qmltests/CMakeLists. txt
- Ability change LightDM fake behavior at runtime, which
makes it possible to merge several tests, reducing code duplication,
such as tst_Shell, tst_TabletShell and tst_ShellWithPin.
- Ability to make building of tests optional, as the demo version
of lightdm mock has been moved into plugins/LightDM, removing
the dependency between producition code and test code - 1562. By Daniel d'Andrada on 2015-01-20
-
Update run.sh
| Daniel d'Andrada (dandrader) wrote : | # |
> Running ./run.sh -f gives:
>
> file://
> plugin cannot be loaded for module "LightDM": Cannot load library
> /home/mike/
> /libMockLightDM
> No such file or directory)
> import LightDM 0.1 as LightDM
> ^
Fixed.
| Daniel d'Andrada (dandrader) wrote : | # |
> Text conflict in tests/mocks/
> Text conflict in tests/plugins/
> 2 conflicts encountered.
Fixed.
- 1563. By Daniel d'Andrada on 2015-01-20
-
Further clean up of run.sh
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1562
http://
Executed test runs:
UNSTABLE: http://
UNSTABLE: http://
SUCCESS: http://
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1563
http://
Executed test runs:
UNSTABLE: http://
UNSTABLE: http://
SUCCESS: http://
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
- 1564. By Daniel d'Andrada on 2015-01-20
-
Init mockMode
| Michael Terry (mterry) wrote : | # |
OK, seems good now, assuming Jenkins approves. Thanks man!
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1564
http://
Executed test runs:
UNSTABLE: http://
UNSTABLE: http://
SUCCESS: http://
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
| Michael Terry (mterry) wrote : | # |
A lot of autopilot tests are failing with errors like:
RuntimeError: LightDM mock 'single-pin' does not exist at path '/usr/lib/
RuntimeError: LightDM mock 'single' does not exist at path '/usr/lib/
Some other autopilot tests are failing for presumably other reasons (indicators). The qmluitests failures are in DashContent and also seem unrelated.
| Daniel d'Andrada (dandrader) wrote : | # |
> A lot of autopilot tests are failing with errors like:
>
> RuntimeError: LightDM mock 'single-pin' does not exist at path '/usr/lib/arm-
> linux-gnueabihf
>
> RuntimeError: LightDM mock 'single' does not exist at path '/usr/lib/arm-
> linux-gnueabihf
Oh, thanks for pointing that out. I forgot to check if the autopilot tests were using those libs.
> The qmluitests failures are in DashContent and also seem
> unrelated.
Yes, Dash has some known unstable tests at the moment. tsdgeos has a branch to make them more reliable.
- 1565. By Daniel d'Andrada on 2015-01-21
-
Enable specifying liblightdm mock mode via env var
- 1566. By Daniel d'Andrada on 2015-01-21
-
Update autopilot tests
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1566
http://
Executed test runs:
UNSTABLE: http://
UNSTABLE: http://
SUCCESS: http://
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
| Albert Astals Cid (aacid) wrote : | # |
Text conflict in tests/mocks/
Text conflict in tests/plugins/
Text conflict in tests/qmltests/
3 conflicts encountered.
- 1567. By Daniel d'Andrada on 2015-01-22
-
Merge trunk
[ Michael Terry ]
* Don't show any power dialogs while the screen is off. This is a
band-aid to avoid the symptom, while we still investigate the source
of why the screen doesn't turn on (and why we seem to not handle the
power-button- release event). (LP: #1409003, #1410830)
* Don't refocus the dialer-app when a call is started by the user.
Because of how qtmir works, this would cause the dialer-app to
unfocus momentarily, causing interface oddities for the user. (LP:
#1413065)
[ CI Train Bot ]
* Resync trunk
[ Albert Astals Cid ]
* Build with ninja
* Fixes for Table Preview Widget (LP: #1410420)
[ Michael Zanetti ]
* Add support for earpiece media buttons (LP: #1398427)
* Add a WindowMoveResizeArea and tests for it
| Daniel d'Andrada (dandrader) wrote : | # |
On 22/01/15 06:39, Albert Astals Cid wrote:
> Text conflict in tests/mocks/
> Text conflict in tests/plugins/
> Text conflict in tests/qmltests/
> 3 conflicts encountered.
Fixed
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1567
http://
Executed test runs:
UNSTABLE: http://
UNSTABLE: http://
SUCCESS: http://
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
| Michael Terry (mterry) wrote : | # |
unity8.
- 1568. By Daniel d'Andrada on 2015-01-22
-
Just to trigger a new build in Jenkins
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1568
http://
Executed test runs:
UNSTABLE: http://
UNSTABLE: http://
SUCCESS: http://
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
| Daniel d'Andrada (dandrader) wrote : | # |
> unity8.
> ycleTests.
> me. Might be due to these changes.
That test no longer fails in the latest Jenkins run. I just triggered a rebuild. No code changes. So this is an unstable test.
/me does the happy dance.
| Michael Terry (mterry) wrote : | # |
Just because it's an unstable test doesn't mean it's not your fault. :) But I agree that your changes would be more likely to reliably break a test than cause it to be flaky.
OK, I'll approve the branch.
| Albert Astals Cid (aacid) wrote : | # |
Text conflict in run.sh
Text conflict in tests/mocks/
Text conflict in tests/plugins/
3 conflicts encountered.
- 1569. By Daniel d'Andrada on 2015-01-27
-
Merge trunk
[ Sebastien Bacher ]
* Set text hint property for the messaging menus inline reply widget
(LP: #1389234)
[ Ying-Chun Liu ]
* Added an initial set of autopilot tests for the display indicator.
[ Nick Dedekind ]
* Removed filtering the indicator model by visibility.
[ Michael Zanetti ]
* fixes left edge drag when in spread
[ Albert Astals Cid ]
* Fix ninja build in a different way The previous one was adding rpath
to some .so that broke our tests that use LD_LIBRARY_PATH to change
which one is picked up
[ Leo Arias ]
* Added an initial set of autopilot tests for the display indicator.
| Daniel d'Andrada (dandrader) wrote : | # |
> Text conflict in run.sh
> Text conflict in tests/mocks/
> Text conflict in tests/plugins/
> 3 conflicts encountered.
Solved.
- 1570. By Daniel d'Andrada on 2015-02-03
-
include(
FindPkgConfig) only once, in the root CMakeLists.txt In order to fix cross compilation
Patch by Saviq

Seems mostly good, thanks man!
The demo plugin shares a lot of code with the liblightdm mock. Any chance they could actually share the file? Like have the mock version subclass the demo version or something?
Also, we've been meaning to rename the demo plugin because we are actually using it in producton now. Maybe something that sounds a little more robust than demo? (though it does still hold some demo code)