=== Ursinha is now known as Ursinha-afk
=== Ursinha-afk is now known as Ursinha
bachi gmb12:44
gmbHi bac12:44
bacgmb: do we prefer Y.Event.simulate over the other ways to do mouse-y type things in yahoo.utils.actionblah?12:45
gmbbac: As far as I know, yes. That's what I've used in the past.12:46
bacok, i was about to head down the other path when i looked and saw no uses of it.  made me wonder if simulate was new-and-improved12:46
gmbbac: IDK; It's fairly straightforward to use though, IIRC.12:47
gmbbac: Also, sorry I didn't ping you again last night; I got sidetracked with home stuff so I didn't get a chance to carry on with the overlay investigation.12:47
bacnp.  any chance you can make some progress today?12:47
gmbbac: I'll take a look at it again after standup; I'm working on mute button cleanup atm and don't want to switch just yet.12:47
bacgmb, sorry to bother you again but i remembered the blocker i had at EOD12:48
bacin tearDown, how do i clean up stuff that i've inserted into the DOM?12:49
baci want to ensure i start with a clean slate but subsequent tests already have the accordion in them12:49
baci tried 'delete this.content_node' where c_n is as you'd expect, but it didn't do anything12:49
gmbbac: That's a tricky one, IIRC. You could do something like:12:50
bacah, good one12:51
baci'll give it a try12:51
gary_posterbac benji gmb mumble/kanban in 213:28
bacgmb, the messages from Y.log() that arrive in my browser console seem to be all jumbled.  should it be expected that they appear out of order?14:44
gmbbac: Only if they're part of functions that are being called asynchronously.14:44
gmbAre they consistently in the same wrong order?14:44
bacand it looks like tearDown is being called before the method i'm trying to test completes14:45
bacgmb: so i'm wondering if having the 'domready' in my JS is causing loads of problems14:45
gmbbac: Can you push your latest changes so I can take a look? I've had problems with asynchronicity in tests before now; usually you have to do some dancing around to fix it.14:46
bacgmb: sure14:46
bacit is riddled with Y.log statements14:46
gmbSounds like one of my tests.14:47
bacgmb: pushed to lp:~bac/launchpad/yellow-accordionoverlay14:48
gmbOkay. Grabbing it now.14:48
bacgmb: have you seen these?  http://www.bhphotovideo.com/c/product/752638-REG/Canon_Mugs_NK3212_Nikon_70_200mm_Vacuum_Insulated.html14:49
bacmakes me wish i had a need for one14:49
* bac eagerly awaiting the UPS man today...14:49
gmbbac: Can you give me an example of a test where the .log() messages are jumbled and what you expect them to look like?15:16
bacgmb: test_logged_in_user fails, then the tearDown is called, then you get "yoohoo" my message from within setup15:17
* gmb looks15:17
bacso tearDown is removing the DOM elements before setup gets a chance to work on them15:19
gmbbac: Right. Your problem is that you've got that on('domready' ...) bit, which is asynchronous, so your test completes before the domready event fires.15:24
gmbAh, race conditions. Love 'me.15:24
gmbbac: I'm not sure how to work around this one, though, unless you extract the stuff that's in on('domready') from setup(), make it into a function on its own and call that instead of setup().15:25
bacgmb: so even if domready is true, that bit will be pushed into a separate thread15:25
bacgmb: i was already thinking of testing the innards separately as you describe15:25
gmbbac: Yes. I think that if you register a handler after an event has taken place YUI has basically got to go and check that the event has actually fired. I'm honestly not sure how nesting handlers of the same event will behave.15:26
bacgmb: can i remove the domready and just make it a requirement on the caller?15:26
gmb(Well, badly, obviously, but htat's not the point)15:26
gmbbac: That would work, yes.15:26
bacok.  i *thought* i was doing the right thing...15:26
gary_postergmb, can you confirm off-hand that bug.owner is the reporter?  The interface says, less than helpfully, "The owner's IPerson"15:44
gary_posterThank you Very Much15:44
gmbgary_poster: Yes, owner is synonymous with reporter15:44
gary_posterawesome thank you15:44
benjiok, I managed to sort out a merge from ~yellow/launchpad/accordionoverlay (I need to merge more often) and am about to push it back up16:27
bacgmb: YUI unit test hygiene tip:  if there is anything but "Test complete" in the main browser area after the test finishes then you haven't cleaned up the dom properly.16:57
gmbbac: Duly noted.16:58
gmb(Might be worth adding that to a wiki page, if there is one for this stuff)16:58
bacgmb: that was mainly for my benefit...16:58
bacfinally, i am now ready to start the interaction testing.  my sloppy dom had prevented it until now.16:58
bacbut first, pastrami16:59
* gary_poster is lunching too16:59
gmbbac: AHA! Behold! http://pastebin.ubuntu.com/578450/17:49
gmbU jbwq17:49
gmbI knew there had to be an event *somewhere* that we could hook into. I was just looking a level too low.17:50
gmbbac: That handles clicking off the overlay and also hitting ESC.17:50
gary_posteryay :)17:51
gmbIt doesn't handle the form cancel button, interestingly, because FormOverlay doesn't fire the cancel event when you click 'cancel'; it just hides the overlay directly.17:51
gmbWhich is a bug.17:51
gary_posterdoes it handle the submit button?17:51
gmbgary_poster: Only insofar as it hides the overlay.17:52
gmbThere's a degree of inheriting-but-not-quite going on between Overlay and FormOverlay.17:52
gary_postersure--I guess I meant did it make the line above your change in the pastebin superfluous17:52
gmbgary_poster: Ah, no.17:58
gmbI hoped it would, but the event is 'cancel', not 'hide' (which is what I was originally looking for).17:58
gmbgary_poster, bac: I'm going afk for a while now; if you've any questions please email them to me.18:05
gary_posterack, gmb, thanks18:05
gary_posterhave a nice evening18:05
gary_posterbac, no rush, ready when you are19:31
bacgary_poster: ok, just a sec19:32
gary_posterbenji ready when you are21:00
gary_poster(but no rush)21:00
bac`gary_poster, anything in the TL meeting about thunderdome locale?21:44
=== bac` is now known as bac
gary_posternope bac :-/22:00
gary_posterat least not that I remember22:00
bacnot in minutes22:00
bacdamn.  people got vacations to plan!22:00
gary_poster:-) sorry22:01
gary_posterbac--your "add structural subscription" menu would also include "edit structural subscriptions" (however it is spelled) yeah?  Going to to try and add that tomorrow22:04
gary_posterthe rough version of the edit-per-target page22:04
gary_posterthat is22:04
bacgary_poster: sorry, i didn't see your question.  Yes, if a person has existing structural subscriptions they will get another link in the global-actions portlet for editing those subscriptions22:22
gary_postercool thanks22:22
gary_postera link to a page we still need to create, AFAIK, yeah?22:22
gary_posterbug 674422, on the backlog22:23
gary_poster_mup_ is sleeping22:23

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