/irc-logs / w3c / #testing / 2014-12-01 / end

Options:

  1. # Session Start: Mon Dec 01 00:00:00 2014
  2. # Session Ident: #testing
  3. # [00:05] * Joins: bhill2 (~bhill2@public.cloak)
  4. # [00:12] * Quits: bhill2 (~bhill2@public.cloak) (Ping timeout: 180 seconds)
  5. # [00:38] * Quits: Ms2ger (~Ms2ger@public.cloak) ("nn")
  6. # [00:48] * Quits: artb (~ArtB@public.cloak) ("Leaving.")
  7. # [00:59] * Joins: gitbot (~gitbot@public.cloak)
  8. # [00:59] -gitbot:#testing- [web-platform-tests] sideshowbarker pushed 1 new commit to sideshowbarker/custom-config-file-location: https://github.com/w3c/web-platform-tests/commit/b864d48e2e90dae15a35070c36df1e5442dd8092
  9. # [00:59] -gitbot:#testing- web-platform-tests/sideshowbarker/custom-config-file-location b864d48 Michael[tm] Smith: Minor style tweak.
  10. # [00:59] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  11. # [05:31] * Joins: shepazu (schepers@public.cloak)
  12. # [05:58] * Joins: ptressel (~chatzilla@public.cloak)
  13. # [07:24] * Joins: zcorpan (~zcorpan@public.cloak)
  14. # [08:43] * Quits: zcorpan (~zcorpan@public.cloak) (Client closed connection)
  15. # [08:43] * Joins: tripu (antonio@public.cloak)
  16. # [08:44] * Joins: dom (dom@public.cloak)
  17. # [09:10] * Joins: gitbot (~gitbot@public.cloak)
  18. # [09:10] -gitbot:#testing- [web-platform-tests] zqzhang opened pull request #1435: postMessage with zero or one argument shall throw SyntaxError (master...postMessage-exception) https://github.com/w3c/web-platform-tests/pull/1435
  19. # [09:10] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  20. # [09:17] * Joins: zcorpan (~zcorpan@public.cloak)
  21. # [09:19] * Joins: gitbot (~gitbot@public.cloak)
  22. # [09:19] -gitbot:#testing- [web-platform-tests] siusin opened pull request #1436: Remove a case related to __defineGetter__ (master...rm-webmessaging-withoutport-026) https://github.com/w3c/web-platform-tests/pull/1436
  23. # [09:19] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  24. # [09:33] * Joins: rego (~smuxi@public.cloak)
  25. # [10:14] * Disconnected
  26. # [10:17] * Attempting to rejoin channel #testing
  27. # [10:17] * Rejoined channel #testing
  28. # [10:17] * Topic is '"also, I hope no one actually reads this, but I have to admit that I've kind of gotten used to Critic" --darobin'
  29. # [10:17] * Set by Ms2ger on Mon Nov 03 08:55:12
  30. # [10:17] * Quits: wilhelm (~wilhelm@public.cloak) (Client closed connection)
  31. # [10:17] * Joins: wilhelm (~wilhelm@public.cloak)
  32. # [10:18] * Quits: heycam|away (~cam@public.cloak) (Ping timeout: 180 seconds)
  33. # [10:37] * Joins: Ms2ger (~Ms2ger@public.cloak)
  34. # [11:00] * Joins: AutomatedTester (~AutomatedTester@public.cloak)
  35. # [11:04] * Quits: rego (~smuxi@public.cloak) (Ping timeout: 180 seconds)
  36. # [11:06] * Joins: rego (~smuxi@public.cloak)
  37. # [11:07] * Quits: AutomatedTester (~AutomatedTester@public.cloak) (Ping timeout: 180 seconds)
  38. # [11:16] * Joins: Lachy (~Lachy@public.cloak)
  39. # [12:39] * Joins: abarsto (~abarsto@public.cloak)
  40. # [12:39] * abarsto is now known as ArtB
  41. # [13:55] <tripu> Question about the test suite:
  42. # [13:56] <tripu> What does one need to do after adding a new test to web-platform-tests?
  43. # [13:56] <tripu> If I add /foo/bar/test.html, I get a message on the web page, saying the new path doesn't exist.
  44. # [13:57] <tripu> I also re-ran "python tools/scripts/manifest.py", and I see my new test added to the end of the manifest.
  45. # [13:58] <zcorpan> tripu: which web page do you mean?
  46. # [14:05] * Quits: tripu (antonio@public.cloak) (Ping timeout: 180 seconds)
  47. # [14:16] * Joins: tripu (antonio@public.cloak)
  48. # [14:17] <MikeSmith> tripu: you mean when you're running wptserve locally, right?
  49. # [14:17] <MikeSmith> that is, from http://web-platform.test:8000/
  50. # [14:17] <MikeSmith> if so then you don't need to restart wptserve to have your file show up
  51. # [14:18] <MikeSmith> and the manifest doesn't have any relevance to wptserve itself; wptserve itself never reads the manifest
  52. # [14:19] * Joins: scott_gonzalez (~scott_gonzalez@public.cloak)
  53. # [14:19] <MikeSmith> the only part of the browser-based interface that uses the manifest is http://web-platform.test:8000/tools/runner/index.html
  54. # [14:21] <zcorpan> tripu: if you want an index listing of a path, make sure you have the trailing slash
  55. # [14:21] <MikeSmith> so if you added a file in foo/bar/test.html inside your local web-platform-tests working directory, it should show up immediately if you browse to http://web-platform.test:8000/foo/bar/test.html
  56. # [14:21] <MikeSmith> ah yah
  57. # [14:28] <tripu> ok, MikeSmith
  58. # [14:28] <tripu> I'll try again
  59. # [14:33] <tripu> MikeSmith, I've got /foo/align_start.html and /foo/align_start-ref.html (files copied from another subdir)
  60. # [14:34] <tripu> when I enter "/foo" on the web page ("Run tests under path") and click "Start", I see "No tests found in this path."
  61. # [14:34] <tripu> do you know what I might be missing...?
  62. # [14:34] <MikeSmith> tripu: ah so you do mean on http://web-platform.test:8000/foo/bar/test.html
  63. # [14:35] <tripu> Sorry for not being clear, MikeSmith :)
  64. # [14:35] <tripu> I'm testing locally
  65. # [14:35] <tripu> loading http://web-platform.test:8000/tools/runner/index.html
  66. # [14:35] <MikeSmith> yeah
  67. # [14:35] <tripu> If I go to http://web-platform.test:8000/foo/ I see listed those two files, and i can load them
  68. # [14:35] <MikeSmith> try reloading the page
  69. # [14:36] <MikeSmith> reloading http://web-platform.test:8000/tools/runner/index.html
  70. # [14:36] <tripu> ok
  71. # [14:36] <tripu> mmh same thing
  72. # [14:36] <MikeSmith> and the manifest json file contains that path?
  73. # [14:37] <tripu> [[
  74. # [14:37] <tripu> "reftest": {
  75. # [14:37] <tripu> "foo/align_start.html": [
  76. # [14:37] <tripu> {
  77. # [14:37] <tripu> "ref_type": "==",
  78. # [14:37] <tripu> "ref_url": "/foo/align_start-ref.html",
  79. # [14:37] <tripu> "url": "/foo/align_start.html"
  80. # [14:37] <tripu> }
  81. # [14:37] <tripu> ],
  82. # [14:37] <tripu> ]]
  83. # [14:37] <tripu> that's around the end of MANIFEST.json
  84. # [14:38] <MikeSmith> try it from http://web-platform.test:8000/tools/runner/index.html in a different browser
  85. # [14:38] <tripu> but all that's under "local changes", not in the very long "helper" array
  86. # [14:38] <tripu> ok
  87. # [14:39] <tripu> same thing with a different browser, refreshing cache
  88. # [14:40] <MikeSmith> ok I think it ignores stuff in the local_changes object
  89. # [14:41] <MikeSmith> I think it only puts stuff in there that it thinks are non-tests
  90. # [14:41] <MikeSmith> so it's mis-categorizing the test I think
  91. # [14:41] <MikeSmith> maybe
  92. # [14:41] <tripu> I see
  93. # [14:41] * tripu 's messing with MANIFEST.json now
  94. # [14:44] <MikeSmith> hang on
  95. # [14:44] <MikeSmith> I think there's a switch you can feed to the manifest.py script
  96. # [14:46] <tripu> Oh, I think I was adding my reftest to the wrong object in the json file
  97. # [14:46] <tripu> I just added it to the "reftest" object, like this:
  98. # [14:46] <tripu> [[
  99. # [14:46] <tripu> {
  100. # [14:46] <tripu> "ref_type": "==",
  101. # [14:46] <tripu> "ref_url": "/foo/align_start-ref.html",
  102. # [14:46] <tripu> "url": "/foo/align_start.html"
  103. # [14:46] <tripu> },
  104. # [14:46] <tripu> ]]
  105. # [14:47] <tripu> and now it lets me specify that path on the web interface, and run only tests under that subdir
  106. # [14:49] <MikeSmith> manually you added it by editing the file?
  107. # [14:49] <tripu> yes
  108. # [14:49] <MikeSmith> you shouldn't have to manually edit it
  109. # [14:50] <MikeSmith> I think something may have regressed recently
  110. # [14:50] <MikeSmith> we need jgraham or ato to weigh in
  111. # [14:50] <MikeSmith> hmm
  112. # [14:50] <MikeSmith> or actually, I think it may be intentional
  113. # [14:51] <MikeSmith> tripu: you didn't commit those changes, I guess?
  114. # [14:51] <MikeSmith> I think the intent of the runner it to ignore uncommitted files
  115. # [14:51] <tripu> no commit, MikeSmith
  116. # [14:51] <MikeSmith> yeah so, that what it means by "local changes"
  117. # [14:51] <MikeSmith> = uncommitted
  118. # [14:52] <tripu> oh, right!
  119. # [14:52] <MikeSmith> the way you tell it you want it to pay attention to a file is, you commit it to the current branch
  120. # [14:52] <tripu> ok, that makes sense
  121. # [14:52] <MikeSmith> so you should always be working on a development branch locally, not committing to master of course
  122. # [14:52] <MikeSmith> and you can always delete that local branch without pushing it
  123. # [14:53] <tripu> sure
  124. # [14:53] <MikeSmith> if you're just experimenting
  125. # [14:53] <MikeSmith> so yeah I'm pretty sure it will show up as expected after you commit and regenerated the manifest
  126. # [14:54] * tripu 'll try that
  127. # [14:54] <MikeSmith> and I think the browser runner at http://web-platform.test:8000/tools/runner/index.html will re-generate the manifest automatically, without you needing to regenerate it from the command line
  128. # [14:54] <MikeSmith> I think
  129. # [14:55] <MikeSmith> but of course if you want you can manually re-generate it by running the script
  130. # [14:55] <MikeSmith> anyway, yeah lemme know if that doesn't work
  131. # [15:03] <tripu> confirmed, MikeSmith: after committing the new directory with its tests, and reloading the runner, the new path is accepted as a parameter
  132. # [15:03] <tripu> thanks! :)
  133. # [15:04] <MikeSmith> tripu: supper
  134. # [15:04] <tripu> (no need to regenerate the manifest)
  135. # [15:04] <MikeSmith> *super
  136. # [15:04] <MikeSmith> cool
  137. # [15:04] <MikeSmith> tripu: thank jgraham :)
  138. # [15:16] * Joins: plh (plehegar@public.cloak)
  139. # [17:00] * Quits: zcorpan (~zcorpan@public.cloak) (Client closed connection)
  140. # [17:50] * Joins: lukeis (~Adium@public.cloak)
  141. # [18:07] * Joins: bhill2 (~bhill2@public.cloak)
  142. # [18:07] * Quits: bhill2 (~bhill2@public.cloak) (Client closed connection)
  143. # [18:08] * Joins: bhill2 (~bhill2@public.cloak)
  144. # [18:11] * Joins: zcorpan (~zcorpan@public.cloak)
  145. # [18:18] * Quits: zcorpan (~zcorpan@public.cloak) (Ping timeout: 180 seconds)
  146. # [18:20] * Quits: tripu (antonio@public.cloak) ("Leaving")
  147. # [18:29] * Quits: lukeis (~Adium@public.cloak) ("Leaving.")
  148. # [18:39] * Quits: chaals (~Adium@public.cloak) (Request too long)
  149. # [18:39] * Joins: chaals (~Adium@public.cloak)
  150. # [18:48] * Joins: zcorpan (~zcorpan@public.cloak)
  151. # [18:49] * Quits: dom (dom@public.cloak) ("")
  152. # [18:51] * Quits: Lachy (~Lachy@public.cloak) ("My MacBook Pro has gone to sleep. ZZZzzz…")
  153. # [18:56] * Quits: zcorpan (~zcorpan@public.cloak) (Client closed connection)
  154. # [19:01] * Joins: lukeis (~Adium@public.cloak)
  155. # [19:13] * Quits: ArtB (~abarsto@public.cloak) ("Leaving.")
  156. # [19:19] * Quits: lukeis (~Adium@public.cloak) ("Leaving.")
  157. # [19:23] * Joins: hober (~ted@public.cloak)
  158. # [19:37] * Joins: artb (~ArtB@public.cloak)
  159. # [19:37] * Quits: artb (~ArtB@public.cloak) ("Leaving.")
  160. # [19:38] * Joins: artb (~ArtB@public.cloak)
  161. # [19:57] * Joins: zcorpan (~zcorpan@public.cloak)
  162. # [19:57] * Joins: lukeis (~Adium@public.cloak)
  163. # [20:26] * Joins: gitbot (~gitbot@public.cloak)
  164. # [20:26] -gitbot:#testing- [web-platform-tests] jacobrossi opened pull request #1437: Submission/microsoft/pointer events fix test failures (master...submission/Microsoft/PointerEvents-FixTestFailures) https://github.com/w3c/web-platform-tests/pull/1437
  165. # [20:26] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  166. # [20:38] * Joins: gitbot (~gitbot@public.cloak)
  167. # [20:38] -gitbot:#testing- [web-platform-tests] jacobrossi closed pull request #1437: Submission/microsoft/pointer events fix test failures (master...submission/Microsoft/PointerEvents-FixTestFailures) https://github.com/w3c/web-platform-tests/pull/1437
  168. # [20:38] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  169. # [20:38] * Joins: gitbot (~gitbot@public.cloak)
  170. # [20:38] -gitbot:#testing- [web-platform-tests] jacobrossi pushed 9 new commits to master: https://github.com/w3c/web-platform-tests/compare/08b423ec34da...423bd66c8aa8
  171. # [20:38] -gitbot:#testing- web-platform-tests/master 4b282e8 Evgeny Agafonchikov: Make the test behave equally in different UAs...
  172. # [20:38] -gitbot:#testing- web-platform-tests/master 2db3433 Evgeny Agafonchikov: Add explicit done() call to avoid harness timeout
  173. # [20:38] -gitbot:#testing- web-platform-tests/master 49eccfe Jacob Rossi: Merge branch 'submission/Microsoft/PointerEvents-AddExplicitDone' of https://github.com/InternetExplorer/web-platform-tests-msft into submission/Microsoft/PointerEvents-FixTestFailures
  174. # [20:38] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  175. # [21:45] * Quits: lukeis (~Adium@public.cloak) ("Leaving.")
  176. # [22:14] * Joins: lukeis (~Adium@public.cloak)
  177. # [22:15] * Quits: plh (plehegar@public.cloak) ("Leaving")
  178. # [22:16] * Joins: Lachy (~Lachy@public.cloak)
  179. # [22:43] * terri is now known as terri_offline
  180. # [22:54] * terri_offline is now known as terri
  181. # [22:57] * Joins: zcorpan_ (~zcorpan@public.cloak)
  182. # [22:57] * Quits: zcorpan (~zcorpan@public.cloak) (Client closed connection)
  183. # [23:04] * Quits: zcorpan_ (~zcorpan@public.cloak) (Ping timeout: 180 seconds)
  184. # [23:14] * Quits: shepazu (schepers@public.cloak) (Client closed connection)
  185. # [23:14] * Joins: shepazu (schepers@public.cloak)
  186. # [23:50] * Quits: shepazu (schepers@public.cloak) (Ping timeout: 180 seconds)
  187. # [23:53] * Quits: ptressel (~chatzilla@public.cloak) ("zzz")
  188. # [23:58] * Joins: shepazu (schepers@public.cloak)
  189. # [23:58] * Quits: Ms2ger (~Ms2ger@public.cloak) ("nn")
  190. # Session Close: Tue Dec 02 00:00:00 2014

The end :)