/srv/irclogs.ubuntu.com/2013/09/23/#ubuntu-quality.txt

=== chihchun_afk is now known as chihchun
pittiGood morning03:56
=== chihchun is now known as chihchun_afk
=== chihchun_afk is now known as chihchun
jibelGood morning06:47
DanChapmanGood Morning all06:48
elfymorning jibel DanChapman06:55
DanChapmanelfy, o/06:55
smartboyhwHello DanChapman, elfy06:57
jibelGood morning elfy07:05
jibelGood morning DanChapman07:05
pittijibel: while travelling on the weekend, and now I'm working on autopkgtest07:47
pittijibel: I trimmed the output format, added realtime watching of the tests, added a test suite, etc.; is there anything which you want me to look at while I'm at it?07:48
pittijibel: or adding tests for stuff that you often see fail?07:48
jibelpitti, not really. We've got bug 1227610 with upstart tests but it is not reproducible. This is annoying because it happens after the timeout has been reset, so autopkgtest never times out. But it happens rarely.07:51
ubot5bug 1227610 in autopkgtest (Ubuntu) "adt-run hung on 'tee -a /dev/stderr'" [Medium,New] https://launchpad.net/bugs/122761007:51
jibelpitti, and also the behaviour with an empty Depends field is undefined, currently adt-run fails. This is a common mistake, do you think it should fail with a clear message, or expand dependencies like @ or not install any depends at all?07:52
jibelpitti, on my side, I'm adding support for private PPAs to auto-package-testing07:53
pittijibel: I think it shouldn't install any depends at all07:53
pittijibel: good point, I'll add a test for that and fix this07:53
jibelpitti, +1, that's what I think too07:53
pittijibel: I'd also like to add a test for the mode that we call adt-run with, in particular with a permament output directory; what do we use for that again?07:54
pittijibel: --output-dir?07:55
jibelpitti, --output-dir and --tmp-dir07:56
pittijibel: oh, why --tmp-dir?07:57
jibelpitti, to collect all the artifacts generated by adt07:57
jibelpitti, oh, talking about tmp-dir, we should probably add a safeguard, if you run with --tmp-dir=/ (which can happen if a variable is undefined in shell) then it deletes /07:58
pittijibel: hmm, wouldn't that be --tmpdir= then?07:58
pittithat just ought to fail with "missing argument" indeed07:58
pittijibel: oh, and --output-dir alone doesn't even seem to work08:14
jibelpitti, yes, I don't remember exactly why I had to use tmp-dir instead originally08:15
jibelpitti, I fixed the missing distro-info in the testbed, a dependency has changed at some point and it is not installed by default08:16
=== vrruiz_ is now known as rvr
davmor2_Morning all10:01
=== davmor2_ is now known as davmor2
slickymastergood morning all10:04
rvrpitti: ping10:45
pittiplease don't ping, just ask :)10:46
pittihey rvr10:46
rvrpitti: So, maybe dumb question, but, the dependency for Unity in the phone is unity8-autopilot? (re: python-gi dependency)10:47
pittirvr: not sure what you mean with "dependency for Unity", but if you want to run the unity8-autopilot tests you need to install that, yes; and that ought to pull in python-gi10:47
rvrpitti: Well, so unity-webapps-qml  autopilot tests were programmed for the desktop and needs to run on the phone10:49
rvrin the desktop, it imports unity-autopilot emulators10:49
rvrin the phone, must import unity8-autopilot?10:49
pittirvr: depends, unity and unity8 are completely different codebases10:54
pittirvr: but unity (7) certainly doesn't run on the phone, so I guess you want unity810:55
rvrpitti: Right, should be obvious ...10:56
=== chihchun is now known as chihchun_afk
=== chihchun_afk is now known as chihchun
=== _salem is now known as salem_
=== mzanetti is now known as mzanetti|food
=== iahmad is now known as iahmad|afk
cgoldbergthomi, hi.. you back in NZ??13:19
=== mzanetti|food is now known as mzanetti
cgoldbergpitti, thomi.. hey.  I'm setting up an autopilot dev environment on a new machine.  I'm documenting the steps so I can add to the docs at /autopilot/faq/contribute.rst   ... question...13:36
pitticgoldberg: what  is there to be done other than calling bin/autopilot and setting PYTHONPATH in the source tree?13:37
pittithat's what I do, seems to work quite well13:37
cgoldbergpitti, dependencies13:37
cgoldbergsudo mk-build-deps -i13:37
cgoldbergbut that doesn;t get everything you need to run the functional self tests13:37
pittithat should be the dependencies of python-autopilot-tests, indeed13:38
cgoldbergso basically I mk-build-deps.... then run tests.. then install every paxckage it complains about missing13:38
cgoldbergwould be nice to have that documented or scripted13:39
cgoldbergpitti, so my question is.. is there a way to setup the selftest dependencies automagically?  or should I just list them in the doc?13:40
pitticgoldberg: TBH I'd just apt-get install python-autopilot-tests, that will give them all and we wouldn't need to duplicate them13:40
pitti(in documentation)13:40
cgoldbergahh.. didn't know such existed... so does python-autopilot-tests give you everything (recordmydesktop, python-junit, etc)?13:41
pittiyes, it should; if not, it's a bug13:41
cgoldbergpitti, perfect.. then that's what I'm looking for13:42
cgoldbergi'll make the docs clear13:42
pittiand if there will be new ones, you'll automatically get them on upgrade13:42
cgoldbergpitti, hmm... does python-autopilot-tests also give you all build-deps?  ... is there a need to run mk-build-deps?13:44
pittino, not the build deps13:45
cgoldbergk13:45
pitticgoldberg: are there any extra ones?13:45
pittiliblttng-ust-dev probably13:45
cgoldbergextra what?13:45
pitticgoldberg: build deps which are not already deps of p-a or p-a-tests13:46
cgoldbergnot sure.. im verifying p-a-tests now that I know it exists :P13:46
cgoldbergpitti, also.. should python-tox be part of p-a-tests?13:47
pitticgoldberg: I don't think so; the tests don't call tox, nor do you need it13:48
cgoldbergok.  I will just document it's use a s a convenience in the docs13:49
cgoldbergpitti, btw... the way autopilot doesn't show any progress during tests is killing me :)  I need to get it to stream results.. it would be sooo much better (IMHO)13:51
pitticgoldberg: something like -⅓v ? :-)13:51
pitti-v is rather verbose13:52
pitticgoldberg: yeah, it takes a while, and during that time your computer is pretty much blocked13:52
pitticgoldberg: we actually created autopilot-sandbox-run to run the autopilot tests for some app in xvfb/dbus-launch, so that it doesn't block your computer13:53
pitticgoldberg: that reminds me that we haven't actually tried running autopilot's own tests in autopilot-sandbox-run13:53
pittiif that works, it would already improve the situation quite a bit13:53
cgoldbergpitti, right.. i just want progress... normal mode should be test case name..... then pass/fail and exceptions on failures13:53
pitticgoldberg: but then again, if you work on something particular, you'd usually only run a single test case/suite, which is much quicker13:54
pitticgoldberg: right, I'd like that too13:54
cgoldbergpitti, i was going to add a TextTestRunner from unittest.. but thomi sai it might not play nice with verbose output.  any idea why?   I think I can make it work fine and give us much better output13:55
balloonsDanChapman, I think you've made all the changes I asked about in https://code.launchpad.net/~dpniel/autopilot-gtk/autopilotgtkemulators/+merge/185786 ;-)13:55
pitticgoldberg: i guess he meant that you shouldn't mix -v with the TestTextRunner, but I don't see why the latter wouldn't work without -v13:56
pitticgoldberg: (yay for triple negatives!)13:56
cgoldbergpitti, i'm gonna spend a few mins working on it... i think it will make it a much better experience for test running13:56
pittijibel: landed new autopkgtest FYI; just in case something blows up13:57
pittijibel: we should now be able to see the realtime test output in the jenkins console, which might be interesting for stuff that takes long (to see whther it hangs)13:57
jibelpitti, \o/13:59
pittidoes anyone know how to run app autopilot tests on the phone?13:59
pittie. g. gallery-app-autopilot isn't installed by default; i tried apt-get download gallery-app-autopilot, and running out of lp:gallery-app13:59
pittibut in all those cases it just breaks on Command '['which', '../../src/gallery-app']' returned non-zero exit status 114:00
pittithis might need a built source tree?14:00
jibelpitti, apt-get install gallery-app-autopilot then "autopilot run gallery_app" as user phablet14:01
pittijibel: our CI machiner actually does that?14:01
jibelthat's not from a source tree though14:01
pittiI had assumed it was using the readonly images14:01
jibelpitti, tha's how the daily release does14:01
jibelthat's14:01
pittiok, I'll try that, and to install build deps and run from a built tree14:02
pittiI do this to learn how existing tests works, before I start creating new ones14:02
pittimeh, you can't even install half of the build deps of dialer-apps14:11
pittiseems I need to go back to the cdimage images14:11
elopiogood morning!14:21
=== chihchun is now known as chihchun_afk
balloonsmorning elopio :-)14:27
smartboyhwHey balloons14:27
balloonshey smartboyhw14:29
elopiohey balloons. Want to review this one? https://code.launchpad.net/~elopio/ubuntu-ui-toolkit/toogles_emulator/+merge/18692214:34
balloonselopio, ohh nice.. toggles :-)14:34
balloonsso smartboyhw how's things?14:38
balloonsgetting some kubuntu stuff going?14:38
elfyafternoon balloons14:38
smartboyhwballoons, great14:39
DanChapmanballoons: Hey :-) How are you? Yeah i made the changes you mentioned and all tests are still passing, so all good just waiting on the other reviews now :-)14:39
balloonsafternoon elfy14:45
=== chihchun_afk is now known as chihchun
jfunkping elopio16:05
elopiojfunk: pong.16:06
jfunkelopio, PM16:06
=== zyga is now known as zyga-afk
elopioiahmad: can you please review this one? https://code.launchpad.net/~elopio/ubuntu-ui-toolkit/toogles_emulator/+merge/18692216:53
=== chihchun is now known as chihchun_afk
iahmadelopio, ok17:08
iahmadelopio, meanwhile would you please do the same as I fixed your comments https://code.launchpad.net/~iahmad/ubuntu-ui-toolkit/textfield_tests/+merge/18673717:10
elopioiahmad: of course.17:10
elopioiahmad: a comment for the future, we discussed about removing self.getObject17:11
iahmadelopio, replacing with what?17:12
elopioinstead of that, I think it's better to call self.app.select_single(ObjectType, objectName='object_name')17:13
elopioa lot more verbose, but makes it clearer what you are selecting. A little more future proof too.17:13
slickymaster\quit17:14
iahmadelopio, ok17:14
elopioiahmad: about your test_textfield_numbers, I would split it in two. Using test scenarios it would be a lot nicer, IMO. But I'm not sure if that's something you plan to do on the future branch that you mentioned on the reply to my comment.17:16
elopioit would be nice to have an EnterNumbersTestCase, with some scenarios with parameters like input and expected_output.17:17
elopioinput '123' -> expected_output '123', input '12.3' -> expected output '123', input '-123' -> expected output '123'.17:18
iahmadelopio, there are lot more test cases in my mind for input fields but I am at the moment just trying to do some smoke testing and cover as many features as possible in short time. otherwise all Boundary and Equavalence partitions test cases would have added.17:19
elopioiahmad: yes, that's what I thought. Just sayin' :) I left my approval on your branch.17:21
=== alecu is now known as alecu_errands
elopioiahmad: I was thinking of writing the textfields emulators next. Does that interfere with the things you are doing, or should I proceed?17:22
iahmadelopio, all suggestions are welcome ... thanks...:)17:22
iahmadelopio, go ahead, I think it should be fine17:22
=== salem_ is now known as _salem
jfunkelopio, are you able to see the "More Suggestions" in the applications scope on current?17:34
elopiojfunk: I am. Try performing a search.17:35
elopiojfunk: I suspect you are seeing bug #1225388.17:42
ubot5bug 1225388 in Unity 8 "Scope is blank even after connectivity is restored" [Medium,Triaged] https://launchpad.net/bugs/122538817:42
jfunkelopio, tx17:44
=== _salem is now known as salem_
=== alecu_errands is now known as alecu
=== Ursinha-afk is now known as Ursinha
=== salem_ is now known as _salem

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