/srv/irclogs.ubuntu.com/2015/04/15/#ubuntu-quality.txt

=== chihchun_afk is now known as chihchun
elopiojfunk: when do you want to meet today to catch up with what we are doing here?14:18
jfunkcan do right after the refinement meeting14:19
jfunkin 4014:19
jfunkelopio: ^14:19
elopiojfunk: I will be in the training.14:19
jfunkelopio: ok, put something on my cal14:20
jfunkcan do it tmw too14:20
artmellohey, we are facing some issues with AP tests for gallery-app. We are proposing an MR to sort that out but we would like someone from qa to review it first14:39
artmellosince a couple of them seem to be unreliable after all14:39
artmelloping ubuntu-qa14:43
fgimenezartmello, you can add canonical-platform-qa as reviewer14:43
artmellofgimenez: ok, thx14:44
elopiofgimenez: can you take care of the review as one of the vanguard task? Or will it take more time than that?15:16
fgimenezelopio, yes, i'm setting up the branch for testing, i'll ping the vanguard if i don't finish with it before my eod15:19
elopiocool, thanks.15:19
=== chihchun is now known as chihchun_afk
balloonselopio, I'd like to talk about documenting the various autopilot helpers for system services on touch. Things like location, online services, accounts,  etc. I know there has been a good amount of work on making these less painful.16:46
elopioballoons: right, we need to talk. There are not many yet, but there will be.17:11
elopioballoons: can you schedule a meeting next week?17:11
balloonselopio, sure. I'll invite you and feel free to pass it along to whomever else17:14
elopiook17:15
elopioveebers: did you check if the test was skipped?22:23
elopioI think dobey mentioned about a testtools bug that would run the tests even if the class had the skip annotation.22:23
veeberselopio: I tested with a dummy test locally to make sure decorating the class worked (it did for me).22:24
veebersI'm pretty confident that it skips the test when I run it, I'll double check though22:24
elopioveebers: ok. I prefer your version.22:24
elopioI told brendand to raise the exception, and he didn't fully agree with that.22:24
brendandelopio, :)22:26
elopiobrendand: please review veebers' branch tomorrow, to see if you like it too.22:26
elopioveebers: 8+    """Return if the sim in the provided slot number is locked.22:27
elopioI think that should be: Return True if...22:27
veebersRight, I raise an exception because the method returns bool if the passed sim is locked or not, if the sim is not there that is an exception to the 'locked' or 'unlocked' states22:27
veeberselopio: good catch, will fix22:27
veeberselopio: to recap, you mentioned in the standup that you spoke w/ federico re: what was required for the 'click not visible dash item'? Specifically there is more to add to the link MP.22:31
elopioveebers: yes, I already updated the mp22:31
elopioand left a comment about updating it to use grid units.22:31
veeberselopio: ah cool thanks22:34
veeberselopio: does it need the method 'swipe_to_top' added?22:37
veebersor am I being dumb and it's already there22:37
elopioveebers: it's not already there, but it's not related to the non-visible item22:37
elopioso I would prefer it to come as part of the MPs that the unity team will do.22:37
elopioveebers: that swipe to top is for the uninstall test.22:38
veeberselopio: so that means that we can't remove GenericScopeView from the sanity suite as it still has functionality that we require that's not upstream?22:38
elopioveebers: that's right.22:38
elopioor if you prefer, I can make a quick MP for only swipe_to_top.22:38
veeberselopio, brendand: ^^ so in a nutshell we can't remove the odd inheritence because we're not putting everything upstream22:39
veeberselopio: I see that we also have 'click_scope_item' in GenericScopeView too :-)22:39
brendandelopio, veebers - hmmm did we forget a helper??22:40
veebersbrendand: I'm hoping to clarify. To get rid of the inheritance we have for the ClickScope, everything in dash.GenericScopeView needs to be upstream. Currently we don't have an MP that includes it all (or a collection of MPs that do that)22:44
brendandveebers, do we know what's missing? is it just one function?22:45
veebersThere are 3 methods we've touched. open_preview is sorted (thanks to elopio) click_scope_item and swipe_to_top remain. elopio has suggested to make MPs for one of those 2, perhaps another for the last22:47
veebersbrendand: ultimately the MP for 'remove the inheritence from sanity suite' is pretty straight forward, assuming the push upstream happens22:50
brendandveebers, something that's playing on my mind now, is that i've just noticed that the method 'go_to_store' is not in the GenericScopeView in ubuntu_sanity_tests, which means it already is upstream22:53
brendandveebers, so maybe these issues had nothing to do with inheritance at all? or am i missing something?22:53
veebersbrendand: the inheritance issue we see is due to unityclickscope.ClickScope inheriting from (upstream) ClickScope and (sanity suite) GenericScopeView. If we update the upstream GenericScopeView then there is no need for our ones and the multi-inheritance goes away23:02
brendandveebers, ah so it's only multiple inheritance that causes a problem. i missed that detail23:03
veebersah right, yeah :-)23:15
=== \b is now known as benonsoftware

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