smartboyhwGood morning Noskcaj01:19
Noskcajhey smartboyhw01:19
Noskcajany progress on testdrive? your version has permanently broke any testdrive i install01:20
smartboyhwNoskcaj, no.01:20
smartboyhwAnd I will depart for London tmr, so don't ask me anything until 31st July.01:21
Noskcajwait. london?01:21
smartboyhwNoskcaj, yeah.01:21
smartboyhwStudy tour01:21
DanChapmanGood Morning all :-)07:02
smartboyhwHey DanChapman07:07
DanChapmansmartboyhw, hey. How are you?07:08
smartboyhwDanChapman, good, packing for London07:08
elfygood day DanChapman07:10
DanChapmansmartboyhw, cool, well the weather is pretty good for us at the moment. How long is the journey?07:10
DanChapmanelfy hey :-)07:10
* elfy can't think of a worse place ... 07:11
elfy/wore wellies so they didn't mistake me for a local07:11
DanChapmanelfy I have a few autopilot tests together now for xubuntu, but wasn't sure where they are going.....07:11
DanChapmanelfy the ubuntu-autopilot-tests branch is now running on jenkins so putting xubuntu tests in there would just cause loads of failed tests. I ment to speak to balloons about it yesterday but i forgot07:13
elfyoic - no idea :)07:14
DanChapmannot sure if its a good idea to create a seperate xubuntu-autopilot-tests project07:14
elfythat's probably a conversation for when more than I'm about - just about to wander off for work too07:14
DanChapmanelfy no worries. I will stick what i do in a +junk branch until its decided on a plan of action when everyones about. Have fun at work :-)07:16
elfyknome: ^^ re xubuntu-autopilot-tests project07:16
elfyDanChapman: cheers :)07:16
smartboyhwHey elfy07:45
Noskcajsmartboyhw, you just missed hum07:54
smartboyhwNoskcaj, ouch07:54
knomeforestpiskie, ok, got that08:32
=== yofel_ is now known as yofel
chrisccoulsondoes anyone have any idea what is going on with https://jenkins.qa.ubuntu.com/job/saucy-adt-firefox/ARCH=amd64,label=adt/83/ ?09:33
chrisccoulsonjibel ^^ :)09:33
jibelchrisccoulson, this is a test timeout, now why it times out, I have no idea. I'll run the test again and log into the system09:36
chrisccoulsonjibel, thanks09:36
asacgema_: ho12:17
asacgema_: do you know why "default" is not run on http://reports.qa.ubuntu.com/smokeng/saucy/image/2898/12:17
gema_asac: looking12:18
asacgema_: oh ... i think it might be dashboard screwage12:24
asaci found "default" here: http://reports.qa.ubuntu.com/smokeng/saucy/image/2871/12:24
asace.g. ended up in the entry before12:25
gema_asac: no, I don't think so12:25
asachmm. y0ou are right12:25
asacits the "when its run" vs. "what image it ran on"12:25
gema_those runs are for yesterday, we made two for some reason12:25
asacso that one just ran late12:25
=== zyga_ is now known as zyga
asacyeah but if yo ulook there, the default has the time of Jul 912:26
gema_asac: they are grouped by image they run on, not by what time they run12:26
gema_if we run twice on a particular image12:26
gema_they get grouped together12:26
asacack. makes sense12:26
asacgema_: so maybe default for todays image is still running?12:26
gema_asac: I have found results for sdk and security but not for default12:26
gema_asac: I am trying to determine that in the internal instance12:26
gema_asac: but I didn't expect the other ones to run until default is successful12:26
gema_so I will talk to plars about that tomorrow (he is off today)12:27
asacgema_: here... on the autopilot stuff... is the infrastructure ready and you wait for someone giving you a list of what to run?12:27
gema_asac: not sure what autopilot stuff we are talking about , the normal test cases?12:28
gema_the integration autopilot test suite?12:28
asacautopilot tests for daily image runs12:29
asacso seems noone told you which ones12:29
asacrelated to that12:29
asaclet me get the list then12:29
gema_asac: we haven't got to the autopilot ones yet, I'd like to know which ones are the right ones for us to start adding12:29
gema_asac: i.e. the most stable ones12:30
gema_asac: I am going to get lost in the intranet for a bit, bb as soon as I figure out what happened to default jobs12:30
asacgema_: kk12:30
gema_asac: I am rerunning grouper, see if we get some default result for today12:36
gema_asac: it looks like the phablet tools got the wrong image for that job, but the right one for the other ones (makes no sense)12:37
gema_anyway, if that works I will rerun the others as well12:37
gema_and talk to plars about it tomorrow12:37
asacgema_: http://pad.ubuntu.com/jlIeQVyP8X12:38
asacthe canonical: ones are the goal12:38
asaci would like to group them by team/unity (so we dont have so many jobs)12:39
asace.g. one job for all applications ... one job for all shell12:39
asacone job for sdk12:39
asacgema_: maybe phablet tools does too much implicit magic and you rather want and explicit "list available images" and "download exact image by id"?12:43
asaci can imagine if you just pull "current" that there might be interesting races12:44
=== _salem is now known as salem_
gema_asac: that'd be good, but since there are so many moving pieces atm, we didn't want to reinvent any wheel13:04
gema_asac: we go with the flow and test phablet tools as well13:04
gema_asac: got a result there , it should be in the dashboard soon (within an hour)13:13
gema_asac: I will rerun the others as well13:13
gema_asac: thanks for spotting that problem13:13
gema_asac: we have a theory, we think that the md5 file changed before the image itself in cdimage13:29
gema_asac: so our jobs started thinking that the new build was already there13:29
gema_and the default jobs installed the previous one, rather13:29
gema_asac: by the time we got to the other two, the new image was fully copied and was picked up properly13:30
gema_asac: I will discuss with plars tomorrow13:30
balloonsDanChapman, elfy it's ok to push stuff to the ubuntu-autopilot-tests branch.. only the "prod" branch is being run in jenkins -- hence the reason it exists13:40
smartboyhwGood evening balloons13:47
balloonsevening sm0x13:47
balloonsbah.. evening smartboyhw13:47
smartboyhwballoons, :) Well, maybe you should give us a non-production Jenkins instance to test things;P13:47
balloonsthat's your local box eh? :-p13:48
smartboyhwballoons, whoa? No, buy one yourself:P13:48
gema_asac: results are published for the default jobs already14:17
gema_smartboyhw: if I get you a jenkins instance, how many test cases are you going to give me in return?14:18
smartboyhwgema_, well ask elfy and DanChapman14:18
gema_smartboyhw: you asked for it, so I am asking you :P14:18
smartboyhwThey are the ones who will push things to ubuntu-autopilot-tests branch, I might add few though14:18
smartboyhwStill learning14:18
gema_smartboyhw: ok14:18
smartboyhwgema_, expect that elfy and DanChapman will try to utilize Xfce autopilot testcases:)14:19
smartboyhwWhich will be a lot;P14:19
gema_balloons: keep an eye on these guys and let me know whenever you think they are ready to tame a jenkins14:20
* balloons is silently watching :-p14:20
smartboyhwballoons, lol14:20
gema_smartboyhw: he is watching you get caught on the spider web I am preparing :P14:23
gema_smartboyhw: having a jenkins is a daily task and it takes time14:24
gema_smartboyhw: and keeping it running happily requires a lot of dedication14:24
gema_smartboyhw: we'll start running your tests in ours14:24
gema_and eventually we may get to a point where it makes sense for you guys to have yours14:24
smartboyhwgema_, well what, I am not that sticky (at least, not to spider webs)14:26
smartboyhwgema_, it would be a great idea if we can run tests at your Jenkins instances14:26
smartboyhwThat will surely encourage more people to write autopilot tests14:27
jibelsmartboyhw, we already run tests from the production branch, so if tests from the dev branch are good enough they'll surely be promoted to production and run daily14:27
smartboyhwjibel, oh great14:28
smartboyhwDanChapman, chilicuil add oil on making your autopilot tests stable;P14:28
chilicuilgood morning smartboyhw o/14:29
smartboyhwhey chilicuil14:29
jibelsmartboyhw, https://jenkins.qa.ubuntu.com/job/autopilot-ubuntu-applications/14:30
balloonssmartboyhw, as jibel said.. the biggest piece with running unstable tests is you have to spend time fixing them when they break.. we're already stepping out by maintaining the set running in the production branch :-)14:30
smartboyhwballoons, jibel sure:)14:30
asacgema_: cool. guess we dont know the reasons?14:37
gema_asac: yep, if you read the log you'll see14:38
asacgema_: what will i see from the log? that we dont know the reason?15:02
gema_asac: this channel, I told you a while ago our problem with the jenkins trigger15:03
gema_asac: that's the reason why the jobs didn't run with today's image15:03
gema_asac: not the job's log, this channel's log :)15:03
DanChapmanballoons, jibel, smartboyhw  hey o/15:16
smartboyhwDanChapman, make your autopilot tests stable and get us a specific Jenkins bot!:P15:17
balloonsDanChapman, hello15:19
=== om26er_ is now known as om26er|dinner
DanChapmanballoons, just seen on scrollback about putting xubuntu tests in trunk. so should i put what i have in a seperate xubuntu_autopilot_tests directory rather than sticking it in with the ubuntu_autopilot_tests and them getting 'jumbled up' so to speak15:25
balloonsDanChapman, we can if we wish. I'm not opposed to a separate project eitheir persay, but as we talked about having things in one place makes it nicer..15:26
balloonswe can organize things however we wish. That said, there is debian packaging in the branch that will need to be adjusted. I don't know that we'll ever publish binaries again, but we did at one point15:27
balloonsso similar to manual tests, make a folder and drop them in15:27
balloonsI think that makes sense if they get xubuntu specific.. but gnumeric runs on ubuntu, lubuntu, etc too.. I run it for instance on unity alot :-)15:28
* smartboyhw likes gedit or nano15:28
DanChapmani was thinking rather than have it in ubuntu-autopilot-tests/ubuntu_autopilot_tests/ stick them in ubuntu-autopilot-tests/xubuntu_autopilot_tests/ then the whole suite can be run from the top dir depending on distro. But they all still stay together15:29
balloonsI think that makes fine sense15:32
DanChapmanballoons, cool i will do that then.15:40
smartboyhwballoons, private message?15:41
knomehey balloons16:09
=== om26er|dinner is now known as om26er
balloonshey knome16:13
knomeballoons, what was the reason why upgrade tests had their own product again?16:14
balloonsknome, lol.. you love to ask me questions I've forgotten answers to16:14
knomeyou know, somebody has to16:14
knomethere needs to be challenges for you16:14
balloonsknome, lol..16:20
balloonsknome, "That'd be because a respin of an image shouldn't reset the upgrade16:20
balloonsresults. The upgrade products are typically reset all at the same time16:20
balloonswhenever a massive change happens that affects upgrades."16:20
balloonsI quoted Stephane's answer from last time.. and indeed he's right16:20
knomeyes, and that still makes sense16:23
forestpiskiethanks balloons - read that reply :)16:39
knomeballoons, stgraber: it would be nice though if the xubuntu product filter would show the xubuntu upgrade tests as well though...16:43
knomeballoons, stgraber: or alternatively, if the upgrade tests had more "weight" and always sunk to the bottom16:43
dkesselgood evening :)17:26
DanChapmanevening dkessel17:27
=== psivaa is now known as psivaa-afk
balloonsdkessel, :-) I was going to ping you today to see how things were going17:31
dkesselballoons, well here i am :)17:32
balloonsdkessel, indeed :-) You going to make the workshop today?17:37
* DanChapman having a tough day with ALL virt solutions. 17:39
dkesselballoons, that's why i am here. i am starting with http://developer.ubuntu.com/resources/tutorials/quality/how-to-write-autopilot-tests/17:42
balloonsI'll see everyone in 20 :-)17:43
balloonsalrighty, time for our automated testing workshop to begin today :-)18:00
balloonsWelcome to anyone who's joined us to learn more about autopilot test writing18:01
balloonsWhat I'd like to do is have a brief introduction, then open it for questions from anyone who wants help. If your working on something and need specific help, let's work through it together18:01
balloonsif you wish, g+ hangouts are also an option if needed. So without further ado, let me introduce things18:02
AlanODI'm just starting out, so it's all new.18:02
balloonsFirst I'll give a little introduction on what autopilot is, and what we're doing.18:02
balloons Autopilot itself is a functional testing tool allowing us to interact with an application the same way a user would. it can click, swipe, touch, and type in an application by simulating a user18:02
balloonsThis allows us to use it to pretend we're a human using the application, and thus do functional testing18:03
balloons So, we want to bring this tool to the core apps project. The core apps are written by community developers and represent the core applications for the ubuntu touch platform.18:03
balloonsApps like the calculator, calendar, terminal, weather, and games too like sudoku :-)18:03
balloonsSo, in order to help contribute tests there are a couple things you'll need18:04
balloonsthe first is an installation of ubuntu saucy or raring. It can be in a VM or installed on physical hardware18:04
balloonsthe second is an understanding of autopilot, which is what we're here today to help with :-018:04
balloonsNow to get started learning about autopilot, specifically for the core apps, go through the tutorial on developer.ubuntu.com,  http://developer.ubuntu.com/resources/cookbook/mobile/how-to-write-autopilot-tests/18:05
balloonsif you haven't done that yet, that's your first stop ;-)18:05
balloonsit has an example application you can run and see the tests, along with an introduction to how the tests work18:05
balloonsNow, anyone who has gone through the tutorial, good :-) Let's talk about writing a test for a core app now18:06
balloonsI'll be talking through similar things as the tutorial here:  https://wiki.ubuntu.com/Touch/CoreApps/Testing/ContributeAutopilotTestcase18:07
balloonsIn a nutshell, you'll want to pick a core app and feature to write a test for18:07
balloonsfortunately, there is a handy list you can see on this page: https://wiki.ubuntu.com/Touch/CoreApps/Testing18:08
balloonshere's the list of open tests needed: https://bugs.launchpad.net/ubuntu-phone-coreapps/+bugs?field.tag=needs-autopilot-test18:08
balloonsthat includes all the core apps :-) the page itself has them broken done for you by applicatio18:08
balloonsok, so you have your feature and core app. Next, grab the source code for the core application18:08
balloonsyou can find handy links to the launchpad project pages where you can find the source on the core apps wiki18:09
balloonsso for instance, the weather app page is here: https://wiki.ubuntu.com/Touch/CoreApps/Weather18:09
balloonsclicking through the launchpad page shows me the bzr branch is :  bzr branch lp:ubuntu-weather-app18:10
balloonsSo, you've branched the code, you've picked a test to add. The last step is to write it18:11
balloonsI approach writing the testcase by running the application and exploring the feature first18:11
balloonsonce your ready, go through and test the feature. Write down each action you take, in english (that is plain words, not code), and not what happens as you perform each action18:12
balloonsthis forms the basis of what you will do in your test, and then what you will "assert" about those actions18:12
balloonsSo let's quickly look at the tutorial for an example of what I mean18:12
balloonsInside the test_clear_button function we do a simple action, namely, clicking the clear button18:13
balloonsfrom the user perspective the expected result is to see the fields have been cleared18:13
balloonsarmed with this knowledge we can write the autopilot test18:13
balloonsthis is how it looks:18:13
balloons#click the clear button18:14
balloons#confirm fields have been wiped18:14
balloonsself.assertThat(fromField.text, Eventually(Equals('0.0')))18:14
balloonsself.assertThat(toField.text, Eventually(Equals('0.0')))18:14
balloonsSo we click the button using autopilot, then we use asserts to make sure the screen is cleared afterwards18:14
balloonsthat is the basics of a good autopilot test. interact with the UI to expose the feature, then issue an assert(s) to ensure the feature reacted properly18:15
balloonsOk, enough of my rambling --  Let's talk about what questions you might have and help you get contributing ;-)18:15
balloonsif you have a question or want help, it's an open channel so no need to raise your hand or anything.. type away :-)18:16
knomeaway :-)18:16
* knome bows18:16
balloonsknome, we both know your behind any help I can give you :-p18:16
* balloons hides sheepishly18:16
knomei think you missed my joke... (you said "type away :-)")18:17
balloonsknome, I very much enjoyed it.. well done18:17
knomethank you sir18:17
knome(now that i have broken the ice, i'll leave the floor to more serious questions)18:17
AlanODThis seems fairly straight forward, I need to read your links prior to starting to write any tests.18:18
dkesselstrange... no matter what app i run, unity always shows the "clock" app's icon... :)18:18
AlanODDo your links cover checking test cases in?18:19
balloonsAlanOD, wonderful. Feel free to go through the tutorial now if you haven't and ping me as you have questions :-)18:19
balloonsAlanOD, they do cover contributing things back. The bottom of the https://wiki.ubuntu.com/Touch/CoreApps/Testing/ContributeAutopilotTestcase covers using bzr, setting up your machine, etc18:19
AlanODI think my machine is set up correctly, but I'll confirm that.18:19
balloonsStill, if you get stuck, that's what we're here for. The first time committing can feel confusing or daunting, but we'll help you through it. You learn how it works quickly :-)18:20
dkesselanyway. balloons, I am going to try and fix bug 118873618:20
ubot5bug 1188736 in Ubuntu File Manager App "Autopilot Testcase Needed: Test opening context menu on directory" [Undecided,New] https://launchpad.net/bugs/118873618:20
balloonsdkessel, ohh excellent18:20
balloonsiBelieve is the other person writing file manager testcases, so there's a nice test structure already setup in the application source :-)18:21
balloonsit's much easier if your going through things for the first time18:22
balloonsdkessel, and AlanOD any feedback on the tutorial on developer.ubuntu.com feel free to toss my way.18:24
AlanODI need to leave now. I'll start working through the tutorials tonight. I should be able to start writing a few cases tomorrow.18:24
AlanODIf/when I run into questions, I reach out to you, or the community.18:25
balloonsAlanOD, no worries. I hope this helps you get started. See my pm if you would so I can keep in touch with you, if that's alright18:25
dkesselballoons, is it convention to put all tests in one .py file?18:25
balloonsAlanOD, yes, #ubuntu-autopilot, #ubuntu-quality and #ubuntu-touch are all good places the community hangs out. core app decs in -touch, autopilot hackers in -autopilot, and well us here in -quality :-)18:25
balloonsdkessel, you can split them out or group them depending. I know clock for instance has one file for each view containing all the tests in that view18:26
balloonsthat seems like a nice way to split things18:26
adegoodyerHello everyone18:29
balloonsadegoodyer, hello. how's today treating you? :-)18:30
adegoodyerballoons, Good as always, the sun is out and life is well! How about yourself?18:30
balloonsmuch better today than last week.. The last couple weeks were crazy.. Now life is good again.. time to breathe and enjoy. And see some new tests get written and sharing the joys of automated testing.. It's good18:31
xeranashi, I wonder which best way to debug autopilot tests/emulators?18:32
adegoodyerballoons, That's great to hear! Yes, we all like to be busy but not TOO busy - that's just no fun at all. Still could be worse :oP18:33
balloonsxeranas, hello.. Always such good questions!18:33
balloonsadegoodyer, indeed, it could ALWAYS be worse.. perspective eh?18:33
adegoodyerballoons, thats18:34
adegoodyer.. it, all in the perspective lol18:34
adegoodyerRight, I have decided I am not leaving this desk until I write an autopilot test this evening! I have chose bug #1199112 as my first... so wish me luck!18:35
ubot5bug 1199112 in Dropping Letters "Autopilot Testcase Needed: test starting a new game" [Undecided,New] https://launchpad.net/bugs/119911218:35
balloonsxeranas, so I don't use  a step through debugger or anything like that when I'm trying to figure out how to write or run a test. I will use the autopilot vis tool to examine the application and then just run my test to see if something is broken18:35
balloonsxeranas, that said you can use dbus tools to examine the dbus stack if you think something is really broken.. say like d-feet18:35
balloonsxeranas, does that answer your question or ?18:35
balloonsadegoodyer, :-) That's the ticket! I'll be share to share it when you do as well! I'm going to highlight that app on friday, and you can be a part of the highlight ;-)18:36
balloonssure to share, not share to share18:37
balloonsmy typing is so bad recently18:37
adegoodyerballoons, Oh no don't say that! ... No pressure!! :-O18:38
xeranasballoons: what is autopilot vis tool?18:38
balloonsadegoodyer, no no no.. of course not! The post is written, it would be more work for me to edit it now :-P18:38
balloonsxeranas, ohh excellent. Let me share with you the tool and how it works18:38
adegoodyerballoons, Haha, that's good to hear... of course if my test is simply mind blowing then feel free to use... but I doubt that it will be very much lol18:39
balloonsxeranas, so, lets say I wanted to see what the sudoku app made available to autopilot at runtime18:40
balloonsI would do this: autopilot launch -i Qt sudoku-app18:41
balloonsthen, autopilot vis18:41
balloonsafter the vis tool laucnhes, select the QtQmlViewer connection from the dropdown and you'll be seeing what autopilot sees18:42
balloonswould it help for me to show you via video?18:42
adegoodyerballoons, What does the -i parameter in the autopilot command do?18:42
balloons-i tells autopilot what introspection library to load. It tries to guess automatically and usually gets it just right so it's not needed18:43
balloonsI like to be specific, lol, so I sometimes define it18:43
adegoodyerAh, I am with you then... so Qt is the library right??18:43
balloonsyes, exactly18:43
balloonsyou could also do -i Gtk18:44
adegoodyerballoons, cool, that's simple but really good to know18:44
balloonsand you may not no if the app is Qt or Gtk.. hence it's not needed to use autopilot, but handy in case you need to help autopilot figure it out :-)18:44
xeranasballoons: yea video will be fine too. Maybe I do something wrong but in tool dropdown only Unity exist18:44
balloonsxeranas, alright, let me try streaming quickly and I'll show you :-)18:45
balloonsxeranas, so in just a moment, I'll be up here: http://youtu.be/lTBNOkrzEC418:47
balloonsok you should see me now :-)18:49
adegoodyerSo cool ^^ :-)18:49
adegoodyerGuess who was trying to test using the old version of autopilot... doh!18:50
adegoodyerballoons, My apoligies, what's the command to launch sudoku-app in QML viewer again?18:53
adegoodyerballoons, in autopilot of course..18:54
adegoodyerballoons, thank you! :o)18:55
xeranasthis tool is so time saver!18:56
xeranasballoons: thanks for showing18:57
dkesselsomebody already did my test18:57
dkesselbut named it "test_file_actions_shows"... while it shows the directory context menu18:58
adegoodyerxeranas, I attempted autopilot tests a while ago without the 'vis' tool and it was absolutely impossible.18:58
dkesselhm. they seem identical anyway18:58
adegoodyerdkessel, if there identical then you definitely are on the right track at least :o)18:58
balloonsdid that help?19:00
balloonsthe plugin crash so the stream ended, lol19:00
adegoodyerballoons, Yes definitely helped me. It's the silly small things that take time sorting out so when  you watch someone else breeze through, you learn a lot quicker.19:01
balloonscool..  I can share more if you have more questions :-)19:03
balloonsdkessel, ahh.. sorry mate!19:04
balloonsbut yes, if they are identical, well you've done well in writing it!19:04
balloonswas the bug not updated to say it was done?19:04
balloonswe should do that if so :)19:05
dkesselballoons, now anyway... i have written a test that also tests for the context menu on files....19:05
balloonsohh nice.. I was just playing with the file manager app today19:05
balloonsit's come along quite far19:05
dkesseli guess there must have been a bug for the context menu on files... for which the author built a test for the context menu on FOLDERS19:05
dkesselnow i have written the one for the file and renamed the old test19:06
dkesselhm, i get an error when trying to list the tests... let me paste it19:06
balloonsahh that makes sense.. the functionality might not have been implemented19:06
dkesselwhat am i doing wrong here? http://paste.ubuntu.com/5859314/19:07
dkesselah, the test has a dependency on the mock module.... meh19:08
balloonsheh.. I was just going to mention that19:10
balloonspython import yelling at you19:10
adegoodyerballoons, Before writing a test, would you recommend running apt-get update to update the apps? Is this the best way to keep them updated??19:10
balloonsadegoodyer, since you messing with the code directly, I would run them from the branch you've checked out and are messing with19:11
dkesselballoons, how do i know that module is available when the test is run i.e. on jenkins?19:11
adegoodyerballoons, Yep, cool19:11
balloonsyou might have modifications to the qml files for instance adegoodyer19:11
balloonsdkessel, fginther has been adding modules as needed to run on jenkins, and yes we've already had some surprises19:12
balloonsideally packaging and adding dependencies in debian control file is the right way to go19:12
balloonsfor now, we just work through it dkessel19:13
balloonsLetozaf_, are you still about/19:15
fgintherdkessel, jenkins builds in a very minimal chroot environment, so any missing build dependencies should be discovered during build. Test dependencies are a little harder as the autopilot environment is not quite as minimal. So if a test has a package dependency, it should be in the control file, catching these is a little difficult.19:16
fgintherdkessel, ideally we don't want to add packages to the build environment, we want the packages themselves to pull them in.19:17
dkesselfginther, that's what I guessed you would want. Thanks for clarifying19:18
fgintherdkessel, you're welcome19:18
balloonsfginther, since I've called you in as well, should the missing dependency for rssreader be fixed?19:19
fgintherballoons, are you referring to this? https://code.launchpad.net/~carla-sella/ubuntu-rssreader-app/ubuntu-rssreader-test-add-view-feeds/+merge/17331619:20
fgintherballoons, If so, I did add the missing ubuntu-sdk-team/ppa, but ran into different errors.19:22
* fginther needs to clone self19:22
balloonsfginther, indeed I am19:23
dkesselballoons, I am done. I have run the test to check that it works, and I have pushed into a lp branch. Is there anything else to do before suggesting a merge?19:24
balloonsdkessel, that should be it. Propose the merge and the devs will review.. since I'm here and your proposing I can just review :-019:24
balloonsonce approved jenkins will try running it and if all the existing tests pass, aka, you didn't break anything jenkins will merge it19:25
balloonsso issue an MP, I'll have a look and we'll see if we can't get it merged :-019:25
dkesseldone :)19:26
Letozaf_balloons, sorry was away for a while :)19:26
Letozaf_balloons, I'm back19:26
dkesselballoons, https://code.launchpad.net/~d-kessel/ubuntu-autopilot-tests/test_opening_context_menu_on_directory19:26
fgintherballoons, Letozaf_, I'm taking another look at https://code.launchpad.net/~carla-sella/ubuntu-rssreader-app/ubuntu-rssreader-test-add-view-feeds/+merge/17331619:26
Letozaf_fginther, thank you :)19:27
balloonsLetozaf_,  no worries, see fginther response above. basically I tried again to merge things, but yea, still working on it ;-)19:27
Letozaf_thank you to both :) then19:27
balloonsLetozaf_, are you working on more tests then now?19:27
Letozaf_balloons, I am looking at the stock ticker19:27
Letozaf_balloons, app19:27
balloonsdkessel, I'm on it! Ohh one thing.. Unlike the desktop apps, we push these tests directly into there source trees and not the autopilot project19:28
balloonsso you should push that merge to lp:ubuntu-filemanager-app19:28
adegoodyerballoons, Disaster! All the object names in dropping-letters are black according to vis??19:33
adegoodyer*blank even19:33
dkesselballoons, i fixed the target branch19:33
balloonsadegoodyer, let's rollback a moment so I understand :-)19:33
balloonsdkessel, :-)19:33
adegoodyerballoons, My apologies! I have chosen the 'test to start game' testcase for the dropping letters game...19:34
balloonsadegoodyer, right, so you've figured out how to click the start button eh?19:34
balloonsbut you want to assert the screen is cleared of letters new and the score reset or ?19:35
adegoodyerballoons, Well not exactly, I don;t seem to be able to refer to the play button directly as when I introspect with vis, all the objectName values are blank?19:35
balloonsI guess the english version first is good :-)19:35
balloonsadegoodyer, ohh, heh.. ok,19:35
balloonslet me try launching vis and seeing what you see19:36
adegoodyerballoons, ok19:36
adegoodyerballoons, You ok with the command? :oP19:36
balloonsdkessel, I forgot the file manager app kind of dislikes me19:38
balloonsadegoodyer, ok all loaded up19:39
balloonsahh so I understand now what your saying :-)19:39
balloonsyou get to define the objectName in the qml yourself19:39
adegoodyerballoons, You see?19:39
balloonsso literally whatever object you want, go add a name for it so you can select it during runtime :-)19:39
adegoodyerballoons, Oh, how do I do that? Sound cool19:40
balloonsso yes, we get to edit the qml to add that property as needed :-)19:40
balloonsright so dropping letters has only 1 qml file I think19:40
balloonslet's see19:40
adegoodyerballoons, That's right19:41
balloonsopen it up and have a look.. go ahead an add the objectName property to anything you want to get at :-)19:41
adegoodyerballoons, So it's ok to edit the actual qml file to get the relevant objectNames and then update this upon committing?19:42
balloonsdkessel, so let me set it to approved and jenkins should merge it19:42
balloonsI see some toolbar buttons for new game19:42
dkesselballoons, thanks :)19:43
balloonsyour most welcome :-) so, you have background on gtk vs qml.. what are your thoughts comparing the 2?19:43
balloonsadegoodyer, so I would make names for the play button and the newgame button19:44
balloonsthen you'll need to figure out how to assert properly that the screen clears and game resets when you click it19:45
adegoodyerballoons, Yep am totally on it now, for some reason I didn't think we could edit the source like that19:45
balloonsso there is a Timer you see in the qml with a tickCount that gets reset.. add an object name to it too19:45
balloonsadegoodyer, if you don't specify the way it works is it's given a random name at runtime19:45
balloonswhich doesn't help us :-)19:45
adegoodyerballoons, Ah, didn;t realise that at all, which is also helpful. C'mon developers - it's only one line of code for each object and would sure help us folks out :oP19:46
balloonsdkessel, jenkins bot slapped you :-) https://code.launchpad.net/~d-kessel/ubuntu-filemanager-app/test_opening_context_menu_on_directory/+merge/17380219:48
adegoodyerballoons, Can we not introsospect using id??19:48
balloonspretty cool though eh? you can thank fginther and team for putting it together..19:48
balloonsadegoodyer, sadly afaik no.. I'm not a qml guru, so I'm unsure why the distinction between the too, but ;-)19:49
dkesselballoons, funny how it requires input into a field that is marked as optional on the website...19:49
balloonshmm indeed, that is odd.. seeing as the site too says it should be used for merging19:51
balloonswhy the (optional)?19:51
balloonsI suppose to not force things upon anyone.. it *should* be used19:52
fgintherdkessel, balloons, launchpad allows for a more flexible workflow. When we started building the jenkins environemnt, we wanted a distinction between the commit message and the description. So we can enforce this in the jenkins tools, but we can't change launchpad.19:53
dkesselfginther, and what i just did is that i enterend the exact same text into both fields ;)19:54
fgintherdkessel, yeah, that's usually how it goes. Some people like to put testing notes or other info in the description. So it's not always the same...19:55
fgintherbut we all use the tools differently19:55
dkesselfginther, will the lp bug be closed automatically once a new package with my changes is released?20:06
=== TheDrums_ is now known as TheDrums
fgintherdkessel, I think it will be marked as 'fix committed'20:07
balloonsyes it will mark as fix committed20:08
fgintherdkessel, correction. It is marked 'fix committed' when the merge proposal is merged in. I also think that it will go to 'fix-released' when the package is released to distro.20:08
fgintherballoons, can you correct me on that ^^20:09
balloonsfginther, you've got it correct20:09
balloonsfor these apps, I don't think you'll ever see the fix-released because they are going into the ppa :-)20:09
dkesselboo ;)20:10
balloonsaloha thomi20:12
SergioMeneseshi guys!20:43
SergioMenesesballoons, take a look when you have time enough https://code.launchpad.net/~sergiomeneses/ubuntu-manual-tests/ubuntu-manual-tests/+merge/17381120:43
balloonsSergioMeneses, sure thing. I owe Noskcaj merged too :-)20:43
balloonsI'll do a big ball of them tonight :-)20:44
SergioMenesesballoons, it is a big testcase, maybe 2 reviews will be necessary20:44
SergioMenesesballoons, perfect20:44
NoskcajSergioMeneses, i'll have a look at it, first issue is the line "<dd>you should be redirect to the manual page in your browser <a href="http://apps.ubuntu.com/cat/tos/">http://apps.ubuntu.com/cat/tos/</a>, do you?</dd>"20:45
SergioMenesesNoskcaj, lol20:46
SergioMenesesthe wrong file20:46
SergioMenesesNoskcaj, but is ok... try to check the rest20:47
SergioMenesesI didnt include the link in the final test20:47
* SergioMeneses looking for it20:47
Noskcajit's part of the diff for some reason20:47
SergioMenesesNoskcaj, because I merged the wrong file jeje20:49
SergioMenesesthat's the reason :)20:49
SergioMenesesbut it doesnt matter, please check ( and balloons ) the rest of the test20:49
SergioMenesesNoskcaj, that link was just a copy20:49
NoskcajSergioMeneses, one thing to remember when writing testcases: use a spellchecker20:55
Noskcajalso, your <dd> and <dt> are done all wrong20:56
SergioMenesesNoskcaj, what happen with dd and dt? I've changed them21:00
Noskcajdon't write them as <dd>does this happen?</dd> write them as <dd>This happens</dd>21:00
SergioMenesesNoskcaj, ok got it21:01
Noskcaji also saw a few instruction in <dd> tags21:02
dkesselballoons, I got to go. I'm glad I could contribute something. Will try to do more stuff next week ;)21:08
balloonsdkessel, thanks for hanging out and contributing ;-)21:08
balloonsi'm sure we'll see more21:09
balloonsadegoodyer, btw, how are you coming along?21:09
adegoodyerGot sidetracked a little and it's getting late here so I think i'll probably finish up early tomorrow whilst waiting for the Dev app hack day to start.21:11
adegoodyerThank you very much for the tips and advice though, help me no end! :o)21:11
balloonsadegoodyer, ;-)21:12
kanliothow do i see if MIR is running?21:19
adegoodyerkanliot, Good question? Plymouth crashed when I booted which told me it was probably running :o)21:20
kanlioti had good results, except for ubiqutiy in the daily for lubuntu desktop 64bit21:21
adegoodyerkanliot, apart from that issue I have to say I saw no real difference, which of course is a very good thing21:26
SergioMenesesNoskcaj, I'll upload the new test, thanks for your time :)21:46
Noskcajno problem21:47
=== salem_ is now known as _salem

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