/irc-logs / w3c / #testing / 2013-08-06 / end

Options:

  1. # Session Start: Tue Aug 06 00:00:00 2013
  2. # Session Ident: #testing
  3. # [00:06] * Quits: tobie (tobie@public.cloak)
  4. # [00:25] * Joins: jhammel (~jhammel@public.cloak)
  5. # [00:26] * Joins: AutomatedTester (~AutomatedTester@public.cloak)
  6. # [00:47] * Joins: gitbot (~gitbot@public.cloak)
  7. # [00:47] -gitbot:#testing- [web-platform-tests] plehegar closed pull request #190: Added definition for Event and EventInit (master...submissions/plh) https://github.com/w3c/web-platform-tests/pull/190
  8. # [00:47] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  9. # [00:52] * Quits: zcorpan (~zcorpan@public.cloak) (Client closed connection)
  10. # [01:22] * Quits: AutomatedTester (~AutomatedTester@public.cloak) (Ping timeout: 180 seconds)
  11. # [01:22] * Joins: zcorpan (~zcorpan@public.cloak)
  12. # [01:32] * Joins: AutomatedTester (~AutomatedTester@public.cloak)
  13. # [01:39] * heycam|away is now known as heycam
  14. # [01:43] * Joins: darobin (rberjon@public.cloak)
  15. # [01:43] * Quits: darobin_ (rberjon@public.cloak) (Client closed connection)
  16. # [02:15] * Joins: Automate_ (~AutomatedTester@public.cloak)
  17. # [02:15] * Quits: AutomatedTester (~AutomatedTester@public.cloak) (Client closed connection)
  18. # [02:44] * Joins: AutomatedTester (~AutomatedTester@public.cloak)
  19. # [02:45] * Quits: Automate_ (~AutomatedTester@public.cloak) (Ping timeout: 180 seconds)
  20. # [02:47] * Quits: jhammel (~jhammel@public.cloak) ("leaving")
  21. # [02:57] * Quits: AutomatedTester (~AutomatedTester@public.cloak) (Client closed connection)
  22. # [02:58] * Quits: ArtB (~abarsto@public.cloak) ("Leaving.")
  23. # [03:21] * Quits: rhauck (~Adium@public.cloak) ("Leaving.")
  24. # [03:21] * Joins: rhauck (~Adium@public.cloak)
  25. # [03:28] * Quits: rhauck (~Adium@public.cloak) (Ping timeout: 180 seconds)
  26. # [04:26] * heycam is now known as heycam|away
  27. # [04:57] * heycam|away is now known as heycam
  28. # [04:58] * Quits: zcorpan (~zcorpan@public.cloak) (Ping timeout: 180 seconds)
  29. # [07:51] * Quits: glenn (~gadams@public.cloak) (Client closed connection)
  30. # [09:10] * Joins: dom (dom@public.cloak)
  31. # [09:20] * Joins: zcorpan (~zcorpan@public.cloak)
  32. # [09:21] * Joins: zcorpan_ (~zcorpan@public.cloak)
  33. # [09:21] * Quits: zcorpan (~zcorpan@public.cloak) (Client closed connection)
  34. # [10:00] * Quits: Lachy (~Lachy@public.cloak) ("Computer has gone to sleep.")
  35. # [10:13] * heycam is now known as heycam|away
  36. # [10:17] * Joins: Ms2ger (~Ms2ger@public.cloak)
  37. # [10:31] * Joins: Lachy (~Lachy@public.cloak)
  38. # [11:46] * Quits: darobin (rberjon@public.cloak) (Client closed connection)
  39. # [11:50] * Joins: darobin (rberjon@public.cloak)
  40. # [12:31] * Joins: abarsto (~abarsto@public.cloak)
  41. # [12:31] * abarsto is now known as ArtB
  42. # [12:43] * Quits: darobin (rberjon@public.cloak) (Client closed connection)
  43. # [12:44] * Joins: darobin (rberjon@public.cloak)
  44. # [13:22] * Joins: darobin_ (rberjon@public.cloak)
  45. # [13:28] * Quits: darobin (rberjon@public.cloak) (Ping timeout: 180 seconds)
  46. # [13:30] * Joins: AutomatedTester (~AutomatedTester@public.cloak)
  47. # [13:46] <gsnedders> Is there any metadata to tell if a test is using testharness.js, a reftest, or manual? How is it meant to be determined?
  48. # [13:47] <Ms2ger> Yeah, the MANIFEST files
  49. # [13:48] <gsnedders> Where is this documented? (Nowhere?)
  50. # [13:49] <Ms2ger> https://bitbucket.org/ms2ger/test-runner/src/95937b86c969a4861a0301ee1eabcc67ae2055e3/manifests.txt?at=default
  51. # [13:50] <gsnedders> Yay for obvious places for documentation.
  52. # [13:51] <Ms2ger> Yeah, with its first consumer ;)
  53. # [14:00] <gsnedders> Ms2ger: Do I need MANIFEST files in every single parent directory?
  54. # [14:01] * Joins: gitbot (~gitbot@public.cloak)
  55. # [14:01] -gitbot:#testing- [web-platform-tests] ayg opened pull request #265: Fixes to Range tests (master...master) https://github.com/w3c/web-platform-tests/pull/265
  56. # [14:01] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  57. # [14:01] <gsnedders> With "dir foo"?
  58. # [14:02] <Ms2ger> Probably, yes
  59. # [14:03] <gsnedders> Le sigh.
  60. # [14:03] <gsnedders> Having to go through loads of existing directories…
  61. # [14:06] <gsnedders> Ms2ger: Currently they don't, FWIW
  62. # [14:07] <Ms2ger> Yeah, I've been adding them as I wanted to run / import them
  63. # [14:12] <gsnedders> Well, still, shouldn't the top level have one, then?
  64. # [14:14] <Ms2ger> We cherrypick some subdirs as roots
  65. # [14:14] <gsnedders> https://github.com/gsnedders/web-platform-tests/tree/4ec322ac572e9a6e5662f6b2c83aa42da4d624fd/html/browsers/windows/nested-browsing-contexts has a contains.json — should I be doing something different?
  66. # [14:14] <gsnedders> (Do I want a .gitkeep in there?)
  67. # [14:15] <gsnedders> (In the child I created, I mean?)
  68. # [14:18] <Ms2ger> I don't remember what that was for
  69. # [14:18] <Ms2ger> darobin_?
  70. # [14:20] <gsnedders> It seems to define an alias to the original?
  71. # [14:21] <darobin_> mmmwha?
  72. # [14:21] * darobin_ is now known as darobin
  73. # [14:21] * Joins: glenn (~gadams@public.cloak)
  74. # [14:22] <darobin> ah, I've read the backlog now
  75. # [14:22] <gsnedders> darobin: I'm giving you a chance to shout at me now so you don't during review.
  76. # [14:22] <darobin> gsnedders: contains.json and .gitkeep were made for the HTML TS, you may or may not need them depending on what you're doing
  77. # [14:23] <gsnedders> contains.json was to rename folders, right?
  78. # [14:23] <gsnedders> Then a contains.json where id and original_id are identical is pointless, which is the state that that one is in.
  79. # [14:23] <darobin> no, contains.json was because we decided to only go three levels deep in the spec's sections' hierarchy
  80. # [14:23] <gsnedders> Oh, okay.
  81. # [14:23] <gsnedders> So I have gone wrong.
  82. # [14:23] <darobin> ah, I forgot about original_id
  83. # [14:23] <gsnedders> What should I be doing instead? Prefixing everything?
  84. # [14:24] <gsnedders> (Instead of creating a fourth level of folders?)
  85. # [14:24] <darobin> but the basic idea of contains.json was to keep the sections information around
  86. # [14:24] <gsnedders> Because that implies my having created a forth level of folders is… wrong.
  87. # [14:24] <darobin> .gitkeep is just a dummy file to ensure that empty directories stay in git
  88. # [14:25] <darobin> hmmmm, let me look at the exact thing you're working on, it'll help me swap in
  89. # [14:25] <gsnedders> See the link above.
  90. # [14:25] <darobin> yes looking now
  91. # [14:25] <gsnedders> I created a forth level within there.
  92. # [14:25] <darobin> aha, got it
  93. # [14:25] <darobin> well
  94. # [14:25] <darobin> I don't mind there being a fourth level if it makes sense to you
  95. # [14:26] <darobin> please use the section's ID name to name it
  96. # [14:26] <darobin> and I would say ignore contains.json and .gitkeep
  97. # [14:26] <gsnedders> Which is what I did. :P
  98. # [14:26] <darobin> if it turns out we really need them, we can add them
  99. # [14:27] <darobin> they were created because when I generated the empty tree it's information I had anyway, so I might just as well keep it around
  100. # [14:27] <darobin> but I don't believe that we've been using it
  101. # [14:27] * Joins: Automate_ (~AutomatedTester@public.cloak)
  102. # [14:27] * Quits: AutomatedTester (~AutomatedTester@public.cloak) (Client closed connection)
  103. # [14:27] <gsnedders> I'm happy to pull things up a level, or break into the forth level.
  104. # [14:27] <gsnedders> I don't think it makes much difference.
  105. # [14:27] <darobin> gsnedders: do whatever makes your life easiest
  106. # [14:27] <darobin> we're not in the business of making things harder for test writers
  107. # [14:28] <darobin> well, except in places where we use Python :)
  108. # [14:28] <gsnedders> Well, there's only one test there, and `git mv xxx/* .` isn't exactly hard. :)
  109. # [14:28] <gsnedders> So I really don't care.
  110. # [14:28] <gsnedders> And I've moved it now.
  111. # [14:28] <gsnedders> So that's it.
  112. # [14:28] <darobin> if it's only one test yeah move it up
  113. # [14:28] <darobin> cool, ta
  114. # [14:28] * Joins: gitbot (~gitbot@public.cloak)
  115. # [14:28] -gitbot:#testing- [web-platform-tests] ayg closed pull request #265: Fixes to Range tests (master...master) https://github.com/w3c/web-platform-tests/pull/265
  116. # [14:28] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  117. # [14:28] <gsnedders> Because otherwise we're talking about it more than it takes to change.
  118. # [14:29] <darobin> there may be tools like the coverage thing that wrongly assume three levels; they would need to be fixed eventually but I'd rather not fix them just for one test
  119. # [14:29] <darobin> indeed
  120. # [14:29] <darobin> ta
  121. # [14:34] * Quits: zcorpan_ (~zcorpan@public.cloak) (Client closed connection)
  122. # [14:43] <gsnedders> All scripts should be PHP, right?
  123. # [14:50] <darobin> gsnedders: you mean stuff that's server-side?
  124. # [14:50] <darobin> we're moving away from PHP, but right now I think it's the only thing that works
  125. # [15:02] <gsnedders> What /should/ I use?
  126. # [15:04] <darobin> gsnedders: I reckon for now PHP, unless Ms2ger has another idea
  127. # [15:04] <Ms2ger> Yeah
  128. # [15:09] <gsnedders> Then rewriting a script I don't understand in PHP!
  129. # [15:22] <gsnedders> Where should I put a script shared by several tests?
  130. # [15:22] <gsnedders> PHP script, that is?
  131. # [15:22] * Quits: Automate_ (~AutomatedTester@public.cloak) (Client closed connection)
  132. # [16:16] * Joins: AutomatedTester (~AutomatedTester@public.cloak)
  133. # [16:52] * Quits: AutomatedTester (~AutomatedTester@public.cloak) (Client closed connection)
  134. # [16:58] <darobin> gsnedders: I guess put it at the earliest common ancestor of the tests that use it
  135. # [16:58] <darobin> gsnedders: writing stuff you don't understand is the right way of doing PHP, go you
  136. # [17:07] * Joins: AutomatedTester (~AutomatedTester@public.cloak)
  137. # [17:11] * Quits: Lachy (~Lachy@public.cloak) ("Computer has gone to sleep.")
  138. # [17:18] <gsnedders> darobin: Well, no, I understand what the script /does/, just not how it does it. The new impl is entirely different. :P
  139. # [17:19] * Joins: rhauck (~Adium@public.cloak)
  140. # [17:32] * Quits: AutomatedTester (~AutomatedTester@public.cloak) (Client closed connection)
  141. # [17:39] * Joins: AutomatedTester (~AutomatedTester@public.cloak)
  142. # [17:52] * Quits: dom (dom@public.cloak) ("")
  143. # [17:53] * jgraham wonders what gsnedders is doing
  144. # [18:11] * Quits: AutomatedTester (~AutomatedTester@public.cloak) (Client closed connection)
  145. # [18:11] * Quits: rhauck (~Adium@public.cloak) (Ping timeout: 180 seconds)
  146. # [18:16] * Joins: jhammel (~jhammel@public.cloak)
  147. # [19:08] * Joins: rhauck (~Adium@public.cloak)
  148. # [19:17] * Quits: glenn (~gadams@public.cloak) (Client closed connection)
  149. # [19:28] * Joins: AutomatedTester (~AutomatedTester@public.cloak)
  150. # [19:34] <jgraham> Hmm, did the PR sync break again?
  151. # [19:35] * Quits: AutomatedTester (~AutomatedTester@public.cloak) (Ping timeout: 180 seconds)
  152. # [19:42] * Joins: AutomatedTester (~AutomatedTester@public.cloak)
  153. # [19:56] * Joins: rhauck1 (~Adium@public.cloak)
  154. # [20:02] * Quits: rhauck (~Adium@public.cloak) (Ping timeout: 180 seconds)
  155. # [20:20] * Joins: gitbot (~gitbot@public.cloak)
  156. # [20:20] -gitbot:#testing- [web-platform-tests] AFBarstow opened pull request #266: Tests for the Web Notifications spec (master...master) https://github.com/w3c/web-platform-tests/pull/266
  157. # [20:20] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  158. # [20:22] * Joins: Lachy (~Lachy@public.cloak)
  159. # [20:26] * Joins: gitbot (~gitbot@public.cloak)
  160. # [20:26] -gitbot:#testing- [web-platform-tests] jgraham closed pull request #266: Tests for the Web Notifications spec (master...master) https://github.com/w3c/web-platform-tests/pull/266
  161. # [20:26] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  162. # [20:27] * Joins: glenn (~gadams@public.cloak)
  163. # [20:27] * Quits: rhauck1 (~Adium@public.cloak) ("Leaving.")
  164. # [20:30] * Joins: rhauck (~Adium@public.cloak)
  165. # [20:34] * Quits: glenn (~gadams@public.cloak) (Ping timeout: 180 seconds)
  166. # [20:45] * Joins: gitbot (~gitbot@public.cloak)
  167. # [20:45] -gitbot:#testing- [web-platform-tests] jgraham pushed 3 new commits to submission/AryehGregor: https://github.com/w3c/web-platform-tests/compare/e898d1058fb6...b34b8d81e6af
  168. # [20:45] -gitbot:#testing- web-platform-tests/submission/AryehGregor 5fc2595 James Graham: Add valueOf:null to objects with custom toString
  169. # [20:45] -gitbot:#testing- web-platform-tests/submission/AryehGregor 93558ae James Graham: Update crossOrigin attribute definitions.
  170. # [20:45] -gitbot:#testing- web-platform-tests/submission/AryehGregor b34b8d8 James Graham: input width and height are only reflected on setting.
  171. # [20:45] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  172. # [20:52] * Joins: zcorpan (~zcorpan@public.cloak)
  173. # [21:14] * Quits: AutomatedTester (~AutomatedTester@public.cloak) (Ping timeout: 180 seconds)
  174. # [21:20] * Joins: AutomatedTester (~AutomatedTester@public.cloak)
  175. # [21:28] * Quits: ArtB (~abarsto@public.cloak) ("Leaving.")
  176. # [21:34] * Quits: AutomatedTester (~AutomatedTester@public.cloak) (Ping timeout: 180 seconds)
  177. # [21:36] * Quits: Ms2ger (~Ms2ger@public.cloak) ("nn")
  178. # [21:56] * Joins: AutomatedTester (~AutomatedTester@public.cloak)
  179. # [22:44] * Quits: jhammel (~jhammel@public.cloak) ("leaving")
  180. # [22:48] * Joins: jhammel (~jhammel@public.cloak)
  181. # [22:48] * Quits: zcorpan (~zcorpan@public.cloak) (Client closed connection)
  182. # [22:57] * Quits: Lachy (~Lachy@public.cloak) ("Computer has gone to sleep.")
  183. # [23:32] * Joins: Automate_ (~AutomatedTester@public.cloak)
  184. # [23:32] * Quits: jhammel (~jhammel@public.cloak) (Client closed connection)
  185. # [23:32] * Quits: AutomatedTester (~AutomatedTester@public.cloak) (Client closed connection)
  186. # [23:41] * Joins: jhammel (~jhammel@public.cloak)
  187. # Session Close: Wed Aug 07 00:00:00 2013

The end :)