=== chihchun is now known as chihchun_afk === _salem is now known as salem_ === chihchun_afk is now known as chihchun === Ursinha is now known as Ursinha-afk === Ursinha-afk is now known as Ursinha === salem_ is now known as _salem [04:21] Good morning === chihchun is now known as chihchun_afk === chihchun_afk is now known as chihchun === chihchun is now known as chihchun_afk === _salem is now known as salem_ [12:54] brendand: please review om26er's branch: https://code.launchpad.net/~om26er/messaging-app/use_fixture/+merge/225350 [12:54] you are the one that I think will benefit with it. [12:55] rhuddie: I saw the push team writing autopilot tests. [12:55] congrats! [12:55] elopio, yes, they finally landed my suite yesterday [12:56] elopio, I hadn't realised they were adding new ones already :) [12:56] rhuddie, elopio my test is almost done, just waiting for contactsdbus stuff to be moved to address-book-app [12:56] brendand, ^ [12:56] om26er, that's good, I could use that in some of my address-book-app tests too [12:57] elopio, thanks for the info on the content-hub [12:57] elopio, i've not had a chance to look at it yet [12:58] elopio, looks fine to me [13:01] brendand: thanks. [13:02] om26er: brendand: in the end, did you guys discuss about how to add and remove contacts? [13:03] elopio, yes, will be using the contacts dbus api to add/remove contacts [13:04] I suggested that to be done as a fixture with a stock VCARD, that way we can just use the fixture in any project, though a parameter to the fixture could also be provided to support custom vcards [13:05] om26er, btw that code is in ubuntu-experience-tests now, so you aren't blocked. i am going to propose the branch to add it in address-book-app [13:05] brendand, yeah, my test is currently using what is in ubuntu-experience-tests [13:06] crap I missed it, was on the phone wit hthe wife yesterday during that meeting/presentation/... [13:06] ok, sounds good to me, thanks. [13:13] om26er or brendand, please review https://code.launchpad.net/~elopio/ubuntu-system-settings/refactor_about_tests/+merge/225425 [13:14] the python3 branch already landed, so this is the one that's next for my UX test. [13:15] elopio, i'll review it today [13:15] thanks. [13:30] jfunk, Call time! === Ursinha is now known as Ursinha-afk === Ursinha-afk is now known as Ursinha [14:56] brendand: we are sooo close. We just need your notes app branch to land, and to kill media player. [14:59] elopio, notes app did land - http://ci.ubuntu.com/smokeng/utopic/touch/mako/126:20140711.1:20140709/9000/notes_app/ [14:59] elopio, the media player fix we thought would work, didn't though [14:59] yay! [15:06] balloons: any luck reproducing the issues on barry's branch for the dialer-app? [15:07] elopio, afaict my branch should work, but I'm not sure what jenkins is running. I was curious about the weird results I got, so I added lots of debugging logging and other changes; but they don't appear in the logs for the runs. [15:08] all the tests pass, except for the 2 it shouldn't be running. there are skips, but it runs them anyway [15:08] and fails [15:10] balloons: it seems to fail on the set up, that's run before the skip. [15:11] elopio, ohh., well that probably explains it :-) I was going to do some more work on tweaking the skips so it wouldn't run them, but haven't done it [15:12] balloons: and I see your added logs here: https://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-runner-mako/2017/console [15:12] let me know if you need a hand. I just don't know what to do, but after not looking at it for a week, I have forgotten my frustration. [15:13] elopio, wow, so it did run them.. ok, well I got frustrated and left it. But I too have had a couple days away. After I finish some things this morning, I'll swing back to it [15:20] balloons: thanks. [15:27] brendand: about your helpers on the messaging app [15:27] we shouldn't add address book as a dependency to those tests. [15:28] what we should do is to check that url-dispatcher has received the right message. === chihchun_afk is now known as chihchun [15:29] elopio, ok - good idea [15:29] elopio, any existing tests that do that? [15:29] brendand: remindersa-app is using it's own fake service. [15:30] brendand: there is a fake service on the url-dispatcher that I haven't seen when I did reminders. So I was thinking to check that one out, see if it gets installed with the url dispatcher and how to call it from the tests. [15:31] so you can duplicate the fake service for now, or try to call the one in url-dispatcher. [15:33] elopio, would i need to duplicate the code in reminders? [15:33] brendand: if you want to use the python fake, yes. [15:35] elopio, i was thinking , there are some helpers that might be useful across project, but not actually belong in a specific one. do we need a general place to put these? === chihchun is now known as chihchun_afk [15:58] brendand: with balloons we discussed about two options [15:58] first and ideal, to get them on the related project. So if we have a helper for the url-dispatcher, put it on the url dispatcher project [15:59] second and in case the first option fails, put them on the ubuntu-ui-toolkit-autopilot [16:00] which is not bad. That project should contain all the helpers to test ubuntu sdk apps, so it makes sense to even provide things that are outside of the toolkit code. [16:22] ping om26er: is this still relevant? https://code.launchpad.net/~om26er/mediaplayer-app/fix_test_vid/+merge/212731 [16:31] rvr: I'm ready. [16:31] rvr: lets use the same meeting hangout. [16:31] elopio: Ack [17:11] elopio, don't think so, tests are passing so I guess they did something for that. [17:31] rvr: https://code.launchpad.net/~elopio/ubuntu-system-settings-online-accounts/launch_fixture/+merge/226498 [17:31] om26er: no, the test fails on my machine. [17:32] not sure why. I'll dig after a break. === JoseeAntonioR is now known as jose === roadmr is now known as roadmr_afk === roadmr_afk is now known as roadmr === salem_ is now known as _salem