Comment 3 for bug 1603706

With the attached branch added to the silo, we got failures for 2 out of the 20 builds.

First was the vivid i386 build: https://launchpadlibrarian.net/273878957/buildlog_ubuntu-vivid-i386.online-accounts-api_0.1+15.04.20160719-0ubuntu1_BUILDING.txt.gz

    1468915489.236 GetConnectionAppArmorSecurityContext ":1.11"
    QWARN : FunctionalTests::testAccountChanges() QDBusConnection: name 'com.ubuntu.OnlineAccounts.Manager' had owner '' but we thought it was ':1.8'
    FAIL! : FunctionalTests::testAccountChanges() Compared values are not the same
       Actual (accountInfo.data()) : {"authMethod":0,"changeType":2,"displayName":"New account","serviceId":"com.ubuntu.tests_coolshare"}
       Expected (expectedAccountInfo): {"authMethod":0,"changeType":0,"displayName":"New account","serviceId":"com.ubuntu.tests_coolshare"}
       Loc: [/«BUILDDIR»/online-accounts-api-0.1+15.04.20160719/tests/daemon/functional_tests/functional_tests.cpp(550)]

Second was yakkety arm64: https://launchpadlibrarian.net/273879816/buildlog_ubuntu-yakkety-arm64.online-accounts-api_0.1+16.10.20160719-0ubuntu1_BUILDING.txt.gz

    1468915517.823 GetConnectionAppArmorSecurityContext ":1.11"
    QWARN : FunctionalTests::testAccountChanges() QDBusConnection: name 'com.ubuntu.OnlineAccounts.Manager' had owner '' but we thought it was ':1.8'
    FAIL! : FunctionalTests::testAccountChanges() Compared values are not the same
       Actual (((accountChanged.count()))): 0
       Expected (1) : 1
       Loc: [/«BUILDDIR»/online-accounts-api-0.1+16.10.20160719/tests/daemon/functional_tests/functional_tests.cpp(561)]

Both look consistent with this being related to the "exit on idle" code. As discussed on IRC, it might be best to run the majority of tests with the exit on idle behaviour disabled, and have have one test with it enabled that is explicitly testing the behaviour.