/irc-logs / w3c / #testing / 2015-07-03 / end

Options:

Previous day, Next day

  1. # Session Start: Fri Jul 03 00:00:00 2015
  2. # Session Ident: #testing
  3. # [00:00] * Joins: bhill2 (~bhill2@public.cloak)
  4. # [00:04] * Quits: bhill2 (~bhill2@public.cloak) (Client closed connection)
  5. # [00:07] * Joins: bhill2 (~bhill2@public.cloak)
  6. # [00:08] * terri_offline is now known as terri
  7. # [00:28] * Quits: bhill2 (~bhill2@public.cloak) (Client closed connection)
  8. # [00:28] * Joins: bhill2 (~bhill2@public.cloak)
  9. # [00:35] * Quits: bhill2 (~bhill2@public.cloak) (Ping timeout: 180 seconds)
  10. # [00:55] * heycam|away is now known as heycam
  11. # [01:15] * Joins: bhill2 (~bhill2@public.cloak)
  12. # [01:36] * Joins: playjax (~chatzilla@public.cloak)
  13. # [01:42] * Quits: bhill2 (~bhill2@public.cloak) (Client closed connection)
  14. # [03:22] * terri is now known as terri_offline
  15. # [04:02] * Joins: bhill2 (~bhill2@public.cloak)
  16. # [04:08] * Quits: bhill2 (~bhill2@public.cloak) (Client closed connection)
  17. # [04:35] * Quits: ArtB (~ArtB@public.cloak) ("Leaving.")
  18. # [05:09] * heycam is now known as heycam|away
  19. # [05:12] * Joins: gitbot (~gitbot@public.cloak)
  20. # [05:12] -gitbot:#testing- [web-platform-tests] zqzhang closed pull request #1963: Test templates docs: Use .html in filenames since the code is all non-XML (master...useless-indirection) https://github.com/w3c/web-platform-tests/pull/1963
  21. # [05:12] * Parts: gitbot (~gitbot@public.cloak)
  22. # [05:12] * Joins: gitbot (~gitbot@public.cloak)
  23. # [05:12] -gitbot:#testing- [web-platform-tests] zqzhang pushed 2 new commits to master: https://github.com/w3c/web-platform-tests/compare/31ad1b88b513...a14debae9be8
  24. # [05:12] -gitbot:#testing- web-platform-tests/master edb07fe Chris Rebert: Test templates docs: Use .html in filenames since the code is all non-XML
  25. # [05:12] -gitbot:#testing- web-platform-tests/master a14deba Zhiqiang Zhang: Merge pull request #1963 from cvrebert/useless-indirection...
  26. # [05:12] * Parts: gitbot (~gitbot@public.cloak)
  27. # [05:13] * Joins: gitbot (~gitbot@public.cloak)
  28. # [05:13] -gitbot:#testing- [web-platform-tests] zqzhang closed pull request #1964: Clarify CSS test naming docs (master...patch-1) https://github.com/w3c/web-platform-tests/pull/1964
  29. # [05:13] * Parts: gitbot (~gitbot@public.cloak)
  30. # [05:48] * heycam|away is now known as heycam
  31. # [07:03] * Quits: playjax (~chatzilla@public.cloak) (Client closed connection)
  32. # [08:24] * Joins: playjax (~chatzilla@public.cloak)
  33. # [08:53] * Joins: Ms2ger (~Ms2ger@public.cloak)
  34. # [09:07] * heycam is now known as heycam|away
  35. # [09:22] * Quits: playjax (~chatzilla@public.cloak) (Ping timeout: 180 seconds)
  36. # [09:38] * Quits: Lachy (~Lachy@public.cloak) ("My MacBook Pro has gone to sleep. ZZZzzz…")
  37. # [10:18] * Joins: Lachy (~Lachy@public.cloak)
  38. # [10:49] * Joins: gitbot (~gitbot@public.cloak)
  39. # [10:49] -gitbot:#testing- [wptserve] jgraham created jgraham/shared_stash at 4a7a937 (+0 new commits): https://github.com/w3c/wptserve/commits/jgraham/shared_stash
  40. # [10:49] * Parts: gitbot (~gitbot@public.cloak)
  41. # [10:50] * Joins: gitbot (~gitbot@public.cloak)
  42. # [10:50] -gitbot:#testing- [wptserve] jgraham pushed 1 new commit to jgraham/shared_stash: https://github.com/w3c/wptserve/commit/4cc56ad7b990abc1d54fc566dcf4c8ea3aaec381
  43. # [10:50] -gitbot:#testing- wptserve/jgraham/shared_stash 4cc56ad James Graham: Shared stash
  44. # [10:50] * Parts: gitbot (~gitbot@public.cloak)
  45. # [11:03] <jgraham> Ms2ger, ato: So, about those reviews…
  46. # [11:13] <gsnedders> Ms2ger: r/5435 is done
  47. # [11:14] * Joins: gitbot (~gitbot@public.cloak)
  48. # [11:14] -gitbot:#testing- [web-platform-tests] Ms2ger closed pull request #1958: Stop using innerText. (master...innerText) https://github.com/w3c/web-platform-tests/pull/1958
  49. # [11:14] * Parts: gitbot (~gitbot@public.cloak)
  50. # [11:14] * Joins: gitbot (~gitbot@public.cloak)
  51. # [11:14] -gitbot:#testing- [web-platform-tests] Ms2ger pushed 2 new commits to master: https://github.com/w3c/web-platform-tests/compare/cb727b8c1af7...d0e6c8493110
  52. # [11:14] -gitbot:#testing- web-platform-tests/master b1bd3a3 Ms2ger: Stop using innerText....
  53. # [11:14] -gitbot:#testing- web-platform-tests/master d0e6c84 Ms2ger: Merge pull request #1958 from Ms2ger/innerText...
  54. # [11:14] * Parts: gitbot (~gitbot@public.cloak)
  55. # [11:14] <Ms2ger> Takk
  56. # [11:14] <Ms2ger> jgraham, will try this afternoon
  57. # [11:17] <ato> jgraham: I’m in Belfast, and pending internet access a little later I can have a look unless someone else does first.
  58. # [11:19] * Joins: darobin (rberjon@public.cloak)
  59. # [11:21] <jgraham> Belfast‽
  60. # [11:34] <ato> Yes, gave a talk on the specification here yesterday.
  61. # [12:04] <gsnedders> Try not to get killed!
  62. # [12:06] <wilhelm> Don't tell them about your, uhh, connections.
  63. # [12:08] <gsnedders> Oh god, on the train down south had guy going from Carsile to Blackpool pissed as hell talking very loudly about politics and swearing every two seconds. Ended up with the BTP coming on the give them a talking to at Lancaster.
  64. # [12:09] <gsnedders> Carlisle
  65. # [12:09] <gsnedders> even
  66. # [12:09] <gsnedders> About Scottish nationalism and unionism and such like
  67. # [12:21] * Joins: gitbot (~gitbot@public.cloak)
  68. # [12:21] -gitbot:#testing- [wptrunner] Ms2ger closed pull request #129: Remove unnecessary postMessage from testharness report. (master...jgraham/remove_postmessage) https://github.com/w3c/wptrunner/pull/129
  69. # [12:21] * Parts: gitbot (~gitbot@public.cloak)
  70. # [12:27] * Joins: gitbot (~gitbot@public.cloak)
  71. # [12:27] -gitbot:#testing- [wptrunner] Ms2ger closed pull request #128: Ensure that manifests are always run in the same order. (master...jgraham/manifest_order) https://github.com/w3c/wptrunner/pull/128
  72. # [12:27] * Parts: gitbot (~gitbot@public.cloak)
  73. # [12:27] * Joins: gitbot (~gitbot@public.cloak)
  74. # [12:27] -gitbot:#testing- [wptrunner] Ms2ger deleted jgraham/manifest_order at 1a41e3c: https://github.com/w3c/wptrunner/commit/1a41e3c
  75. # [12:27] * Parts: gitbot (~gitbot@public.cloak)
  76. # [12:31] * Quits: Lachy (~Lachy@public.cloak) (Client closed connection)
  77. # [12:49] * krijnhoetmerbot is now known as krijnhoetmer
  78. # [12:50] <gsnedders> jgraham: do I need to do anything special for huge rebases which had to be manually resolved for Critic?
  79. # [12:50] <jgraham> Define "do anything special"?
  80. # [12:52] <gsnedders> "do I need to touch Critic before I push?"
  81. # [12:52] <jgraham> No
  82. # [12:52] <jgraham> But you do after you push
  83. # [12:55] <gsnedders> OK, I've pushed, now what do?
  84. # [12:57] <jgraham> Press the "rebase review button"
  85. # [12:57] <gsnedders> What do I want as the upstream?
  86. # [12:57] <gsnedders> the merge-base?
  87. # [12:58] <jgraham> Yes
  88. # [12:58] <jgraham> It should pick a reasonable default?
  89. # [12:58] <gsnedders> It didn't.
  90. # [12:59] <jgraham> Which review?
  91. # [12:59] <gsnedders> I can't even find the commit it used!
  92. # [12:59] <gsnedders> r/111
  93. # [12:59] <jgraham> Oh, right
  94. # [13:00] <gsnedders> truly ancient :)
  95. # [13:00] <jgraham> Yes, the merge base then
  96. # [13:02] <jgraham> Did you add extra stuff during the rebase?
  97. # [13:02] <jgraham> gsnedders: ^
  98. # [13:02] <Ms2ger> Oh wow :)
  99. # [13:02] <gsnedders> jgraham: no
  100. # [13:03] <gsnedders> jgraham: I removed stuff because stuff like MANIFEST is long dead
  101. # [13:03] <jgraham> Oh wow
  102. # [13:04] <jgraham> The following untracked working tree files would be overwritten by checkout: tools/scripts/id2path.js tools/scripts/id2path.json tools/scripts/manifest.js tools/scripts/package.json tools/scripts/toc.js tools/scripts/update-directory-structure.js
  103. # [13:04] <gsnedders> hah, wow
  104. # [13:04] <Ms2ger> I wonder if we could share code between that and the classList tests
  105. # [13:05] * gsnedders wonders if sof based them on the classList test
  106. # [13:09] * Joins: gitbot (~gitbot@public.cloak)
  107. # [13:09] -gitbot:#testing- [wptrunner] Ms2ger closed pull request #131: Disable debug-level logging for marionette component when running in Firefox. (master...jgraham/marionette_less_logging) https://github.com/w3c/wptrunner/pull/131
  108. # [13:09] * Parts: gitbot (~gitbot@public.cloak)
  109. # [13:09] * Joins: gitbot (~gitbot@public.cloak)
  110. # [13:09] -gitbot:#testing- [wptrunner] Ms2ger deleted jgraham/marionette_less_logging at fdbe2c8: https://github.com/w3c/wptrunner/commit/fdbe2c8
  111. # [13:09] * Parts: gitbot (~gitbot@public.cloak)
  112. # [13:12] <gsnedders> jgraham: lemme know when you've fixed it
  113. # [13:13] <jgraham> I don't think I can? Well not too easily at least
  114. # [13:14] <gsnedders> so what should I do?
  115. # [13:14] <jgraham> Create a new PR?
  116. # [13:15] <gsnedders> there are all of zcorpan's review comments on that review, though
  117. # [13:15] * Joins: ArtB (~ArtB@public.cloak)
  118. # [13:16] <jgraham> zcorpans's?
  119. # [13:17] <gsnedders> uh, Ms2ger's
  120. # [13:18] <gsnedders> I'm looking at multiple reviews at once :)
  121. # [13:18] * Ms2ger is now known as zcorpan
  122. # [13:18] <jgraham> Yes, that's true
  123. # [13:18] <zcorpan> Ahem
  124. # [13:18] * zcorpan is now known as Ms2ger
  125. # [13:19] <jgraham> You could fix the comments and then rebase
  126. # [13:19] <gsnedders> Ms2ger: go away, troll
  127. # [13:28] <gsnedders> jgraham: trying to do it with merge doesn't help either, bah
  128. # [13:34] <gsnedders> I don't get why you're getting untracked files in the working tree. Shouldn't you be cleaning the tree first?
  129. # [13:39] <Ms2ger> Submodules!
  130. # [13:39] <gsnedders> Yeah, I know /that/
  131. # [13:39] <gsnedders> but shouldn't we clean it first?
  132. # [13:58] * Joins: dom (dom@public.cloak)
  133. # [14:19] * Joins: plh (plehegar@public.cloak)
  134. # [15:03] * Joins: gitbot (~gitbot@public.cloak)
  135. # [15:03] -gitbot:#testing- [wptrunner] jgraham closed pull request #130: Switch to more compact results representation format. (master...jgraham/compact_wire_protocol) https://github.com/w3c/wptrunner/pull/130
  136. # [15:03] * Parts: gitbot (~gitbot@public.cloak)
  137. # [15:09] * Quits: birtles (~sid16523@public.cloak) ("")
  138. # [15:10] * Joins: birtles (~sid16523@public.cloak)
  139. # [15:27] <gsnedders> jgraham, ato: fwiw, I don't have much in the way of the plans for all of the weekend. I guess I'll do stuff outside tomorrow while it's dry, maybe going further out than I often do. *shrugs*
  140. # [15:29] * Quits: ArtB (~ArtB@public.cloak) ("Leaving.")
  141. # [15:40] * Joins: gitbot (~gitbot@public.cloak)
  142. # [15:40] -gitbot:#testing- [web-platform-tests] sideshowbarker created sideshowbarker/notifications (+1 new commit): https://github.com/w3c/web-platform-tests/commit/35623592614b
  143. # [15:40] -gitbot:#testing- web-platform-tests/sideshowbarker/notifications 3562359 Michael[tm] Smith: Cleaned up the Notifications tests.
  144. # [15:40] * Parts: gitbot (~gitbot@public.cloak)
  145. # [15:40] * Joins: gitbot (~gitbot@public.cloak)
  146. # [15:40] -gitbot:#testing- [web-platform-tests] sideshowbarker opened pull request #1967: Cleaned up the Notifications tests. (master...sideshowbarker/notifications) https://github.com/w3c/web-platform-tests/pull/1967
  147. # [15:40] * Parts: gitbot (~gitbot@public.cloak)
  148. # [15:41] <ato> gsnedders: I’ve been travelling for two weeks, so I’d like to spend some time with Maja this weekend.
  149. # [15:41] <MikeSmith> r? for anybody who has a few minutes to review those just from a wpt best-practices POV
  150. # [15:42] <gsnedders> ato: I'm here for at least the start of next week, FWIW
  151. # [15:42] <gsnedders> but yeah, don't have so much time :)
  152. # [15:42] <ato> The following weekend I’m at a wedding, then on to France for holiday.
  153. # [15:43] <gsnedders> I don't know if I'm here then anyway
  154. # [15:43] <ato> Start of week I really have to get some work done. Behind on a lot of things.
  155. # [15:43] <ato> Sorry to be asocial )-:
  156. # [15:44] <gsnedders> Eh, we can see how stuff goes. See how long I stay here for, see how busy you are.
  157. # [15:44] <gsnedders> I probably can't take that much time out anyway.
  158. # [15:44] <ato> This is the result of a work week without actually doing any work. (-:
  159. # [15:45] <gsnedders> :)
  160. # [15:46] <jgraham> I have in theory either been working or travelling for the last 19 days (although less so on some of them at least)
  161. # [15:46] <jgraham> So this weekend I might try sleeping
  162. # [15:54] * Joins: gitbot (~gitbot@public.cloak)
  163. # [15:54] -gitbot:#testing- [web-platform-tests] sideshowbarker pushed 1 new commit to sideshowbarker/notifications: https://github.com/w3c/web-platform-tests/commit/6798ecc1dd6fd112eec0b38ac51af34cc4bcc234
  164. # [15:54] -gitbot:#testing- web-platform-tests/sideshowbarker/notifications 6798ecc Michael[tm] Smith: fixup! Cleaned up the Notifications tests.
  165. # [15:54] * Parts: gitbot (~gitbot@public.cloak)
  166. # [16:37] <darobin> heya good folks of #testing
  167. # [16:38] <darobin> the honourable MikeSmith and I have a question
  168. # [16:38] <darobin> if you look at https://w3c.github.io/test-results/notifications/FF39.json and scroll all the way down
  169. # [16:38] <darobin> you will see that manual test results have no subtests
  170. # [16:38] <darobin> right now, wptreport assumes everything has subtests, and in fact that's all it looks at
  171. # [16:39] <darobin> I would tend to think that the problem is with the test runner and the way that manual tests are reported
  172. # [16:39] <darobin> for automatic tests, if they didn't blow up, the test has status=OK even if the subtests are FAIL; it's an indication that the test ran to completion
  173. # [16:40] <darobin> but for manual tests the status is set to PASS|FAIL
  174. # [16:40] <darobin> I *could* process this format in wptreport, but I think the format is buggy
  175. # [16:40] <darobin> so... any thoughts?
  176. # [16:41] <jgraham> Do you mean manual tests or CSS tests?
  177. # [16:42] <jgraham> For CSS tests, certainly, the pass/fail is on the overall test because there is always exactly onee result
  178. # [16:42] <jgraham> We can't really change that now (or at least a lot of existing systems assume it works that way)
  179. # [16:43] <MikeSmith> jgraham: so you saying that for CSS tests, the output looks like this:
  180. # [16:43] <MikeSmith> {
  181. # [16:43] <MikeSmith> "test": "/notifications/tag-same-manual.html",
  182. # [16:43] <MikeSmith> "subtests": [],
  183. # [16:43] <MikeSmith> "status": "PASS",
  184. # [16:43] <MikeSmith> "message": ""
  185. # [16:43] <MikeSmith> }
  186. # [16:45] <darobin> if that's what we have for CSS, then we can support it in wptreport
  187. # [16:45] <MikeSmith> jgraham: that is, an empty array for "subtests", and "PASS" for status, rather than "OK"
  188. # [16:45] <jgraham> s/CSS/reftests/
  189. # [16:45] <darobin> yeah, same difference :)
  190. # [16:45] <jgraham> But essentially, yes
  191. # [16:45] <darobin> ok
  192. # [16:45] * darobin fires up his wptreport project
  193. # [16:55] <darobin> MikeSmith: have you updated the Notification tests with the new results?
  194. # [16:57] <MikeSmith> darobin: yeah, pushed them about ~10 minutes ago
  195. # [16:59] <darobin> ok
  196. # [17:01] <darobin> MikeSmith: check the updated less-than-2
  197. # [17:02] <darobin> MikeSmith: also, you can npm install -g wptreport to upgrade
  198. # [17:04] * MikeSmith looks
  199. # [17:04] <MikeSmith> ah, beautiful
  200. # [17:04] <MikeSmith> thanks man
  201. # [17:04] * MikeSmith will also npm install -g wptreport now
  202. # [17:05] <MikeSmith> darobin: you do quick work
  203. # [17:06] <darobin> glad it works :)
  204. # [17:07] <darobin> well, gotta work fast if I'm going to wrap up everything that's on my plate and still take a couple of weeks off this summer!
  205. # [17:11] <Ms2ger> Just a couple weeks?
  206. # [17:12] <darobin> ayup
  207. # [17:25] * Quits: darobin (rberjon@public.cloak) (Client closed connection)
  208. # [17:37] <jgraham> I guess darobin's holidays started ;)
  209. # [18:01] * Joins: JEN (~JEN@public.cloak)
  210. # [18:10] <MikeSmith> mkwst: about the webex thing, I deal with it just be now using any webex client at all but instead just dial in the same as I did with Zakim calls
  211. # [18:11] <MikeSmith> mkwst: see https://www.w3.org/2006/tools/wiki/WebExFAQ#Is_the_WebEx_Java_application_mandatory_to_attend_an_audio-only_meeting_hosted_by_WebEx.3F
  212. # [18:11] <MikeSmith> * I deal with it just by not using any webex client at all
  213. # [18:15] * Parts: JEN (~JEN@public.cloak)
  214. # [18:16] * Quits: plh (plehegar@public.cloak) ("Leaving")
  215. # [18:19] * Quits: dom (dom@public.cloak) ("")
  216. # [20:21] * Quits: burnik (~burnik@public.cloak) (Ping timeout: 180 seconds)
  217. # [21:31] * Joins: burnik (~burnik@public.cloak)
  218. # [21:34] * Joins: burnik1 (~burnik@public.cloak)
  219. # [21:35] * Quits: burnik (~burnik@public.cloak) ("Page closed")
  220. # [21:35] * Parts: burnik1 (~burnik@public.cloak)
  221. # [21:35] * Joins: burnik1 (~burnik@public.cloak)
  222. # [21:36] * Parts: burnik1 (~burnik@public.cloak)
  223. # [21:38] * Joins: burnik1 (~burnik@public.cloak)
  224. # [22:20] * Quits: Ms2ger (~Ms2ger@public.cloak) ("nn")
  225. # Session Close: Sat Jul 04 00:00:00 2015

Previous day, Next day

Think these logs are useful? Then please donate to show your gratitude (and keep them up, of course). Thanks! — Krijn