/irc-logs / w3c / #testing / 2015-05-23 / end

Options:

Previous day, Next day

  1. # Session Start: Sat May 23 00:00:00 2015
  2. # Session Ident: #testing
  3. # [00:21] * Joins: Lachy (~Lachy@public.cloak)
  4. # [00:40] * Quits: Lachy (~Lachy@public.cloak) ("My MacBook Pro has gone to sleep. ZZZzzz…")
  5. # [00:57] * Joins: Lachy (~Lachy@public.cloak)
  6. # [00:58] * Quits: Lachy (~Lachy@public.cloak) ("Textual IRC Client: www.textualapp.com")
  7. # [00:59] * Joins: Lachy (~Lachy@public.cloak)
  8. # [00:59] * Quits: Lachy (~Lachy@public.cloak) ("Textual IRC Client: www.textualapp.com")
  9. # [01:22] * terri is now known as terri_offline
  10. # [01:57] * terri_offline is now known as terri
  11. # [03:36] * Quits: bhill2 (~bhill2@public.cloak) (Client closed connection)
  12. # [03:36] * Joins: bhill2 (~bhill2@public.cloak)
  13. # [03:43] * Quits: bhill2 (~bhill2@public.cloak) (Ping timeout: 180 seconds)
  14. # [05:34] * terri is now known as terri_offline
  15. # [09:44] * Joins: playjax (~chatzilla@public.cloak)
  16. # [10:08] * Joins: Ms2ger (~Ms2ger@public.cloak)
  17. # [10:14] <MikeSmith> jgraham: I thought you had created a PR for the "running tests on sauce labs" but I guess you haven't yet
  18. # [10:15] * MikeSmith wanted to try that
  19. # [10:15] <MikeSmith> jgraham: when you're back around can you DM me the username and password place
  20. # [10:15] <MikeSmith> *please
  21. # [10:16] * Joins: gitbot (~gitbot@public.cloak)
  22. # [10:16] -gitbot:#testing- [web-platform-tests] mikewest created sri-w3ctest-example (+1 new commit): https://github.com/w3c/web-platform-tests/commit/1261622a90b5
  23. # [10:16] -gitbot:#testing- web-platform-tests/sri-w3ctest-example 1261622 Mike West: SRI: s/w3c-test.org/example.test/g...
  24. # [10:16] * Parts: gitbot (~gitbot@public.cloak)
  25. # [10:16] * Joins: gitbot (~gitbot@public.cloak)
  26. # [10:16] -gitbot:#testing- [web-platform-tests] mikewest opened pull request #1847: SRI: s/w3c-test.org/example.test/g (master...sri-w3ctest-example) https://github.com/w3c/web-platform-tests/pull/1847
  27. # [10:16] * Parts: gitbot (~gitbot@public.cloak)
  28. # [10:16] <MikeSmith> or tobie if you're around and remember the saucelabs username and password
  29. # [10:58] * Joins: gitbot (~gitbot@public.cloak)
  30. # [10:58] -gitbot:#testing- [wptrunner] sideshowbarker closed pull request #97: Use postMessage for notifications to the harness window. (master...jgraham/notify_postmessage) https://github.com/w3c/wptrunner/pull/97
  31. # [10:58] * Parts: gitbot (~gitbot@public.cloak)
  32. # [10:58] * Joins: gitbot (~gitbot@public.cloak)
  33. # [10:58] -gitbot:#testing- [wptrunner] sideshowbarker deleted jgraham/notify_postmessage at 7cc13d2: https://github.com/w3c/wptrunner/commit/7cc13d2
  34. # [10:58] * Parts: gitbot (~gitbot@public.cloak)
  35. # [11:36] * Joins: gitbot (~gitbot@public.cloak)
  36. # [11:36] -gitbot:#testing- [wptrunner] sideshowbarker closed pull request #101: Fix Chrome support so that --webdriver-binary is required and --binary is optional (master...jgraham/chrome_args_check) https://github.com/w3c/wptrunner/pull/101
  37. # [11:36] * Parts: gitbot (~gitbot@public.cloak)
  38. # [11:36] * Joins: gitbot (~gitbot@public.cloak)
  39. # [11:36] -gitbot:#testing- [wptrunner] sideshowbarker deleted jgraham/chrome_args_check at ae6dc5f: https://github.com/w3c/wptrunner/commit/ae6dc5f
  40. # [11:36] * Parts: gitbot (~gitbot@public.cloak)
  41. # [11:57] * Joins: gitbot (~gitbot@public.cloak)
  42. # [11:57] -gitbot:#testing- [wptrunner] sideshowbarker created sideshowbarker/docs (+1 new commit): https://github.com/w3c/wptrunner/commit/802082001b31
  43. # [11:57] -gitbot:#testing- wptrunner/sideshowbarker/docs 8020820 Michael[tm] Smith: [docs] Sync docs to current wptrunner behavior.
  44. # [11:57] * Parts: gitbot (~gitbot@public.cloak)
  45. # [12:07] * Joins: gitbot (~gitbot@public.cloak)
  46. # [12:07] -gitbot:#testing- [wptrunner] sideshowbarker opened pull request #115: [docs] Sync docs to current wptrunner behavior. (master...sideshowbarker/docs) https://github.com/w3c/wptrunner/pull/115
  47. # [12:07] * Parts: gitbot (~gitbot@public.cloak)
  48. # [12:19] * Joins: gitbot (~gitbot@public.cloak)
  49. # [12:19] -gitbot:#testing- [web-platform-tests] sideshowbarker closed pull request #1819: Add a couple more test cases to urltestdata.txt (master...patch-1) https://github.com/w3c/web-platform-tests/pull/1819
  50. # [12:19] * Parts: gitbot (~gitbot@public.cloak)
  51. # [12:22] * Joins: gitbot (~gitbot@public.cloak)
  52. # [12:22] -gitbot:#testing- [web-platform-tests] sideshowbarker closed pull request #1829: Add tests for Document#getElementById(null/undefined). (master...gEBI-null-undefined) https://github.com/w3c/web-platform-tests/pull/1829
  53. # [12:22] * Parts: gitbot (~gitbot@public.cloak)
  54. # [12:22] * Joins: gitbot (~gitbot@public.cloak)
  55. # [12:22] -gitbot:#testing- [web-platform-tests] sideshowbarker pushed 2 new commits to master: https://github.com/w3c/web-platform-tests/compare/1ce518c835f6...f68c1bfccf2e
  56. # [12:22] -gitbot:#testing- web-platform-tests/master 0082d4e Ms2ger: Add tests for Document#getElementById(null/undefined).
  57. # [12:22] -gitbot:#testing- web-platform-tests/master f68c1bf Michael[tm] Smith: Merge pull request #1829 from Ms2ger/gEBI-null-undefined...
  58. # [12:22] * Parts: gitbot (~gitbot@public.cloak)
  59. # [12:27] <MikeSmith> https://travis-ci.org/w3c/web-platform-tests/builds/63676814 fails linting.. why?
  60. # [12:27] <MikeSmith> "ImportError: No module named tools.lint"
  61. # [12:27] <MikeSmith> "Could not locate requirements.txt. Override the install: key in your .travis.yml to install dependencies."
  62. # [12:38] <Ms2ger> While you're reviewing... :)
  63. # [13:00] <MikeSmith> Ms2ger: what else you got?
  64. # [13:01] <Ms2ger> https://github.com/w3c/web-platform-tests/pull/1808 https://github.com/w3c/web-platform-tests/pull/1805 https://github.com/w3c/web-platform-tests/pull/1800
  65. # [13:01] <Ms2ger> Any you feel like, of course :)
  66. # [13:02] * MikeSmith looks
  67. # [13:04] * MikeSmith takes a break from trying to figure out what's wrong the linter and instead find problems in Ms2ger PRs
  68. # [13:05] <Ms2ger> If there's an issue with the linter, maybe the submodule doesn't get initialized right?
  69. # [13:06] <MikeSmith> yes, it seems so
  70. # [13:07] <MikeSmith> but the question is why the submodule doesn't get initialized right
  71. # [13:07] <MikeSmith> this is with travis
  72. # [13:07] <MikeSmith> on PR branches
  73. # [13:07] <MikeSmith> recent ones
  74. # [13:07] <Ms2ger> See topic :)
  75. # [13:08] <MikeSmith> Ms2ger: about "Failed to construct 'URL': Invalid URL", it seems like the test file should cause that to be a pass, not a fail
  76. # [13:09] <MikeSmith> I mean if a URL is invalid it's expected that the constructor should fail
  77. # [13:09] <MikeSmith> heh
  78. # [13:10] <Ms2ger> You make a good point
  79. # [13:10] * Ms2ger copied blindly from the the a.href setter
  80. # [13:11] <MikeSmith> I think the topic should be more like, "A programmer has a problem. They think "I know I'll use submodules". Now they want to set everything else aside and murder the person who invented submodules."
  81. # [13:11] <Ms2ger> ... and still have a tree of problems
  82. # [13:12] <MikeSmith> heh
  83. # [13:15] <MikeSmith> what requirement does this "is not a valid URL"/"Failed to construct 'URL': Invalid URL" error map to in the spec?
  84. # [13:16] <MikeSmith> trying to remember what the spec says about a specific error condition for invalid URLs
  85. # [13:16] <MikeSmith> other than just TyperError
  86. # [13:19] <Ms2ger> Yeah, looks like it's a TypeError
  87. # [13:23] <MikeSmith> but it just throws TypeError for everyting (not just invalid URLs), right?
  88. # [13:24] <Ms2ger> What's "everything"?
  89. # [13:25] <Ms2ger> Anyway, fixup!'d
  90. # [13:26] <MikeSmith> k
  91. # [13:29] <MikeSmith> Ms2ger: pulled and merged the fixup but still getting some "is not a valid URL" failures
  92. # [13:30] <Ms2ger> Did you get both fixups?
  93. # [13:31] <MikeSmith> not sure, retrying now
  94. # [13:34] <MikeSmith> so yeah I got both fixups but I'm still getting "is not a valid URL" failures
  95. # [13:37] <MikeSmith> plus lint failue for "subresource-integrity/refresh-header.js.headers" https://travis-ci.org/w3c/web-platform-tests/builds/63761930
  96. # [13:37] <Ms2ger> Oh
  97. # [13:38] <Ms2ger> Did someone merge something that failed travis?
  98. # [13:40] <MikeSmith> seems somebody must have
  99. # [13:40] <Ms2ger> Yup
  100. # [13:40] <MikeSmith> brad hill I think
  101. # [13:41] <MikeSmith> but that error message from the linter is basically useless
  102. # [13:41] <MikeSmith> can't tell why exactly it's failing
  103. # [13:41] <Ms2ger> W3C-TEST.ORG: subresource-integrity/refresh-header.js.headers line 1
  104. # [13:41] <Ms2ger> There was 1 error (W3C-TEST.ORG: 1)
  105. # [13:41] <MikeSmith> yeah
  106. # [13:41] <MikeSmith> but no idea what the cause is
  107. # [13:42] * Joins: gitbot (~gitbot@public.cloak)
  108. # [13:42] -gitbot:#testing- [web-platform-tests] Ms2ger pushed 1 new commit to master: https://github.com/w3c/web-platform-tests/commit/090cbcfc9265985acd3c36d542085ce87ec803ce
  109. # [13:42] -gitbot:#testing- web-platform-tests/master 090cbcf Ms2ger: Whitelist a lint failure.
  110. # [13:42] * Parts: gitbot (~gitbot@public.cloak)
  111. # [13:42] <Ms2ger> It mentions w3c-test.org
  112. # [13:42] <Ms2ger> We can't have that :)
  113. # [13:42] <MikeSmith> ah that
  114. # [13:43] <MikeSmith> "Refresh: 0; url=http://w3c-test.org/"
  115. # [13:43] <MikeSmith> so what should that url value be instead?
  116. # [13:43] <Ms2ger> No idea, so I whitelisted for now
  117. # [13:44] <MikeSmith> ok
  118. # [13:59] * Joins: gitbot (~gitbot@public.cloak)
  119. # [13:59] -gitbot:#testing- [web-platform-tests] sideshowbarker closed pull request #1808: Add a test for the URL constructor. (master...url-constructor) https://github.com/w3c/web-platform-tests/pull/1808
  120. # [13:59] * Parts: gitbot (~gitbot@public.cloak)
  121. # [14:01] <MikeSmith> I hope Brad understands why we have a linter and isn't going to again ignore linting errors when he commits stuff to trunk
  122. # [14:01] <MikeSmith> maybe he just needs to know how to whitelist things
  123. # [14:03] <MikeSmith> botie, inform bhill2 please whitelist any linter failures that you think are false positives or don't think need to be fixed; see https://github.com/w3c/web-platform-tests/commit/090cbcfc9265985acd3c36d542085ce87ec803ce
  124. # [14:03] <botie> will do
  125. # [14:17] * Joins: gitbot (~gitbot@public.cloak)
  126. # [14:17] -gitbot:#testing- [web-platform-tests] sideshowbarker closed pull request #1805: Add a test for CharacterData#data. (master...CharacterData-data) https://github.com/w3c/web-platform-tests/pull/1805
  127. # [14:17] * Parts: gitbot (~gitbot@public.cloak)
  128. # [14:21] * Joins: gitbot (~gitbot@public.cloak)
  129. # [14:21] -gitbot:#testing- [web-platform-tests] sideshowbarker closed pull request #1847: SRI: s/w3c-test.org/example.test/g (master...sri-w3ctest-example) https://github.com/w3c/web-platform-tests/pull/1847
  130. # [14:21] * Parts: gitbot (~gitbot@public.cloak)
  131. # [14:21] * Joins: gitbot (~gitbot@public.cloak)
  132. # [14:21] -gitbot:#testing- [web-platform-tests] sideshowbarker deleted sri-w3ctest-example at 1261622: https://github.com/w3c/web-platform-tests/commit/1261622
  133. # [14:21] * Parts: gitbot (~gitbot@public.cloak)
  134. # [14:28] <Ms2ger> MikeSmith, <3
  135. # [14:38] * Quits: Ms2ger (~Ms2ger@public.cloak) (Ping timeout: 180 seconds)
  136. # [14:50] * Joins: gitbot (~gitbot@public.cloak)
  137. # [14:50] -gitbot:#testing- [web-platform-tests] mikewest closed pull request #1845: Referrer-Policy: Debug/Release mode generator templates (master...referrer-policy-test-case) https://github.com/w3c/web-platform-tests/pull/1845
  138. # [14:50] * Parts: gitbot (~gitbot@public.cloak)
  139. # [15:07] * Joins: gitbot (~gitbot@public.cloak)
  140. # [15:07] -gitbot:#testing- [web-platform-tests] kristijanburnik opened pull request #1849: Referrer-Policy: Update generated tests for release (master...referrer-policy-test-case) https://github.com/w3c/web-platform-tests/pull/1849
  141. # [15:07] * Parts: gitbot (~gitbot@public.cloak)
  142. # [15:40] * Joins: gitbot (~gitbot@public.cloak)
  143. # [15:40] -gitbot:#testing- [web-platform-tests] sideshowbarker closed pull request #1841: Resolve against URL with credentials (master...credentials-in-base) https://github.com/w3c/web-platform-tests/pull/1841
  144. # [15:40] * Parts: gitbot (~gitbot@public.cloak)
  145. # [16:15] * Quits: playjax (~chatzilla@public.cloak) (Client closed connection)
  146. # [18:50] <jgraham> MikeSmith: Thanks for the review
  147. # [18:51] <jgraham> If that refresh is supposed to do something it should presumably be Refresh: 0; url=http://{{host}}:{{ports[http][0]}}
  148. # [20:02] * Joins: ben (~ben@public.cloak)
  149. # [20:09] * Quits: ben (~ben@public.cloak) (Ping timeout: 180 seconds)
  150. # [20:53] * Joins: playjax (~chatzilla@public.cloak)
  151. # [21:33] * Joins: klusark (~klusark@public.cloak)
  152. # [21:39] * Quits: klusark (~klusark@public.cloak) ("Page closed")
  153. # [21:50] * Joins: scott_gonzalez (~scott_gonzalez@public.cloak)
  154. # [21:56] * Quits: scott_gonzalez (~scott_gonzalez@public.cloak) (scott_gonzalez)
  155. # Session Close: Sun May 24 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