[00:02] hey stgraber [00:02] stgraber, did you notice my bugfixes to the trackers (in the main branch, waiting to be pushed upstream) [00:04] knome: I noticed your IRC ping at that time but haven't had the chance to look into it. [00:04] oki [00:04] the JS files itself are tested and will work; [00:04] what you should review is how they are called in the module; i think i have it finally right.. === _salem is now known as salem_ === salem_ is now known as _salem [05:58] Good morning [06:01] good morning pitti :) [06:01] hey elfy, how are you? [06:02] I'm good thanks - glad that xubuntu have finally decided what to do about ibus [06:02] * elfy was worried about releasing with that [06:03] in other news - back to work today after week off :( [06:04] elfy: hah, did you enjoy you holidays? [06:05] it was a break - poor as a church mouse I am :) [06:06] pitti: actually I work p/t so I have 2 weeks a month off - but my week starts on a thursday ... === maclin_ is now known as maclin === knome_ is now known as knome [07:01] Good morning [07:03] morning jibel [07:04] good morning knome [07:04] morning jibel knome [07:04] allö elfy [07:09] hey elfy [08:39] jibel: hah, this time it's you who are too fast on the "retry" trigger :) [08:39] (udisks, nova) [08:47] pitti, heh, actually we should have a bot which restarts the tests if it fails on one arch and not on the other === vrruiz_ is now known as rvr [09:39] jibel: we doing yet another round of testing today or is RC ready? [09:43] davmor2, if you have time, sure. We should test every image until the release. [09:44] jibel: no worries, I'm off tomorrow though :) [09:47] davmor2, cool, you won't be bothered by phone testing then. That leaves you plenty of time for desktop ;) [09:48] jibel: HA!! Not a pray I won't be anywhere near a computer all day, I'm going outside, I know I didn't think it existed either :) === vrruiz_ is now known as rvr === chihchun is now known as chihchun_afk === fader_ is now known as fader === _salem is now known as salem_ === Ursinha is now known as Ursinha-afk === Ursinha-afk is now known as Ursinha === qwebirc984668 is now known as slickymasterWork === maclin_ is now known as maclin === chihchun_afk is now known as chihchun === chihchun is now known as chihchun_afk === bfiller is now known as bfiller_afk === bfiller_afk is now known as bfille === bfille is now known as bfiller [19:07] elopio, hello [19:08] hey Letozaf_ how are you? [19:08] elopio, I'm fine and you ? [19:09] good [19:09] Letozaf_: I had just opened your branch 15 minutes ago [19:09] elopio, how is it ? [19:09] Letozaf_: it looks very good. [19:10] just some more details [19:10] elopio, I had some doubts about those two methods you wanted me to be private [19:10] Letozaf_: you asked that if you made the helpers private in emulators.py, you shouldn't call them from the test [19:11] that's precisely the idea :) [19:11] elopio, oh! so probalby I misunderstood :P [19:11] you should move open_feed_item and get_article_count to emulators.py, and those are the methods you call from the test. [19:12] elopio, oh fine I will [19:12] Letozaf_: another thing is that you removed the assertion at the end of the test [19:12] we need to check something, but I'm not sure what's the thing you want to check here. [19:14] elopio, I wanted to check that the count property of the article view was greater than 0 before opening a feed it is 0 [19:14] elopio, so if the feed is opened is should be greater than 0 [19:14] elopio, I put that in get_current_article_count [19:15] Letozaf_: ok, that could work. But as we are testing from the user point of view here, it sounds that it would be better to check that the article you opened is visible. [19:15] elopio, and also get_current_index should assert that the current_index property is greater than -1 [19:15] elopio, yes that is true [19:15] so, on pseudocode, something like: assertEqual(get_current_article().title, 'blah') [19:16] Letozaf_: and I have a question. [19:17] are you calling ensure_list_view_mode because open_feed_item only works in list view? [19:17] elopio, yes [19:17] Letozaf_: so, what would you think about this: [19:17] - move ensure_grid_view_mode to emulators.py, probably to MainView [19:18] - on the first statement inside open_feed_item, call ensure_grid_view_mode [19:19] elopio, ok fine I will make the changes you suggested [19:20] Letozaf_: I'm throwing a lot of things to you, I hope you are not hating me :) [19:20] elopio, no instead thanks for teaching me how to do things correctly [19:21] as I said, the test is good, so if you feel that it would be better to land it and then improve it little by little, that's fine by me. [19:21] elopio, I think you are only trying to make tests better [19:21] elopio, let's do it the way you think is better :P [19:23] elopio, so I will fix it as you suggested so you can see if it's ok first [19:23] Letozaf_: yes, thanks! [19:23] balloons, o/ [19:23] elopio, ok [19:29] * elopio <- lunch break [19:59] Letozaf_, howdy [19:59] balloons, fine and you ? [20:00] Letozaf_, sunburned, tired, good to be home :-) [20:00] hi balloons - welcome back [20:00] they brought release forward to tomorrow - hope you're realdy :p [20:00] balloons, lucky you :D [20:00] balloons, here it not hot to sun tan yet, maybe in a few weeks [20:04] elfy, :-) === salem_ is now known as _salem [21:27] balloons: how can you forward an email for RC when the iso tracker is not RC ? [21:28] makes me look like an idiot when I take on trust emails from you and post them to social media :( [21:42] phillw, ? final RC images will come after the freeze of course [21:44] balloons: is okay.... they froze tonight, but there is no deffinate answer as to when they will make an RC... the cron should be turned off, but then who will make the build? [21:45] phillw, I expect an RC by Monday morning.. sometime between now and Monday it will get started [21:45] the earliest would be tomorrow [21:45] and, of course... release will be 17th :P [21:46] to hell with the testers....