=== salem_ is now known as _salem [05:08] Good afternoon Noskcaj [05:20] Noskcaj, are you sure you want to do the Google+ Hangout? I heard from balloons that you plan to do it using audio, but it would be no use if audio only [05:38] smartboyhw, I don't plan to do audio only. My internet is better now [07:56] Good Morning all :-) [09:19] good morning all === iahmad is now known as iahmad|afk === iahmad|afk is now known as iahmad === _salem is now known as salem_ [13:22] is there an error tracker handbook somewhere? I was looking at the error reports for a package and I'm clueless what that graph is trying to tell me [13:24] yofel, better ask at #ubuntu-devel... [13:26] well, won't hurt I guess [14:35] hey smartboyhw [14:36] I saw jackson is thinking he can do video again, cool [14:38] balloons, good morning :-) [14:38] DanChapman, morning :-) [14:41] balloons, DanChapman my thoguht on Edge http://smartboyhwubuntu.wordpress.com/2013/08/07/ubuntu-edge-target-too-high-but-a-phone-that-one-will-want/ [14:44] smartboyhw, hmm.. there's not denying there usefulness.. wants and needs however are 2 different things. I think your spot on with much of your assesment [14:48] balloons, :) [14:57] balloons, stop to change status to bugs, you're filling my mailbox :P [14:58] WebbyIT, I just closed like 50 bugs.. hehe [14:58] balloons, 45, to be exact! ahahahah [17:03] so fginther elopio om26er since your all in this channel, I'm going to put together a blog post of best practices for autopilot tests. We've been talking about it for some time, and I know when we "fix" tests, it's what we're really doing. So, with that in mind, what should I mention? I have the following already [17:03] use eventually, more asserts (for all interactions!), don't use strings, use objectNames [17:05] balloons, wait for the interface to stop moving before clicking objects, either check its .animating property (if it have) or .moving [17:05] om26er, ohh good one === alesage is now known as alesage|lunch === alesage|lunch is now known as alesage [17:43] fginther, om26er elopio http://www.theorangenotebook.com/2013/08/autopilot-best-practices.html. om26er thanks for the input :-) [17:52] balloons, ;) [18:00] balloons: nice. I'm late, but for me, the main thing is to keep the tests with one action-verification. [18:04] yes.. tedious or not, do something, assert something is the way to go [18:11] balloons, have you got roughly 203.632s spare to run my gedit branch? would like your thoughts so far :-) [18:12] DanChapman, of course I do! [18:12] I'm curious to see what you've been able to do :-) [18:15] Well so far ive been doing all the keyboard shortcuts. Im moving on to using mouse etc now. [18:19] DanChapman, ok, branching now.. let's see what we've got [18:31] DanChapman, well the tests still pass, so that' something :-) [18:33] test_end_key_goes_to_end_of_line is a clone of home test ;-) [18:34] balloons, yeah i wasn't sure with that one, was thinking of trying a different route. But it passes atm so i'll leave that till later [18:34] otherwise, that looks like your being pretty thorough on testing the keyboard shortcuts. you plan to test the default buttons as well, then the basic save and open? [18:35] DanChapman, well on the test_end_key_goes_to_end_of_line you could make it similar to home and send an end key + backspace then check the last letter is gone [18:35] balloons: I've not bothered much even looking at testcases that aren't xubuntu up to now - but I can now that the majority of mine are done [18:36] elfy, if you wanted to review the synaptic contribution, go for it. It would be helpful. I can then sync everything after you merge it [18:37] balloons, yeah I didn't want to try going through whole processes of saving a file etc just with shortcuts. I will do more using default buttons etc. I just thought it was nice that while we can see the buffer why not test the shortcuts work [18:38] * balloons hands forestpiskie a cookie and tells him not to share with elfy [18:38] DanChapman, yes, exactly! [18:38] this has SO much more potential [18:39] balloons, for the test_end_key_goes_to_end_of_line the cursor is already at end after typing the phrase in the setup_test_docs so i need to jump back and then hit End just to check it works. ahhh i 'll think on it [18:39] DanChapman, true that it's already at the end, heh [18:40] DanChapman, did you look at shotwell again to see if we can do more with it, given the updated gtk stuff? I wonder if something more is exposed that would let us [18:41] it was really a cool testcase, just very hacky because of the inability to select the camera [18:47] balloons, is the old test with umockdev camera setup on it laying about anywhere? I might have a way to select the camera. [18:50] DanChapman, indeed, let me find it [18:50] not to sidetrack you :- [18:51] :-) [18:51] DanChapman, btw do you have the link for the desktop tests.. I don't have it on this box, and it's easier than searching for it on jenkins :-) [18:52] sorry I mean the jenkins run results [18:52] yep 2 secs [18:52] https://jenkins.qa.ubuntu.com/job/autopilot-ubuntu-applications/lastCompletedBuild/testReport/ [18:52] balloons: going to ask smiddy to add a couple of things - not going to be merging it today [18:53] elfy, kk [18:53] will do as soon as I see it come back though [18:54] ahh yes, knome's 'tests' removal [18:54] nothing to do with me - but seemed logical :) [18:55] DanChapman, hmm https://code.launchpad.net/~carla-sella/ubuntu-autopilot-tests/shotwell-carla [18:57] DanChapman, I also have some emails.. carla should be around soon, we can ask her directly [18:57] pastebin is gone :-( [19:00] balloons: you mean, as in the paste you had or paste.ubuntu.com is down? [19:00] jose, yea, it's expired I'd guess [19:00] oh, ok then :) [19:00] http://paste.ubuntu.com/1605319/ [19:02] elfy, synced up to rev206 to tracker [19:02] balloons: ok [19:05] balloons: why do last 2 have diffrerent icon against them here http://bazaar.launchpad.net/~ubuntu-testcase/ubuntu-manual-tests/trunk/files/head:/testcases/packages/? [19:06] elfy, ?? [19:06] I don't know [19:06] ahh.. they have code in them [19:06] they look different to me, got some sort of 'circly' thing [19:07] aah [19:07] shall remember that then [19:07] xpad doesn't tho.. fail [19:07] ty [19:07] oh [19:07] ignore me then ... [19:07] they do have broken " in them [19:08] check out line 17: http://bazaar.launchpad.net/~ubuntu-testcase/ubuntu-manual-tests/trunk/view/head:/testcases/packages/1602_Xpad [19:08] yep - see that [19:08] also a blank file got snuck in: http://bazaar.launchpad.net/~ubuntu-testcase/ubuntu-manual-tests/trunk/view/head:/testcases/packages/Xfce/blank [19:08] didn't sneak in balloons [19:09] it got synced as test 1603 :p [19:09] we weren't sure what to do - as it was a dupe [19:09] lol, well now it should be interesting [19:09] so let's try and clean it up.. [19:09] what needed to happen was it be deleted [19:10] elfy, what's "it"? [19:10] as in which testcase? [19:10] the 'blank' one in xfce [19:11] balloons: in http://bazaar.launchpad.net/~ubuntu-testcase/ubuntu-manual-tests/trunk/view/head:/testcases/packages/1601_Lxterminal [19:12] line 50 I got a [ made red - would that be an error? [19:13] wild, why did you name it blank and move it into xfce? anyways if we need to delete something, it's easy enough to do. Just remove it from the repo altogether. Then we can blank it and/or remove it from the tracker, but we should keep it as a manual process [19:14] elfy, no.. I'm afraid I made too much of peering at launchpad's attempts to parse the files [19:14] I was positing that maybe that's why the icon was different. [19:14] balloons: it was already in XFCE - I'd writtent the same test twice - the first got put into trcker and given a number - then we realised we had another of the same [19:15] rather than have it there confusing us, I removed the contents and renamed it [19:15] elfy, gotcha.. best to just remove it from the repository. you can delete it now. As far as the tracker, we can manually clean it up [19:15] it isn't in the tracker - it is ONLY there in the branch [19:15] OHH.. [19:15] :-) [19:16] then yea, just remove it [19:16] any new file without an id will be added as a new testcase.. [19:16] so it IS as simple as that then - remove it and then bzr push :parent [19:16] yep, just remove it. especially if you don't need to do anything in the tracker [19:16] ok [19:17] I'll do that when I merge either slicky or smiddy's tests tomorrow [19:19] :-) [19:19] Letozaf_, hello! [19:19] balloons, hello :) [19:21] balloons, made a banana and peanut butter smoothie today.... mmm delicious, just that i put too much peanut butter and now I feel a bit sick [19:21] Letozaf_, ROFL! Nice, it's a delicate balance! [19:21] :| [19:21] balloons, yeah very nice [19:21] that stuff should be launched at the sun - ALL of it [19:21] I'd rather eat liver [19:21] elfy, peanut butter? [19:21] eeeew [19:22] I'll take that as a yes [19:22] awful stuff :p [19:22] elfy, I looooove peanut butte [19:22] sorry butter [19:22] elfy, it's super delicious [19:23] can't see what you're saying I have my fingers in my eyes [19:23] elfy, lol [19:23] to be fair, you might not have had the good stuff.. I like peanut butter that is literally just peanuts [19:24] balloons, I used to make it my self putting peanuts in a blender [19:24] Letozaf_, mmmm [19:25] we should stop scaring elfy [19:25] :-) [19:25] LOLOLOL [19:25] too late [19:25] balloons, lol [19:25] balloons:elfy hatez you :p [19:25] balloons, I cannot figure how to remove a topic on the "new" version of rssreader, no swipe and no popup, could it be a bug [19:25] forestpiskie, is it because I gave you the cookie? [19:25] balloons, or are they working on it ? [19:26] :) [19:26] anyways, so DanChapman and I were looking at shotwell again, and wondered if you had your old stuff or not [19:26] I think with the changes we might be able to finish implementing it [19:26] Letozaf_, ohh, let me look at the new version of RSS reader [19:26] indeed a bug is possible [19:26] balloons, just wanted to be sure before reporting a bug [19:28] Letozaf_, weird.. I agree [19:29] balloons, do you think it's better to wait as they might just be working on it or should I report a bug ? [19:31] let's report it [19:33] balloons, ok fine [19:33] balloons, I will now [19:33] Letozaf_, there are like 6 active reviews lol [19:34] balloons, so I better wait [19:35] Letozaf_, no no file away.. I was just saying it's quite a jumble in there [19:35] I'll talk to the rss folks tomorrow and I'll mention the bug and reviews and changing code [19:38] balloons, ok thanks [19:45] calling it a day folks see you all tomorrow :-) [20:01] morning all [20:06] howdy thomi [20:07] o/ === salem_ is now known as _salem [22:02] elopio, ping [22:03] balloons: pong [22:03] elopio, so I've converting over the clock app to the emulator, and I'm confused on using the tabs. doesn't seem to be working for me [22:04] balloons: want to share the code? [22:04] elopio, sure.. [22:05] elopio, lp:~nskaggs/ubuntu-clock-app/convert-aptests-to-sdk [22:06] elopio, look at test_stopwatch and run it [22:06] autopilot run ubuntu_clock_app.tests.test_stopwatch.TestStopwatch.test_start_stop_reset_stopwatch [22:09] balloons: I get that it has more than one header. [22:09] that shouldn't be. [22:09] elopio, same.. [22:09] heh [22:09] let me check the tree. [22:10] are we back to this silliness.. I'm going to keep working on converting the timer over [22:13] balloons: it has 6 headers. So, we need to assign an objectName to the main header. [22:14] ok, but otherwise I've done things correctly? good :-) [22:14] care to do it and I'll merge it back so I can see what you did? [22:16] I think everything is converted, barring a fix for that issue [22:16] ty! [22:17] balloons: the fix should be on the toolkit. But then we hit a problem with the autopilot emulators. [22:18] elopio, you simply mean your changes today fixed everything? [22:18] balloons: no, I mean that I'm trying to make a new MP. [22:18] OHH, gotcha :-) [22:18] but, I need thomi because I might make a bigger mess. [22:18] thomi: you around? [22:19] yup [22:19] sup? [22:19] the sky [22:19] * balloons can't help himself [22:19] wow, we're on fire today ;) [22:19] thomi: :) with the emulators we are forced to use the same name as the QML object. [22:19] in this case, Header. [22:20] but there's a top header, and some other types of headers. [22:20] all of them with the same QML type Header. So if I make the emulator called Header to be the top bar, then I won't be able to use it for the other types of headers. [22:20] does it make sense? [22:21] it's a long way from my ideas to my fingers. [22:21] elopio: yes, that makes sense. [22:21] elopio: that's one nasty part about Qml, IMO [22:21] ideally, if things are called the same, they should be at least similar :) [22:22] well, the thing is that they are similar in the sense that they are a kind of title, a header. The problem is that they have really different children. [22:22] I agree that it would be cool if the top header was a separate qml object, more specific than header. But that would be too restrictive, I think. [22:33] balloons: this will fix your problem https://code.launchpad.net/~elopio/ubuntu-ui-toolkit/fix1209405-header_object_name/+merge/179060 [22:33] however, I think this will hit us again in the future. [22:34] for now to test, let me try it [22:34] I'm not sure timp is going to like it either. They rejected one of my branches that added a objectName, so, it's likely we will to talk again tomorrow. [22:37] balloons: about your branch, the main_view @property is ok. I don't like the header one. Instead, I would do main_view.get_header. [22:37] elopio, I shouldn't need header, but yes that was a question I had :-) [22:37] elopio, sure enough it worked [22:37] and actuallyl, it's not likely you will need a reference to the header. main_view.switch_tab sounds better. [22:38] sweet.. so this is converted [22:38] but pending your emulator change [22:39] balloons: and, I think that this is for a future branch. But I would make a main view emulator specific to this app. It would live in emulators.py, and be defined as MainView(toolkit_emulators.MainView) [22:39] with methods like: open_alarm_tab, open_timer_tab [22:40] that way you are even more resilient to changes on the toolkit. [22:40] elopio, let me push the working version in just a moment and you can "show" me [22:41] balloons: yes, an MP that I can see on launchpad would be lovely. [22:41] gotta lambda a missing function is all :-p [22:43] cool works.. pushing [22:46] elopio, https://code.launchpad.net/~nskaggs/ubuntu-clock-app/convert-aptests-to-sdk/+merge/179062 [22:47] so whatever magic you have that would let this go without waiting, I'd love to see it and merge it :-) more green tests tomorrow will be good [22:47] balloons: well, I should have read better because you already did what I was asking :) [22:51] balloons: I approved your branch. I would do some things different, but on a follow up branch. [22:52] mainly, move most of the methods you currently have on main view to an emulator for their correspoing page or tab. [22:52] and while doing this, we might hit the same problem as with the header. Many different objects with the same emulator name. [22:53] ahh I get what you mean [22:53] yes, heh, I went back and forth a bit while you were talking with thomi. Eventually we'll get there, but I wanted to keep exploring this to keep pushing the emulator forward.. and the testcases as well [22:54] and this app likes to randomly fail, so, I thought doing this would seal it up [22:54] the emulator has all the methods from the previous apps all lumped together, so yea :-) [22:57] elopio, left a comment on your merge also.. Well, that's it for me. thanks for the help [22:58] balloons: would you mind if I try to split it in smaller emulators after your branch lands? [22:59] elopio, not at all.. if you want to keep driving one app to test out the emulator and best practices go for it. otherwise, I would say let's get them all converted to it. I think we'll find more surprises doing that anyway [23:00] balloons: this one looks interesting. I want to see if we will hit the same problem, and if so, how to work it around. [23:01] but it's your call. If you prefer, I can work on another one to start using the emulator. Whatever works better for you. [23:02] well, making the builds green makes everyone happier, so i would say converting all first. But I'm certainly not opposed to you driving ahead on this; especially if you feel there will be issues. I'd like to drive out as many issues as early as possible [23:02] so in this case, if you feel there's a landmine in there, go for it [23:02] with that, I'm out.. really [23:03] balloons: ok, I'm busy today anyway, so tomorrow I'll see what looks fun. [23:03] have a good evening.