Merge lp:~robru/cordova-ubuntu-tests/drop-unity-autopilot into lp:cordova-ubuntu-tests

[Robert Bruce Park]

Drop unity-autopilot dependency.

unity-autopilot brings in a lot of unnecessary desktop components, like compiz-core and other things that are hilariously inappropriate on a phone.

Does anybody know why this dependency was added in the first place? I just tried running our autopilot tests on the phone without it and only got 2 failures.

I *guess* that it was added because we really depend on some component that unity-autopilot brings in. If anybody can identify what that component might be, I'll add it directly. Otherwise, we need to land this as-is, because unity-autopilot is simply not a correct dependency of cordova-ubuntu-autopilot.

[PS Jenkins bot (ps-jenkins)]

PASSED: Continuous integration, rev:28
http://jenkins.qa.ubuntu.com/job/cordova-ubuntu-tests-ci/24/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/cordova-ubuntu-tests-trusty-amd64-ci/1
    SUCCESS: http://jenkins.qa.ubuntu.com/job/cordova-ubuntu-tests-trusty-armhf-ci/1

Click here to trigger a rebuild:
http://10.97.0.26:8080/job/cordova-ubuntu-tests-ci/24/rebuild

review: Approve (continuous-integration)

[Víctor R. Ruiz (vrruiz)]

Hi Robert. I was testing yesterday the package without the unity-autopilot dependency installed, apparently it's not needed, but I want to be sure.

[Robert Bruce Park (robru)]

Victor, agreed. I did see 2 failures when i tested this, but i didn't look closely, and i didn't check if the failures were resolved by installing unity-autopilot. I can dig a bit deeper into this today and we'll see where it goes.

[Robert Bruce Park (robru)]

Still testing this. The failures I get seem to be related to an inability to create files?

http://paste.ubuntu.com/6372030/

[Robert Bruce Park (robru)]

In fact, installing unity-autopilot didn't cause these tests to pass, so I'm going to go ahead and approve this. I'm pretty sure it's not necessary at this point.

review: Approve

=== modified file 'debian/control'
--- debian/control	2013-10-15 17:03:13 +0000
+++ debian/control	2013-11-06 01:39:53 +0000
@@ -26,7 +26,6 @@
 Depends: cordova-ubuntu-2.8,
          cordova-ubuntu-tests,
          libautopilot-qt,
-         unity-autopilot,
          dpkg-dev,
          ${misc:Depends},
          ${python:Depends},