/srv/irclogs.ubuntu.com/2015/03/03/#ubuntu-quality.txt

=== chihchun_afk is now known as chihchun
=== chihchun is now known as chihchun_afk
=== chihchun_afk is now known as chihchun
elopiogood morning team.14:20
balloonsmorning!14:25
elopiofgimenez: on your bluetooth branch, you have to add the config option to ubuntu_sanity_tests/config.py15:49
elopiofgimenez: also, the BluetoothPage class was upstreamed to system settings.15:51
elopioand ensure_system_settings_closed can be moved to system_settings.py and used in the sim pin test.15:53
fgimenezelopio: ack, on it15:55
elopiorhuddie: can I help you with the camera test?16:15
rhuddieelopio, hey. one thing that i found is that the camera helper to close the bottom edge doesn't seem to work very well.16:18
rhuddieelopio, i changed so it works in the middle of the screen rather than at 0 x co-ord.16:18
rhuddiebut i'm not sure how it works in the other tests if it fails for me16:19
elopiorhuddie: I don't know. Might not be used in their tests.16:19
rhuddieelopio, yes, could be16:20
elopiorhuddie: what's the name of the method?16:21
rhuddieelopio, _drag_to_close: http://bazaar.launchpad.net/~phablet-team/camera-app/trunk/view/head:/tests/autopilot/camera_app/emulators/panel.py#L5216:22
elopiorhuddie: it is used in four times.16:23
rhuddieelopio, if you see the _drag_to_open method works in the x centre, but the close one doesn't16:23
elopiorhuddie: ah, it is used four times, but at the end of the test.16:24
elopioit has no assertions afterwards.16:24
rhuddieelopio, ah, now it makes sense16:24
elopiorhuddie: you can report a bug to the camera.16:24
rhuddieelopio, yep will do16:24
elopioit's unlikely they'll fix it for us, but it will bring some visibility to what we are doing.16:25
rhuddieelopio, i'll propose an mp, as my fix seems to work fine16:25
elopiorhuddie: ok, but ideally your fix has a test with an assertion after the close, so it never breaks.16:25
rhuddieelopio, well, how about changing the existing 4 instances to assert it is closed afterwards?16:26
elopiorhuddie: that works too.16:26
elopiorhuddie: those tests are ugly, you are warned.16:27
elopiorhuddie: also it would work to add a "wait for closed" in the close method.16:27
fgimenezelopio: it seems that the upstreamed BluetoothPage is missing the required get_disconnected_list method16:27
rhuddieelopio, yes, that would be neater solution16:28
elopiofgimenez: it's called get_disconnected_devices.16:29
fgimenezelopio: ok thanks, where can i check the code?16:29
fgimenezelopio: the upstreamed BluetoohPage class, i mean16:31
elopiofgimenez: that's lp:ubuntu-system-settings16:35
elopioin the folder tests/autopilot.16:36
elopiobarry: is there a way to call system-image-dbus with --no-reboot?20:27
Letozaf_balloons, :)20:28
barryelopio: no, unfortunately there isn't.  in si 3.0 we can set a configuration file option to basically not do a reboot when the update is applied, and in si 3.0 the cli has a --no-reboot flag.  but si 3.0 hasn't been released to vivid20:28
elopiobarry: I'm using system-image-cli --no-reboot on vivid.20:29
barryelopio: ah, sorry.  si 3.0 renamed --no-reboot to --no-apply.  you're right20:30
barryelopio: and actually, i think we can hack the config file to not reboot20:30
elopiobarry: that sounds good.20:30
elopiobarry: I got this: http://paste.ubuntu.com/10519178/20:31
elopioany idea why is the file not there?20:31
barryelopio: it should be acceptable not to have a blacklist keyring20:31
Letozaf_balloons, should we chat here about the filemanager app failing tests ?20:32
barryelopio: but if a blacklist keyring was found and downloaded but then it disappeared, that would be a problem.  i don't know why that would happen in this case20:32
elopiobarry: yes, on the log I see this:20:33
elopiohttps://system-image.ubuntu.com/gpg/blacklist.tar.xz -> /var/lib/system-image/keyring.tar.xz20:33
elopiosorry, this20:34
elopiohttps://system-image.ubuntu.com/gpg/blacklist.tar.xz.asc -> /var/lib/system-image/keyring.tar.xz.asc20:34
barryelopio: can you pastebin the full log?20:34
elopiobarry: http://paste.ubuntu.com/10519237/20:35
elopioI re-run it, and now it's doing the upgrrade.20:35
barryelopio: weird!  nothing obvious from the log file20:36
elopiobarry: veebers: this one succeeded :)20:39
elopiohttp://paste.ubuntu.com/10519265/20:39
barry:-o20:41
veeberselopio: awesome ^_^20:41
balloonsLetozaf_, did you have anything else to chat about? I was concerned about calendar failing in trunk20:41
balloonsbut yes, shoot20:41
Letozaf_balloons, well if you want I can come back tomorrow or when you have more time, it's not urgent I suppose!20:42
balloonsLetozaf_, no time like the present20:42
balloons:-)20:42
Letozaf_balloons, :) ok so I will shoot20:42
Letozaf_balloons, I took a look at the filemanager test failures.20:43
Letozaf_I am unable to reproduce the failures on my desktop, but a lot of tests20:43
Letozaf_are failing with a StateNotFoundError on bject not found with name20:43
Letozaf_'ActionSelectionPopover' and properties {'objectName':20:43
Letozaf_'fileActionsPopover'}, but that object exists an also the objectName is20:43
Letozaf_correct20:43
Letozaf_balloons, It could be a timing issue, but the get_action_selection_popover() is in20:44
Letozaf_ _main.py of ubuntuuitoolkit, so not quite sure if it should be fixed there.20:44
Letozaf_https://dl.dropboxusercontent.com/u/10083981/Ubuntu%20phone/Actionspopover.png20:44
Letozaf_balloons, and I created a mp with no changes, just to run filemanager tests on Jenkins...20:45
Letozaf_Well, the tests passed :O.20:45
Letozaf_balloons, so do not exactly know why they fail here: http://ci.ubuntu.com/smokeng/vivid/touch/mako/117:20150301.1:20150210/12398/filemanager/20:46
Letozaf_balloons, any idea ?20:46
Letozaf_balloons, ah! the mp that is not failing: https://code.launchpad.net/~carla-sella/ubuntu-filemanager-app/just-for-test20:47
* balloons reads20:52
balloonshehe20:52
balloonsI rebuilt that just for another run20:52
balloonsso where is the mp that is failing?20:52
Letozaf_balloons, the tests are failing in  http://ci.ubuntu.com/smokeng/vivid/touch/mako/117:20150301.1:20150210/12398/filemanager/20:53
elfyevening balloons Letozaf_20:53
balloonsevening elfy20:53
balloonsohh, on the dashboard20:53
Letozaf_elfy, evening to you20:53
balloonsLetozaf_, so right, no worries on jenkins20:54
elfyballoons: it got delivered to my mum today - picking it up tomorrow :)20:54
balloonselfy, excellent. enjoy yourself now20:54
elfyeventually I will ;)20:54
balloonsLetozaf_, so if you want to reproduce the results on the CI dashboard you'll need to follow some directions.20:59
balloonsci is actually working on it right now20:59
Letozaf_balloons, ok so should I leave it to them?21:00
balloonsLetozaf_, no lol, I meant they are working on proper directions for you to reproduce results from the dashboard locally21:01
Letozaf_balloons, oooh! thats even better :P21:01
balloonsin the interim however, do you know about looking at subunit files using trv?21:01
balloonsLetozaf_, https://code.launchpad.net/~canonical-ci-engineering/ubuntu-test-cases/testing-docs/+merge/25150721:01
balloonsLetozaf_, so save http://jenkins.qa.ubuntu.com/job/vivid-touch-mako-smoke-daily/349/artifact/clientlogs/filemanager/test_results.subunit/*view*/21:02
Letozaf_balloons, I remember looking at a video time ago about trv but do not remember...21:02
* Letozaf_ is reading 21:03
balloonsyou can grab trv from https://launchpad.net/~thomir/+archive/ubuntu/trv/+build/662648821:06
elfyballoons: is https://wiki.ubuntu.com/Touch the go to page?21:06
Letozaf_balloons, yes I found it and also this: https://www.youtube.com/watch?v=jkLtbmQxXYc21:06
balloonselfy, that's an excellent place to begin yes21:07
elfyta21:07
balloonsLetozaf_, so looking at the screenshots it's clear the popover didn't appear21:09
Letozaf_balloons, yes I see :) nice this trv21:10
balloonsLetozaf_, so I think you are correct in thinking it's a timing thing21:11
Letozaf_balloons, yes looks like, I will try to find a solution21:11
balloonsso Letozaf_ once you think you have a solution, feel free to follow the instructions in: http://bazaar.launchpad.net/~ubuntu-test-case-dev/ubuntu-test-cases/touch/view/head:/README-cli.rst. But it means wiping your device completely21:13
Letozaf_balloons, :O21:13
balloonsI would mess around on the desktop and merge it in jenkins before worrying about that part :-)21:13
Letozaf_balloons, well I still have my Nexus4, can wipe that :)21:13
Letozaf_balloons, sure21:14
balloonsright21:14
balloonsLetozaf_, a little sneak preview: http://people.canonical.com/~nskaggs/test-index.html21:15
Letozaf_balloons, woow!!! :)21:17

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!