/srv/irclogs.ubuntu.com/2014/03/04/#ubuntu-quality.txt

elopioballoons: still around?00:28
=== TheLordOfTime is now known as teward
wreckertheonlyhello?04:13
DanChapmangood morning jibel and elfy08:05
elfyhi DanChapman08:06
jibelHey DanChapman08:06
jibelmorning elfy08:06
elfyhi jibel :)08:06
DanChapmanjibel what would be the best way to apend additional repo's to a sources.list file say to enable universe. I have tried echo "universe" | sudo tee -a /etc/apt/sources.list but that appends to a new line which i don't want. How do i append to the end of the first line?09:22
DanChapmanjibel never mind, sed -e 's/$/ universe/' does the trick09:28
jibelDanChapman, that or sudo add-apt-repository universe10:00
davmor2Morning all10:59
DanChapmanjibel, what would the best way to get the ubiquity runner to work from a local directory. Would it be zipping it up and pack it in when preparing initrd then just unpack & move when run-autopilot kicks off on the other side?11:38
* DanChapman is getting bored of having to push EVERYTHING!11:40
jibelDanChapman, this is a solution, another is to use the vvfat driver of qemu, but that might interfere with the installer because it'll see an additional drive11:40
jibelDanChapman, let me try something11:40
DanChapmanjibel, awesome thanks :-)11:41
jibelDanChapman, I tried with vvfat but it makes the test fail because it is presented as sda and the partitioner takes the 1rst volume available. I'll try something else.13:30
jibelDanChapman, however it is interesting to note that ubiquity doesn't complain if you install ubuntu on the 512MB volume :)13:31
DanChapmanjibel, did it get past the 'preparing to install' page it usually should stop there with the next button becoming inactive13:33
DanChapmanjibel, well it did when i trialed it a while back. Should really get a testcase for that13:34
jibelDanChapman, it does because there is a second disk with the required amount of space13:34
DanChapmanjibel, thats interesting :-)13:35
jibelDanChapman, I'll try 9p instead of fat, just to see how the partitioner presents it13:35
jibelDanChapman, it works with virtfs and it is invisible to partman13:43
DanChapmanjibel, brilliant :-) thanks for taking the time to figure it out. Will be so much nicer working with it13:59
elopiodavmor2: the tests we are missing on the jenkins jobs are: online accounts ui, ubuntu system settings, ubuntu terminal, ubuntu weather.14:30
davmor2elopio: thanks I'll have a look if I can ever re-trace these crashes14:31
DanChapmanballoons, jibel, quick question are packages from the partner repository allowed for test dependencies on jenkins? disc0tech is working on some rhythmbox tests and wants to mock a radio station but currently depends on gstreamer0.10-fluendo-plugins-mp3-partner14:33
davmor2elopio: terminal will possibly be down to the fact that the backend needs to be rebuilt against 5.2.1 it only shows the shell and nothing else,  Weather app only shows the loading page and goes no further too so I will look if those are fixed now but I doubt they are14:36
jibelDanChapman, you can install any package from archive.ubuntu.com14:42
DanChapmanjibel, great thanks :-)14:42
jibelDanChapman, http://paste.ubuntu.com/7033346/ should save you some bzr pushes :)14:47
jibeland comments too14:47
DanChapmanjibel, excellent! thank you so much :-D14:48
jibelDanChapman, I'm doing few more tests to make sure I didn't break too many things and will push this14:48
jibelDanChapman, added option -D to r6127, ie run: ./ubiquity-autopilot-runner/run-ubiquity-test -D --sdl ~/iso/ubuntu/trusty-desktop-amd64.iso15:10
balloonsDanChapman, I assume we have a bit more to do then on https://code.launchpad.net/~dpniel/ubuntu-autopilot-tests/test-runner/+merge/206581?15:15
DanChapmanjibel thats brilliant thanks :-)15:38
DanChapmanballoons, howdy :-) yes it will also come in handy for that aswell. I'll look into it once i have got rid of this pesky bug in the custom install tests.15:40
senanballoons,DanChapman : helloo...is this code enough for reporting the bug http://paste.ubuntu.com/7033834/16:35
balloonssenan, I'm not sure exactly what the problem was you were encountering. But in general, the smallest amount of code that will show the issue is the way to go :-)16:36
senanballoons, its some object identification issue Scan Folder... is not detecting in autopilot316:38
senanballoons, I'm also not sure what description I should give. :(16:39
senanDanChapman, need your help ;)16:40
DanChapmansenan you can simplify that testcase to just simply selecting the menuitem there isn't any need to go clicking round the UI :-)16:48
senanDanChapman, like this http://paste.ubuntu.com/7033915/16:53
senanDanChapman, I'm not sure what description I should give16:54
senanpls help me on that too16:54
DanChapmansenan it doesn't need the test_open_remote_folder the issue is with the '...' / '\u2026' not being recognised/selectable in autopilot3. Also it could do with an assert just to check it's selected17:03
DanChapmansenan, just describe what the problem is and if you give me the bug number after i'll add any details that are missing17:04
balloonselopio, is there a good time to have a look at weather today?17:27
elfywhen the sun is up?17:27
DanChapmanballoons, weathers pretty grey here ;-p17:27
elfyoh dear17:27
elfyEnglish humour strikes twice :p17:27
* balloons shakes head slowly17:28
DanChapman:-D17:28
balloonsbelieve it or not, it's grey here too!17:28
elfyI bet our grey is greyer :p17:28
elfyactually I had sun - but shhh17:29
balloonsohh it surely is. But it's an odd day for us17:29
DanChapmanballoons, i'm reviewing the rhythmbox test atm. Could i get your thoughts on the utility function naming style?17:41
balloonsDanChapman, quick link to mp? I was just looking at https://code.launchpad.net/~adam-disc0tech/ubuntu-autopilot-tests/fileroller/+merge/20818317:42
DanChapmanhttps://code.launchpad.net/~adam-disc0tech/ubuntu-autopilot-tests/rhythmbox17:43
DanChapmanballoons, ^^17:43
DanChapmanthe majority of them are in base.py17:43
balloonsDanChapman, I prefer _ as leading char and I like camelCase17:44
balloonsDanChapman, I believe pep8 suggests function naming to be that way. However, I think most tests stick to _ anyway in the autopilot world17:45
balloonsin regards to this mp, it feels a bit weird, since it feels like he's mixing styles17:46
DanChapmanballoons, ok cool and what about the given/when/then flow of the tests?17:47
balloonsohh hehe17:47
balloonswell, I think we spoke a bit about how we'd like to see things; so that would apply here as well17:47
balloonsnamely, the object class model idea17:48
balloonstestsuite run ok?17:48
balloonsDanChapman, I think I can appreciate his desire to make tests more readable, but I would rather see it done a different way. I would support renaming and dropping the given, when, and then from the function names17:49
balloonscomments and logger debug statements can fill in instead.17:50
DanChapmanballoons, yes i agree. Ok well I will I have made a start on the eog test to see how it goes today which i can then show him as a reference. It ran ok but the combobox selection is a bit flaky though  (failed a couple of times)so that needs toughening up17:50
balloonsis disc0tech here atm?17:50
balloonsDanChapman, are we comfortable merging with the changes we mentioned above? Again, I think I would pursue the refactor as a separate mp and land this17:52
DanChapmanballoons, yes I agree I will make the recommendation I have for the combobox but apart from that it works good locally and in the runner17:56
balloonsDanChapman, I'll add a quick note as well. It looks like a nice testcase for a big app.. Good stuff ;-)17:56
elopioballoons: today is not good for me. I'm doing slides for tomorrow.17:57
elopioI had it planned for tomorrow, but if you can start earlier, even better.17:57
DanChapmanballoons, yes it's comae along nicely. He's actually working on improving the radio test case by mocking a radio station so it has no external sources which is cool :-)17:58
balloonselopio, ahh.. yes do prepare for your talk.. no worries18:00
senanDanChapman, balloons : Bug #128783418:07
ubot5bug 1287834 in autopilot-gtk "GTK elements with property contains '...' / '\u2026' not being recognised/selectable in autopilot3" [Undecided,New] https://launchpad.net/bugs/128783418:07
senanDanChapman, balloons : I've added bug number to the test and resubmitted the MP..Dan can you please take a look at the bug number and please add if I missed something18:13
=== bfiller is now known as bfiller_afk
=== WebbyIT is now known as rpadovani
=== bfiller_afk is now known as bfiller
=== veebers_ is now known as veebers
=== bfiller is now known as bfiller_afk

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