[00:03:55] JeffHall: links interface busted in FF: https://wmf.ci.cloudbees.com/job/VisualEditor-en.wikipedia.beta.wmflabs.org-linux-firefox/lastCompletedBuild/testReport/ Chrome looks OK [00:04:24] JeffHall: guessing you can repro that on beta labs [00:05:28] chrismcmahon: I tried reproing that manually via beta labs just now, and it seems to work fine - will also try it running the automated test locally [00:07:57] JeffHall: I repro'd on FF. clicking Link icon without touching anything else, nothing happened. on Chrome, first word is highlighted and Link UI appears [00:08:25] chrismcmahon: Yeah, something is wrong testing links via Firefox - I'll look into that more tomorrow. Luckily we're not deploying any of these changes to production this week! [00:09:05] JeffHall: want me to file the BZ for that then? I think this is a new issue. [00:09:30] chrismcmahon: Sure, that would be great if you can log the BZ ticket [03:05:49] Yippee, build fixed! [03:05:50] Project browsertests-commons.wikimedia.beta.wmflabs.org-linux-chrome build #484: FIXED in 4 min 4 sec: https://wmf.ci.cloudbees.com/job/browsertests-commons.wikimedia.beta.wmflabs.org-linux-chrome/484/ [03:06:15] Project browsertests-commons.wikimedia.beta.wmflabs.org-linux-firefox build #447: FAILURE in 4 min 27 sec: https://wmf.ci.cloudbees.com/job/browsertests-commons.wikimedia.beta.wmflabs.org-linux-firefox/447/ [03:10:21] Project browsertests-commons.wikimedia.beta.wmflabs.org-windows-internet_explorer_10 build #314: FAILURE in 4 min 31 sec: https://wmf.ci.cloudbees.com/job/browsertests-commons.wikimedia.beta.wmflabs.org-windows-internet_explorer_10/314/ [03:10:44] Project browsertests-commons.wikimedia.beta.wmflabs.org-windows-internet_explorer_9 build #425: FAILURE in 4 min 28 sec: https://wmf.ci.cloudbees.com/job/browsertests-commons.wikimedia.beta.wmflabs.org-windows-internet_explorer_9/425/ [03:25:05] Project browsertests-en.wikipedia.beta.wmflabs.org-linux-chrome build #496: FAILURE in 14 min: https://wmf.ci.cloudbees.com/job/browsertests-en.wikipedia.beta.wmflabs.org-linux-chrome/496/ [03:45:11] Yippee, build fixed! [03:45:11] Project browsertests-en.wikipedia.beta.wmflabs.org-windows-internet_explorer_10 build #311: FIXED in 20 min: https://wmf.ci.cloudbees.com/job/browsertests-en.wikipedia.beta.wmflabs.org-windows-internet_explorer_10/311/ [04:08:31] Yippee, build fixed! [04:08:32] Project browsertests-en.wikipedia.beta.wmflabs.org-windows-internet_explorer_7 build #379: FIXED in 23 min: https://wmf.ci.cloudbees.com/job/browsertests-en.wikipedia.beta.wmflabs.org-windows-internet_explorer_7/379/ [04:25:49] Yippee, build fixed! [04:25:49] Project browsertests-test2.wikipedia.org-linux-chrome build #745: FIXED in 17 min: https://wmf.ci.cloudbees.com/job/browsertests-test2.wikipedia.org-linux-chrome/745/ [04:44:38] Yippee, build fixed! [04:44:38] Project browsertests-test2.wikipedia.org-linux-firefox build #719: FIXED in 18 min: https://wmf.ci.cloudbees.com/job/browsertests-test2.wikipedia.org-linux-firefox/719/ [04:49:56] Yippee, build fixed! [04:49:57] Project browsertests-test2.wikipedia.org-windows-internet_explorer_10 build #302: FIXED in 23 min: https://wmf.ci.cloudbees.com/job/browsertests-test2.wikipedia.org-windows-internet_explorer_10/302/ [06:06:15] Project Translate-sandbox.translatewiki.net-linux-firefox build #73: FAILURE in 4 sec: https://wmf.ci.cloudbees.com/job/Translate-sandbox.translatewiki.net-linux-firefox/73/ [06:06:17] Project TwnMainPage-sandbox.translatewiki.net-linux-firefox build #81: FAILURE in 1.4 sec: https://wmf.ci.cloudbees.com/job/TwnMainPage-sandbox.translatewiki.net-linux-firefox/81/ [06:06:34] Project VisualEditor-test2.wikipedia.org-linux-firefox build #178: FAILURE in 3.4 sec: https://wmf.ci.cloudbees.com/job/VisualEditor-test2.wikipedia.org-linux-firefox/178/ [06:36:32] https://wmf.ci.cloudbees.com/view/r-uls/job/UniversalLanguageSelector-en.wikipedia.beta.wmflabs.org-linux-firefox/lastBuild/console is this issue known? [08:39:23] hi zeljkof [08:39:57] hashar: mayankmadan_ [08:40:00] sorry :) [08:40:04] hi mayankmadan_ [08:40:37] I speak to hashar too much, my IRC client thinks I want to ping him when I type "h" :) [08:40:48] hello [08:41:13] hashar: good morning, sorry about the ping, my mistake [08:42:09] mayankmadan_: do you have a question? [08:42:43] zeljkof, is there anything wrong with this page? https://www.mediawiki.org/wiki/Browser_testing/setup_instructions [08:42:48] if you have time [08:43:10] you are a part of Google code-in? [08:43:19] yep [08:43:49] mayankmadan_: since you are our first ever code-in student, I have all the time in the world :) [08:43:57] but I have a meeting in 15 minutes [08:44:04] let me see the page [08:44:08] thanks [08:44:41] mayankmadan_: the page looks good to me [08:44:45] great [08:45:10] do I have to close the ticket at code-in site? [08:45:24] zeljkof, can you close the task then? http://www.google-melange.com/gci/task/view/google/gci2013/5781683647283200 [08:45:40] https://wmf.ci.cloudbees.com/view/r-uls/job/UniversalLanguageSelector-en.wikipedia.beta.wmflabs.org-linux-firefox/lastBuild/console is this issue known? [08:46:00] mayankmadan_: sure, in 5-10 minutes [08:46:05] Nikerabbit: let me see... [08:46:12] okey doke [08:46:23] Nikerabbit: uh oh [08:47:25] it's all our tests failing with the same error btw [08:47:30] prolly others too [08:48:35] zeljkof: would you get time to review my browser tests tag for ULS ? [08:49:01] I would like to trigger the browser tests from Gerrit by the end of this week :D [08:50:35] wait everybody! there is only one of me :) [08:50:46] Nikerabbit: I will take a look at the failures with Amir today [08:51:18] hashar: your commits are high on my todo list, I will try to take a look today, but it will be merged by the end of the week for sure [08:55:17] mayankmadan_: ok, closing the code-in ticket in a minute or two [08:55:38] if you are looking for more documentation tickets, I will be more than happy to create a few more [08:59:08] zeljkof: it is not that much about the commit though, but more about how the scenario are tagged :D [08:59:33] the idea is to identify any tests which are not going to pass on a freshly installed wiki. [09:06:53] zeljkof, can you close the task if everything is ok? i was kinda in a hurry. But if you are busy then no problem [09:07:13] mayankmadan: closing it right now [09:07:19] thanks [09:07:33] closed [09:07:44] if you are looking for more documentation tasks, let me know [09:07:56] I would be glad to create a few more, we do need help [09:10:53] zeljkof, i wouldnt mind some more :) [09:11:11] but are there any coding tasks from you? [09:12:50] of course! :) [09:13:14] I am in a meeting for the next two hours, I will create a few tasks, both documentation and coding after the meeting [09:13:56] great [09:55:41] Yippee, build fixed! [09:55:42] Project Translate-sandbox.translatewiki.net-linux-firefox build #74: FIXED in 4 min 6 sec: https://wmf.ci.cloudbees.com/job/Translate-sandbox.translatewiki.net-linux-firefox/74/ [09:57:03] Yippee, build fixed! [09:57:03] Project UniversalLanguageSelector-sandbox.translatewiki.net-linux-firefox build #71: FIXED in 1 min 21 sec: https://wmf.ci.cloudbees.com/job/UniversalLanguageSelector-sandbox.translatewiki.net-linux-firefox/71/ [10:05:42] Yippee, build fixed! [10:05:43] Project TwnMainPage-sandbox.translatewiki.net-linux-firefox build #82: FIXED in 12 min: https://wmf.ci.cloudbees.com/job/TwnMainPage-sandbox.translatewiki.net-linux-firefox/82/ [10:17:26] (CR) Amire80: [C: 2] Added links to Jenkins jobs [selenium] - https://gerrit.wikimedia.org/r/96023 (owner: Zfilipin) [10:17:30] (Merged) jenkins-bot: Added links to Jenkins jobs [selenium] - https://gerrit.wikimedia.org/r/96023 (owner: Zfilipin) [10:34:47] hashar: do you know why this page has "edit links" at the bottom-left http://en.wikipedia.beta.wmflabs.org/wiki/Main_Page [10:34:50] but this page does not [10:34:55] http://en.wikipedia.beta.wmflabs.org/wiki/Boleyn_family [10:35:06] css [10:35:17] what about it? [10:35:18] ahh [10:35:26] no I have no idea [10:35:29] :) [10:35:35] should I ask wikidata people? [10:35:54] yup definitely [10:36:29] that might be related to a user permission which we dont have onbeta [10:39:33] hashar: can you take a look at wikidata channel [10:39:43] I got the reply there, but I do not understand what they mean [10:53:50] zeljkof, this task doesnt make any sense [10:53:52] http://www.google-melange.com/gci/task/view/google/gci2013/5814239499386880 [10:53:58] the page doesnt exist now [10:54:10] yes, sorry [10:54:16] we have deleted the page and forgot to delete the task [11:29:36] zeljkof, what about the tasks that you were about to add? [11:29:56] mayankmadan: I have just finished with the meeting, still writing meeting notes [11:30:12] okay [11:30:15] as soon as I am done with that I will add a few more documentation and coding tasks [11:30:18] no problem :) [11:30:32] I did not find a way to delete that obsolete task [11:30:54] I will probably reuse the task, just change the title and text to something else [11:31:18] hmm [11:31:38] Nikerabbit: looks like the failure that you have noticed earlier today is a cloud bees problem, [11:36:37] it is fixed now [11:43:21] zeljkof: oki [11:43:31] zeljkof: will someone trigger reruns of tets or should we just wait? [11:47:12] (PS1) Hashar: tweak phpunit-junit macros [integration/jenkins-job-builder-config] - https://gerrit.wikimedia.org/r/97725 [11:47:15] (PS1) Hashar: phpunit job + wikimedia/wikimania-scholarships [integration/jenkins-job-builder-config] - https://gerrit.wikimedia.org/r/97726 [11:47:23] (CR) Hashar: [C: 2] tweak phpunit-junit macros [integration/jenkins-job-builder-config] - https://gerrit.wikimedia.org/r/97725 (owner: Hashar) [11:47:30] (CR) Hashar: [C: 2] phpunit job + wikimedia/wikimania-scholarships [integration/jenkins-job-builder-config] - https://gerrit.wikimedia.org/r/97726 (owner: Hashar) [11:48:07] (Merged) jenkins-bot: tweak phpunit-junit macros [integration/jenkins-job-builder-config] - https://gerrit.wikimedia.org/r/97725 (owner: Hashar) [11:48:22] (Merged) jenkins-bot: phpunit job + wikimedia/wikimania-scholarships [integration/jenkins-job-builder-config] - https://gerrit.wikimedia.org/r/97726 (owner: Hashar) [11:52:19] (PS1) Hashar: triggers for wikimedia/wikimania-scholarships [integration/zuul-config] - https://gerrit.wikimedia.org/r/97727 [11:53:05] (CR) Hashar: [C: 2] triggers for wikimedia/wikimania-scholarships [integration/zuul-config] - https://gerrit.wikimedia.org/r/97727 (owner: Hashar) [11:53:14] (Merged) jenkins-bot: triggers for wikimedia/wikimania-scholarships [integration/zuul-config] - https://gerrit.wikimedia.org/r/97727 (owner: Hashar) [11:59:54] (PS1) Hashar: phpunit-junit builder missed full path to phpunit [integration/jenkins-job-builder-config] - https://gerrit.wikimedia.org/r/97729 [12:00:41] (PS2) Hashar: phpunit-junit builder missed full path to phpunit [integration/jenkins-job-builder-config] - https://gerrit.wikimedia.org/r/97729 [12:00:50] (CR) Hashar: [C: 2] phpunit-junit builder missed full path to phpunit [integration/jenkins-job-builder-config] - https://gerrit.wikimedia.org/r/97729 (owner: Hashar) [12:01:15] (Merged) jenkins-bot: phpunit-junit builder missed full path to phpunit [integration/jenkins-job-builder-config] - https://gerrit.wikimedia.org/r/97729 (owner: Hashar) [12:12:37] Nikerabbit: I have already rerun the tests [12:52:57] zeljkof, are you free now? [12:53:11] mayankmadan: I am for the next 5 minutes :( [12:53:15] then I have another meeting [12:53:29] I did not have the time to add more tasks, if that is what you wanted to ask [12:53:36] zeljkof, so you are like a super busy person :) [12:53:45] I promise I will do my best to add more tasks after this meeting [12:53:55] mayankmadan: not usually, but today is crazy :) [13:35:08] (Abandoned) Hashar: pipelines.test: Change from Verified+2 to Verified+1 [integration/zuul-config] - https://gerrit.wikimedia.org/r/61720 (owner: Krinkle) [13:37:46] (Abandoned) Hashar: experimental breadcrumb display [integration/docroot] - https://gerrit.wikimedia.org/r/68343 (owner: Hashar) [13:38:04] (Abandoned) Hashar: experimental HTML console publishing [integration/jenkins-job-builder-config] - https://gerrit.wikimedia.org/r/74424 (owner: Hashar) [13:38:11] (Abandoned) Hashar: publish-console.py script [integration/jenkins] - https://gerrit.wikimedia.org/r/75637 (owner: Hashar) [13:40:37] (PS2) Hashar: Actually trigger mwext-MobileFrontend-phpcs-HEAD . [integration/zuul-config] - https://gerrit.wikimedia.org/r/94239 (owner: JanZerebecki) [13:40:53] (CR) Hashar: [C: 2] "sorry , I have been long to visit this change." [integration/zuul-config] - https://gerrit.wikimedia.org/r/94239 (owner: JanZerebecki) [13:40:59] (Merged) jenkins-bot: Actually trigger mwext-MobileFrontend-phpcs-HEAD . [integration/zuul-config] - https://gerrit.wikimedia.org/r/94239 (owner: JanZerebecki) [13:44:28] (CR) Hashar: "deployed" [integration/zuul-config] - https://gerrit.wikimedia.org/r/94239 (owner: JanZerebecki) [13:46:03] (PS4) Hashar: Only run linting and not tests [integration/zuul-config] - https://gerrit.wikimedia.org/r/97061 (owner: Jeroen De Dauw) [13:47:09] (CR) Hashar: [C: 2] "That is a pity, makes the WMF Jenkins useless so you might as well disable all jobs." [integration/zuul-config] - https://gerrit.wikimedia.org/r/97061 (owner: Jeroen De Dauw) [13:49:00] (Merged) jenkins-bot: Only run linting and not tests [integration/zuul-config] - https://gerrit.wikimedia.org/r/97061 (owner: Jeroen De Dauw) [14:00:15] mayankmadan: are you here? [14:01:48] (CR) Hashar: "deployed" [integration/zuul-config] - https://gerrit.wikimedia.org/r/97061 (owner: Jeroen De Dauw) [14:05:45] zeljkof: so what about my proposed Selenium tag "specific-settings" ? To flag out tests which are expecting something more than a fresh wiki ? :] [14:06:11] hashar: sounds good to me [14:06:28] did anybody else comment (Chris, Jeff...)? [14:09:04] hashar: alone or in addition to default-settings? [14:10:22] Nikerabbit: default-settings was my original plan :D [14:10:30] wanted to white list the changes that would pass on a fresh wiki [14:10:42] then I thought it would be better to blacklist list the one requiring a specific setup [14:18:11] zeljkof, yep [14:18:34] mayankmadan: do you have a minute to chat here? [14:18:43] yep [14:18:44] I would like to know what tasks are you interested in [14:19:15] to start, my name is Ċ½eljko, I live in Zagreb (Croatia) and I work as a software tester for Wikimedia Foundation [14:19:45] we have a lot of tasks related to testing/QA [14:19:56] two main groups are documentation and code [14:20:05] are you more interested in documentation or code? [14:20:11] both [14:20:14] (and feel free to introduce yourself) [14:20:20] great [14:20:25] do you know ruby? [14:20:32] or would you like to learn ruby? :) [14:20:33] my name is Mayank Madan, from india [14:20:41] i dont know ruby at all [14:20:48] but ive been meaning to learn it :) [14:20:52] ok, great [14:21:00] I have a few easy tasks just for you [14:21:09] beginner ruby? [14:21:12] yes [14:21:16] just to get you feet wet [14:21:23] how old are you? [14:21:35] I know you are in high school, just curious [14:21:43] feel free not to answer [14:22:18] back to code, I would like us to follow ruby style conventions [14:22:22] this is a good guide [14:22:22] https://github.com/styleguide/ruby [14:22:30] I have a couple of tasks in mind [14:22:43] they could take you a few days each to complete [14:22:46] zeljkof, im 16 [14:23:20] how much time did it take you to do the first task? (moving setup to a separate age) [14:23:32] 20 mins [14:23:32] \ [14:23:43] ok, that is pretty short, right? :) [14:23:55] better than nothing :D [14:24:06] I was told the tasks should be big enough to take at least a few hours to complete [14:24:25] hashar: the first task is a way to get them interested :) [14:24:44] mayankmadan: are you interested in tasks that will take you a few hours to complete? [14:24:53] yes [14:25:10] mayankmadan: ok [14:25:16] two tasks that I had in mind [14:25:47] take a look at "Strings" section of guide [14:25:51] https://github.com/styleguide/ruby [14:26:04] most of our code base is violating this [14:26:08] "Prefer string interpolation instead of string concatenation" [14:26:27] zeljkof, first i wanted to ask, how will be these 2 tasks weighed in final evaluation? [14:26:29] so the first task would be to find all double quotes in ruby files in about 10 repositories [14:26:47] mayankmadan: I have no idea how the tasks are wighted [14:27:03] this is the first time I participate in the google code-in [14:27:18] but this two ones will be more complicated than the first one [14:27:31] i really wanna be a grand prize winner so i need to make sure that i do a lot of tasks and some of them needs to be good coding tasks too :) [14:27:57] and I have more complicated tasks in mind, if you finish the two introductory ones [14:28:05] mayankmadan: I like the spirit! :) [14:28:16] there is another thing [14:28:24] we do have a lot of things that needs to be done, and there is a lot of code to be written [14:28:28] go ahead [14:28:49] i dont know anything about ruby. i am good at php js python but not ruby [14:29:06] no problem, you will pick up enough ruby [14:29:17] im gonna do it!!! [14:29:54] if you want more php/js/python tasks, you will have to look elsewhere [14:29:54] if you want to learn ruby, stay tuned :) [14:30:07] i want to learn ruby!!! [14:30:19] but i will need help from you guys too :) [14:30:32] mayankmadan: I promise to teach you everything I know, if you want to learn :) [14:30:49] thank you [14:31:04] ok, the second task I had in mind [14:31:17] if you look at "Naming" section of the guidelines [14:31:20] https://github.com/styleguide/ruby [14:31:31] I have noticed that we violate this one [14:31:33] Use snake_case for methods and variables. [14:31:37] in at least one repository [14:31:58] so the task would be to replace all camelCase methods with snake_case [14:32:11] okey doke [14:32:14] ok [14:32:24] I will enter the tasks in melange and you can get started [14:32:32] when they are finished, we can move forward [14:32:34] both the tasks? [14:32:44] I think you can take just one at a time, right? [14:32:50] yes [14:33:05] so as soon as the first one is finished I promise to close the task as soon as possible [14:33:07] can you add just 1 now so that i can get the next one too? [14:33:11] wait, phone [14:33:24] no problem, that is an option too [14:37:56] mayankmadan: sorry, family emergency, have to do something, will be back in 30 or so minutes and I will add the task to the code-in site [14:38:03] when it is done I will add another one [14:38:08] no problem [14:38:46] Nikerabbit: I am dropping the @default-settings tag from the ULS browser tests [14:38:58] feel free [14:39:08] Nikerabbit: blacklisting the one requiring a specific wiki sounds better, this way the test will always be run on fresh wiki [14:39:40] Nikerabbit: I tend to dislike that tag name but could not find out something better :/ [14:40:40] @needs-custom-setup ;D [14:40:50] yeah good idea [14:46:44] (PS12) Hashar: WIP WIP browsertests for extension WIP WIP [integration/jenkins-job-builder-config] - https://gerrit.wikimedia.org/r/86868 [14:46:56] (CR) Hashar: "rebased" [integration/jenkins-job-builder-config] - https://gerrit.wikimedia.org/r/86868 (owner: Hashar) [14:48:54] (PS13) Hashar: WIP WIP browsertests for extension WIP WIP [integration/jenkins-job-builder-config] - https://gerrit.wikimedia.org/r/86868 [14:49:19] (CR) Hashar: "+ MobileFrontend" [integration/jenkins-job-builder-config] - https://gerrit.wikimedia.org/r/86868 (owner: Hashar) [14:49:42] (PS14) Hashar: browsertests for some extensions [integration/jenkins-job-builder-config] - https://gerrit.wikimedia.org/r/86868 [14:50:01] (CR) Hashar: [C: 2] "lets unleash them. deployed!" [integration/jenkins-job-builder-config] - https://gerrit.wikimedia.org/r/86868 (owner: Hashar) [14:50:23] (Merged) jenkins-bot: browsertests for some extensions [integration/jenkins-job-builder-config] - https://gerrit.wikimedia.org/r/86868 (owner: Hashar) [14:51:24] Nikerabbit: and now I am adding mwext-browsertests-UniversalLanguageSelector-phantomjs to the list of job triggered in Zuul [14:53:10] ok [14:54:42] (PS1) Hashar: ULS browsertests on check pipeline [integration/zuul-config] - https://gerrit.wikimedia.org/r/97741 [14:55:34] (CR) Hashar: [C: 2] "deploying" [integration/zuul-config] - https://gerrit.wikimedia.org/r/97741 (owner: Hashar) [14:55:44] (Merged) jenkins-bot: ULS browsertests on check pipeline [integration/zuul-config] - https://gerrit.wikimedia.org/r/97741 (owner: Hashar) [15:37:08] zeljkof, you are back :) [15:37:25] mayankmadan: yes, starting to work on the task [15:37:28] was having a look at ruby syntax [15:37:39] how to you like it? [15:38:51] its nice [15:41:04] I will add a basic task to melange site, and link to the bugzilla [15:41:13] and add all details to bugzilla [15:41:29] I will post the link here when it is created [15:41:34] so you can claim the task [15:41:44] and get the data in bugzilla as I write it [15:55:34] JeffHall: would you like to be one of the google code-in mentors? [15:55:57] chrismcmahon and I are already mentors, but we could use some help [15:56:03] Sure - I'm interested, but not really sure what's involved [15:56:06] it would not take a lot of your time [15:56:22] OK, sounds good to me! [15:56:27] all you need to do is review task that students do [15:56:38] and optionally, create new tasks for studens [15:56:43] students, that is [15:57:16] OK - I did some review for mayankmadan yesterday, so I can definitely do that for other students as well [15:58:05] JeffHall: great, I have asked what you need to do [15:58:40] JeffHall: Quim will let you know what you need to do [16:00:22] zeljkof: Yep, Quim just talked to me, so I'm getting setup now [16:00:30] JeffHall: great [16:00:36] I am writing new tasks [16:03:13] please ping me when the task is created :) [16:07:05] here it is [16:07:14] https://www.google-melange.com/gci/task/view/google/gci2013/5896231096156160 [16:08:10] zeljkof, the page is inactive :( org admin has to approve it [16:08:21] mayankmadan: I will ping them [16:09:52] approved! :) [16:09:59] claimed it :) [16:10:03] I will add more data to the bug after lunch [16:10:06] back in 30 minutes [16:10:30] wait [16:10:44] zeljkof, which git repo do i clone? [16:10:46] I am adding data to the bug as I write this :) [16:10:52] take a look at https://bugzilla.wikimedia.org/show_bug.cgi?id=57597 [16:10:59] there is 10 or so repos in the list [16:12:06] * zeljkof is out to lunch [16:12:51] there is no list [16:13:09] zeljkof, ^ [16:13:29] or anyone who can help as zeljkof is gone [16:13:40] JeffHall :^^ [16:19:05] chrismcmahon, ^^ [16:20:19] mayankmadan: read the bug carefully please [16:20:35] chrismcmahon, i cannot find the list in the given url [16:21:47] mayankmadan: do you see where it says "list of repositories"? [16:21:58] chrismcmahon, in the link section there are 9 repos but the task said 10 [16:22:19] yes [16:23:13] mayankmadan: the 10th is mediawiki/selenium itself I believe [16:23:19] ok [16:23:40] so do i change all the single quotes including the ones that are in require statements? [16:25:45] mayankmadan: do you have an account it gerrit? you'll have to be able to submit patches http://www.mediawiki.org/wiki/Developer_access [16:25:54] yes i do [16:26:00] chrismcmahon, ^^^ [16:26:04] mayankmadan: OK [16:26:24] so do i change all the single quotes including the ones that are in require statements? [16:30:21] mayankmadan: what does the style guide say? [16:31:20] Yippee, build fixed! [16:31:21] Project MobileFrontend-en.m.wikipedia.beta.wmflabs.org-linux-firefox build #186: FIXED in 17 min: https://wmf.ci.cloudbees.com/job/MobileFrontend-en.m.wikipedia.beta.wmflabs.org-linux-firefox/186/ [16:33:59] mayankmadan: did you get the list of repos? [16:34:06] yep [16:34:15] great [16:34:20] do you have any questions? [16:34:31] all you have to do is clone all repos, 10 or so of them [16:34:51] and replace all double quotes with single quotes in .rb and .feature files [16:34:55] are you free for sometime? [16:35:06] yes, I have 25 minutes now until the next meeting [16:35:26] cause i wont be able to reply for about 10 mins(having dinner) [16:35:35] sure, I will be here [16:35:49] I will probably be able to answer short questions during the meeting too [16:45:34] zeljkof, do i have to change all the single quotes to double or just including require, if etc [16:45:53] and how do i test out the scripts after making changes? [16:45:56] mayankmadan: change all single to double [16:46:09] and I will change back the ones that need to be single [16:46:15] ok [16:46:21] do you have gerrit account? [16:46:26] yes [16:46:29] meaning: did you clone from gerrit? [16:46:31] ok, great [16:46:48] zeljkof, how do i test out the scripts after making changes? [16:46:55] do you know how to push changes back to gerrit? [16:47:05] by git review [16:47:14] great, do you have git-review installed? [16:47:24] which OS do you use? windows, mac, linux? [16:47:58] yes [16:48:01] i use linux [16:48:14] ubuntu 13.04 [16:48:31] ok, great [16:48:44] I will be back in a minute, this IRC client is behaving [16:48:47] will try another one [16:50:01] I am back [16:51:14] mayankmadan: do you know how to add me as a reviewer once you push the commit to gerrit? [16:51:21] yes [16:51:31] mayankmadan: wow, you know everything! :) [16:51:50] ok, ping me if you have any questions, I will be online for the next hour or two [16:52:25] zeljkof, how do i test scripts after making changes to them? [16:52:43] mayankmadan: it depends on the repository [16:52:56] the simplest check is to run ruby linter [16:53:08] do you have everything set up to run browser tests? [16:53:39] i dont even have ruby [16:53:41] zeljkof, ^ [16:53:45] :) [16:53:54] ok, you have to follow the advice from the page you have created :) [16:54:05] and if you find something wrong there, please do update the page [16:56:40] mayankmadan: anyway, just push the change to gerrit, I can test it on my machine [16:56:45] okay [16:56:56] and I can let you know how to test it on your machine when you make all the changes [16:57:13] the code in gerrit can be changed, so feel free to push there [16:58:09] zeljkof, are all the problems with concatenation fixed? [17:00:22] mayankmadan: what do you mean by that? [17:00:47] is that a general question, or something related to replacing quotes? [17:00:59] Prefer string interpolation instead of string concatenation: [17:01:10] mayankmadan: we have a few of those too [17:01:14] in first section of Strings in styleguide [17:01:20] in wikidata repo [17:02:16] chrismcmahon: I noticed that even the Chrome jobs for VisualEditor failed in their last CloudBees run: https://wmf.ci.cloudbees.com/view/r-ve/ [17:02:30] zeljkof, getting this error while installing ruby [17:02:33] chrismcmahon: Is it OK if I kick those jobs off manually to try and trouble-shoot them? [17:02:49] JeffHall: one sec... [17:02:55] JeffHall: yes, you can start all jenkins jobs you want :) [17:03:01] zeljkof, specifically while this step: rvm install ruby-2.0.0-p247 [17:03:07] mayankmadan: can you paste the error here? [17:03:25] sorry forgot to paste it :) [17:03:28] http://etherpad.wikimedia.org/p/mayankmadan [17:03:31] https://dpaste.de/K5Rj [17:03:32] paste here [17:03:39] zeljkof, ^^ [17:03:40] ok, that is good [17:04:07] mayankmadan: uh oh [17:04:16] did you read the log file? [17:04:24] from the error message? [17:04:34] JeffHall: I guess. https://wmf.ci.cloudbees.com/job/VisualEditor-en.wikipedia.beta.wmflabs.org-linux-chrome/163/console shows the msg. "/use-ruby: line 1: syntax error near unexpected token `<'" [17:06:09] zeljkof, https://dpaste.de/Nf8c [17:06:19] i think the repository in the last is not added [17:06:48] chrismcmahon: Yeah, that error seems environmental to me, so want to see if it will repeat on a re-run of the same jobs [17:07:00] mayankmadan: ok, can you install ruby via apt-get? [17:07:06] JeffHall: go for it [17:07:08] (are you on ubuntu?) [17:07:20] yep im on ubuntu [17:07:57] do you have rvm installed? [17:08:02] yes [17:08:04] maybe you can install different ruby version? [17:08:40] which version? [17:09:34] lemme see [17:09:54] mayankmadan: go to this page [17:09:55] http://etherpad.wikimedia.org/p/mayankmadan [17:10:26] zeljkof, what do i do with etherpad [17:10:38] I have pasted output of this command [17:10:41] rvm list known [17:10:48] that lists available rubies [17:10:52] if you have the same list [17:10:56] try installing this ruby [17:11:13] rmv install ruby-2.0.0-p353 [17:11:28] and please paste the output to the etherpad [17:12:41] you wrote rmv :) [17:12:57] you know what I mean :) [17:17:28] mayankmadan: any progress? [17:18:34] zeljkof, same error with that too [17:18:44] ok, try installing ruby with apt-get [17:18:58] let me see how to get a recent version of ruby [17:18:58] im trying something else. i removed the repo and tried it again and it worked [17:19:04] ok, great [17:19:10] you managed to install ruby now? [17:19:14] *i tried something else [17:19:23] almost [17:20:58] my internet connection is slow, still downloading [17:21:15] ok, good news is that it is not failing now [17:27:59] zeljkof, finally installed ruby [17:28:30] mayankmadan: yeah! :) [17:28:39] my bash-fu is not the best [17:28:56] this is a way to test if you broke anything in a ruby file: [17:29:39] ruby -c name_of_the_file.rb [17:29:52] there is probably a simple way to test all ruby files in the repo [17:30:04] something with "find", or something like that [17:30:16] example: https://integration.wikimedia.org/ci/job/mwext-Wikibase-ruby1.9.3lint/880/console [17:33:05] JeffHall, chrismcmahon: is your bash-fu better than mine? [17:33:23] how would we run "ruby -c name_of_the_file.rb" for the every file in a repo? [17:33:36] something with find and xargs, right? [17:34:24] mayankmadan: did you make the change in a repo? and did you push the change to gerrit? [17:34:55] zeljkof, just did [17:35:23] mayankmadan: did you add me as a reviewer (and chrismcmahon and JeffHall) [17:35:27] ? [17:35:28] (PS1) Mayankmadan: Prefer double-quoted strings in Ruby code [selenium] - https://gerrit.wikimedia.org/r/97756 [17:35:43] oh, I see it here in the channel :) [17:36:01] You can use a command like this to check syntax recursively: find . -name "*.rb" -exec ruby -wc {} \; [17:36:30] JeffHall: thanks, I will add it to the bug [17:36:54] zeljkof, you are zfilipin right? [17:37:03] mayankmadan: yes, that is me [17:37:47] zeljkof, i have to clone these repos right? https://github.com/wikimedia/mediawiki-selenium#links [17:38:07] mayankmadan: yes, you have to clone all repos [17:40:48] great :) [17:40:52] (CR) Zfilipin: "$ find . -name "*.rb" -exec ruby -wc {} \;" [selenium] - https://gerrit.wikimedia.org/r/97756 (owner: Mayankmadan) [17:46:23] zeljkof, btw why to replace ' with " ? [17:46:35] it is a style thing [17:46:41] did you read the guidelines? [17:47:06] it actually works both ways, but the convention is to use double quotes where ever possible [17:48:34] i know it works both ways but when i was learning php, i read that single quotes were better [17:48:54] mayankmadan: well, I guess it depends on the language [17:48:59] or the style [17:51:24] more the style. I think we preferred single quotes at the beginning because we wanted to make any string interpolation explicit within double quotes. that's how I remember it at least. [17:52:14] chrismcmahon: there are actually conflicting guidelines :) [17:53:03] this one actually says the opposite style should be used https://github.com/bbatsov/ruby-style-guide [17:53:22] we probably used that one at the beginning [17:53:37] but I think github guidelines should be followed [18:05:11] sorry i disconnected [18:05:35] no problem, you have reconnected again [18:06:06] added you to the reviewer list https://gerrit.wikimedia.org/r/97761 [18:06:09] zeljkof, ^ [18:07:42] mayankmadan_: so, when you push the commit to gerrit, a Jenkins job runs and check the syntax [18:07:59] for example: https://integration.wikimedia.org/ci/job/mwext-CirrusSearch-ruby1.9.3lint/149/console [18:08:01] and that is an easy way to know that you did not break anything [18:08:19] the jenkins job would make the build as failed if you broke something [18:08:34] and it would mark the commit as not good for merging into master branch [18:08:35] (CR) Jhall: [C: 1] "Looks good!" [selenium] - https://gerrit.wikimedia.org/r/97756 (owner: Mayankmadan) [18:09:48] JeffHall: maintenance time! :-) https://wmf.ci.cloudbees.com/job/VisualEditor-en.wikipedia.beta.wmflabs.org-linux-firefox/172/ [18:10:38] chrismcmahon: Yeah, I noticed the same thing running some of the VE tests locally - I'm on it [18:10:54] JeffHall: I'm looking also... [18:11:46] Yippee, build fixed! [18:11:46] Project browsertests-commons.wikimedia.beta.wmflabs.org-linux-firefox build #448: FIXED in 4 min 19 sec: https://wmf.ci.cloudbees.com/job/browsertests-commons.wikimedia.beta.wmflabs.org-linux-firefox/448/ [18:15:54] JeffHall: first thing I see is that we're no longer dismissing the beta warning [18:16:31] Yippee, build fixed! [18:16:32] Project browsertests-commons.wikimedia.beta.wmflabs.org-windows-internet_explorer_10 build #315: FIXED in 4 min 44 sec: https://wmf.ci.cloudbees.com/job/browsertests-commons.wikimedia.beta.wmflabs.org-windows-internet_explorer_10/315/ [18:18:00] zeljkof, 3 down 7 to go [18:18:14] https://gerrit.wikimedia.org/r/#/c/97762/ [18:18:34] mayankmadan_: good job! :) [18:18:46] do not forget to add JeffHall and chrismcmahon to reviewers in gerrit [18:18:51] ok [18:19:09] I will go offline soon, but the two of them just started the day, they will be online for a few more hours [18:19:17] and they can close the task [18:19:43] I will add a couple more tasks, so you have more to work on [18:19:54] and do not worry, there are more tasks than students :) [18:21:28] Yippee, build fixed! [18:21:28] Project browsertests-commons.wikimedia.beta.wmflabs.org-windows-internet_explorer_9 build #426: FIXED in 4 min 55 sec: https://wmf.ci.cloudbees.com/job/browsertests-commons.wikimedia.beta.wmflabs.org-windows-internet_explorer_9/426/ [18:22:28] zeljkof, how can they close the task? they are not in the mentors list [18:22:38] mayankmadan_: yes they are :) [18:23:04] they werent at the beginning :) [18:23:14] I have been editing the tasks [18:23:31] chrismcmahon: Yeah, looks like the Beta warning window has been changed a bit. I can update the relevant bits and sumbit a change [18:23:32] it is better for you (and other students) if there are more mentors for each task [18:23:42] so you do not have to wait for me to come online to close the task [18:24:51] Yippee, build fixed! [18:24:51] Project VisualEditor-test2.wikipedia.org-linux-chrome build #153: FIXED in 44 min: https://wmf.ci.cloudbees.com/job/VisualEditor-test2.wikipedia.org-linux-chrome/153/ [18:26:07] yep [18:29:48] JeffHall: beta warning seems to be in in_frame(:index => 0) now [18:30:37] chrismcmahon: Yep, do you want me to go ahead and submit a change for that, or are you working on it? [18:31:07] JeffHall: go ahead, I'm just surfing :-) [18:31:24] chrismcmahon: OK, will do [18:31:30] JeffHall: hoping that iframe index change doesn't mess with the Links test, checking [18:37:42] Yippee, build fixed! [18:37:42] Project browsertests-en.wikipedia.beta.wmflabs.org-linux-chrome build #497: FIXED in 16 min: https://wmf.ci.cloudbees.com/job/browsertests-en.wikipedia.beta.wmflabs.org-linux-chrome/497/ [18:40:37] zeljkof: I think this should be WONTFIX https://bugzilla.wikimedia.org/show_bug.cgi?id=48218 Selenium only emulates what users can do, it won't alter HTTP headers, you need something like a proxy to do that [18:42:18] JeffHall: Links test is failing because of https://bugzilla.wikimedia.org/show_bug.cgi?id=57568 [18:43:22] chrismcmahon: could you please add that to the bug? [18:43:38] maybe the solution is not to use selenium, but something else [18:43:47] chrismcmahon: Yep, that makes sense [18:43:48] but we could still help the zero team [18:43:55] zeljkof: I put a link to that discussion earlier [18:44:03] chrismcmahon: thanks, did not notice it [18:44:39] zeljkof: people *always* ask the Selenium team for that, and they *always* say NO. Jim Evans is pretty funny about it. [18:45:11] chrismcmahon: I remember a post or two that he made [18:46:10] JeffHall: Links tests seems OK in Chrome once the beta warning gets dismissed [18:52:45] https://gerrit.wikimedia.org/r/#/c/97765/ [18:58:42] mayankmadan_: more work for you, when you finish this task https://www.google-melange.com/gci/task/view/google/gci2013/5259991820795904 [18:58:54] and if it gets taken, there will be more tasks [18:58:59] okay [18:59:13] I will try to have 5 tasks open at all times [18:59:22] and at least one or two coding related [18:59:29] so you can pick [18:59:51] and if you do not like any of the tasks, let me know, I will create something that fits you [19:00:09] and feel free to apply for a pairing session with me: https://www.mediawiki.org/wiki/Pair_programming_Friday_for_fun_and_profit [19:00:10] Project browsertests-en.wikipedia.beta.wmflabs.org-windows-internet_explorer_10 build #312: FAILURE in 22 min: https://wmf.ci.cloudbees.com/job/browsertests-en.wikipedia.beta.wmflabs.org-windows-internet_explorer_10/312/ [19:02:30] JeffHall: How about now [19:04:54] mayankmadan_: I'll take a look at your change - I need to finish one other task first, so give me about 15 minutes [19:05:05] no prob [19:05:48] zeljkof, Does the Pair Programming Friday happens every friday? [19:06:09] mayankmadan_: yes [19:06:15] great [19:06:54] but as you can see from the list of previous sessions, it tends to happen not only on Friday :) [19:07:08] so we can pair any time both of us are available [19:16:31] https://gerrit.wikimedia.org/r/#/c/97769/ [19:22:50] JeffHall: shout if you want to pair up or reason about VE test maintenance [19:25:16] chrismcmahon: Yeah, there seem to be several page elements that have moved around, so I'm re-running individual tests locally to make sure my changes won't break anything [19:29:25] chrismcmahon: It would be good to pair up - the changes I want to make should work with master in Betalabs, but will apparently fail in test2, so I', not quite sure how to proceed [19:37:16] zeljkof, have a look at this http://etherpad.wikimedia.org/p/mayankmadan [19:37:19] how can this work? [19:37:43] did you test it with ruby -c? [19:37:55] it says syntax ok [19:37:56] I have to go in a few minutes [19:38:08] if ruby does not complain, it is fine with me :) [19:38:09] but single quotes after autonym never ends [19:38:28] just upload the file to gerrit and I will take a look tomorrow [19:38:41] or JeffHall/chris will take a look today [19:39:06] JeffHall: let them fail in test2, that's OK [19:39:50] i did it [19:39:53] chrismcmahon: ^ [19:39:57] i figured out the problem [19:40:21] mayankmadan_: zeljkof make sure to put me on the review list [19:40:31] i have [19:41:01] great [19:41:05] you are doing a good job [19:41:10] thank you [19:41:17] I am adding tasks to code-in site you have stuff to do :) [19:53:01] did you add any other task beside that? [19:53:15] beside this https://www.google-melange.com/gci/task/view/google/gci2013/5259991820795904 [20:24:39] JeffHall, There are 120 ruby files in Wikibase repository :) [20:26:40] mayankmadan_: Are all of those Ruby files in a "test" directory? I haven't looked at the repo before, so I'm not very familiar with it [20:27:15] JeffHall, do i only have to change files in test dir? [20:27:54] mayankmadan_: Let me take a look at that repo - I'm not sure if it has Ruby files that are not related to QA/testing [20:28:13] JeffHall, there are 118 files out of 120 in test folder [20:28:15] :) [20:33:23] JeffHall: can we remove the commented-out page element # text_area(:describe_change... [20:33:39] chrismcmahon: Per the question from mayankmadan_, it looks like the QA team did not write most of the tests in the Wikibase repo - should be applying the style change to Ruby files in that repo? [20:34:29] chrismcmahon: Yes to your question about the commented-out line - that was just a placeholder for myself - I'll get rid of it and amend [20:34:31] JeffHall mayankmadan_ the Wikibase stuff is all from WMDE (Germany), but if it's browser testing, it should be in that category [20:34:37] thanks JeffHall [20:35:33] JeffHall, chrismcmahon i dont understand, do i still have to do those 118 files? [20:36:12] mayankmadan_: if you can, I think that is what Zeljko intended [20:36:37] okey doke [20:37:13] all of them arent in test folder [20:37:17] JeffHall, ^ [20:37:24] do i do those too? [20:39:23] mayankmadan_: neither JeffHall nor I know the Wikibase repo very well, let me take a look... [20:39:25] JeffHall, chrismcmahon and which test folder are we talking about here? there are 3 tests folder [20:41:59] mayankmadan_: I think just do the ones in the /Wikibase/selenium_cuc directory [20:42:31] okey doke [20:42:33] thank you [20:42:35] mayankmadan_: I am pretty sure that the other directories are either obsolete or managed under a different set of rules [20:53:20] JeffHall: running the Links test in Chrome for your branch, I think there is more iframe tweaking for Links [20:54:13] chrismcmahon: OK, will look at that - skipped links test because of the known bug, but forgot that bug is Firefox-specific [20:55:12] JeffHall: you may remember when we walked through the code your first week, the Links diff view is (sigh) in a different iframe than the normal diff view [20:55:48] JeffHall: which happens to be the same iframe index that the beta warning was in [20:55:58] JeffHall: which is why we get paid the Big Bucks [20:56:47] chrismcmahon: OK, well I guess I'm about to get pretty familiar with links in Visual Editor! [21:03:54] JeffHall: guessing that :links_review_changes elements also moves from iframe-index-1 to iframe-index-0, etc. [21:06:12] chrismcmahon: Makes sense - I'm going to take a lunch break, and then I'll review the VE links test top-to-bottom via Chrome [21:24:45] hi zeljkof [21:24:48] task almost done [21:25:01] mayankmadan_: great! [21:25:14] what time is it for you, 3am? :) [21:25:44] ye [21:25:47] yep [21:26:42] wow, I have forgot the times when I was able to stay up coding that long :) [21:26:46] I guess I am too old [21:34:58] chrismcmahon: If you have some time, I would like to pair with you on the VE links test - I cannot seem to get around the "Element is not clickable at point (xxx, xxx) errors when running locally with Chrome [21:35:56] JeffHall_lunch: OK [21:36:21] But I need to go to lunch for real this time - back shortly... [21:59:26] zeljkof, all done [21:59:57] hey greg-g you should meet ryasmeen. we're discussing the issue where this bug got deployed and at least one user complained about it: https://bugzilla.wikimedia.org/show_bug.cgi?id=57269 [22:01:57] JeffHall_lunch, chrismcmahon task done [22:04:09] mayankmadan_: great [22:04:13] mayankmadan_: OK, I'm still reviewing the Gerrit changes (there were a lot of them!) so it will take me some time to get through all of those reviews [22:04:17] are you in a hurry? [22:04:32] JeffHall: please close the task if code looks good to you [22:04:47] I will go to bed in a few minutes [22:04:53] zeljkof: OK, will do [22:05:00] ill go to bed too [22:05:07] in about 10 mins [22:05:49] zeljkof, the task was assigned to me 5 mins ago by quim [22:05:57] :) [22:08:01] mayankmadan_: you are already working on a new task?! :) [22:08:16] zeljkof, no.. the earlier task [22:08:38] I thought it was assigned to you hours ago [22:08:54] i claimed it hours ago but no one assigned it to me [22:09:06] we were busy working on the task [22:09:07] I see [22:09:09] :) [22:09:11] I do not know how it works [22:10:05] Quim just showed me that we (the mentors) need to assign the task after someone claims it - I'll try to keep an eye on that going forward [22:10:35] greg-g, around? [22:11:01] JeffHall, can you close the task now? i wanted to claim the other one before i go to sleep so i can start working on it tomorrow [22:11:25] if there is a problem with my review requests i promise that i will fix those before starting the next one [22:11:43] mayankmadan_>: Need a little more time to review the changes in each of the repositories - shouldn't take me too much longer [22:11:58] okay [22:12:06] JeffHall: I am on hangout with ryasmeen atm [22:12:30] chrismcmahon: OK [22:13:36] JeffHall: invited you to the hangout [22:13:59] chrismcmahon: OK, let me see if I can join... [22:46:05] (PS2) Mayankmadan: Prefer double-quoted strings in Ruby code [qa/browsertests] - https://gerrit.wikimedia.org/r/97849 [22:57:11] trying to use VisualEditor on beta labs just now: Error loading data from server: parsoidserver-http-bad-status: 404. Would you like to retry? [22:58:08] not sure what config got overwritten though [23:02:08] I don't see the VisualEditor tab in beta labs at all - seems to have disappeared :( [23:03:58] JeffHall: are you logged in as a user with VE enabled? [23:04:57] chrismcmahon: Yes, I am logged in, but I'll double-check my editing preferences [23:05:08] I know that the parsoid server host is 'deployment-parsoid2'; but I don't know where that is set [23:06:35] chrismcmahon: I think it's deployment-parsoid3, actually. [23:07:23] chrismcmahon: Just made a test edit without incident: http://en.wikipedia.beta.wmflabs.org/w/index.php?title=User:Jdforrester_(WMF)/Sandbox&diff=68038&oldid=59090 [23:07:28] chrismcmahon: yeah, my VE preferences had become unset somehow - all better now [23:08:32] James_F: I have an email from 19 Nov that says deployment-parsoid2, but whatever [23:09:53] chrismcmahon: Yeah, but Roan says otherwise. :-) [23:09:56] beta seems to be back [23:10:17] Anyway, Roan and I need to pack if we're going to make our flight and sleep as well. :-) [23:10:19] * James_F waves. [23:10:44] James_F: ryasmeen and JeffHall and I miss you ;-) [23:10:55] * James_F|Away grins. [23:11:26] yes true :) [23:23:38] (CR) Jhall: [C: 1] "Looks good!" [qa/browsertests] - https://gerrit.wikimedia.org/r/97849 (owner: Mayankmadan) [23:32:02] heh. JeffHall, the :links_review_changes element is now in iframe-index-2. the plot thickens... [23:40:38] chrismcmahon: Great, if beta labs is back to life, I'll re-visit the links test [23:41:08] JeffHall: seems to be [23:44:30] chrismcmahon: I can't seem to get around the "Element is not clickable at point (xx, xxx)" when testing VE with Chrome locally - I'll try via Linux on vagrant to see if that makes any difference [23:45:25] JeffHall: a) is your local test dismissing the beta warning and b) are you running the Links test with the focus on the Chrome browser? [23:46:11] JeffHall: I *think* you should be able to see whatever is appearing "in front" of the not-clickable thingie if you watch the run [23:47:30] chrismcmahon: Yeah, the beta warning dismissal is working fine the changes I made earlier [23:47:44] JeffHall: hmm, OK [23:48:37] chrismcmahon: But you're right, somehow the focus is getting messed up before the link icon becomes acccessible to the test - will dig into that more to try and figure out what is going on [23:49:22] JeffHall: that error should also tell you what element would be getting the click [23:50:17] chrismcmahon: Yeah, it's the oo-ui-dialog-closing that's the source of the problem, so I'm looking into that now [23:57:12] chrismcmahon: I can solve the problem with a brute-force "sleep" statement, so I'll try implementing a "wait_until" and see if that will provide a better solution [23:58:03] hrm. a coyote just sauntered by my office window. that was a bit distracting. [23:59:09] We don't get coyotes much here in downtown San Fran