Closed marcoscaceres closed 5 years ago
Sure @marcoscaceres I'll do it
Thanks for the help!!
@marcoscaceres While running the test in the browsers using "payment-request/" directory, it made a json file while running under "payment-method-id/" directory it didn't made any json file. So is it what is expected or something unusal?
So, just to confirm, did you copy/paste the file “payment-request/payment-request-ctor-pmi-handling.https.html” file into to the “payment-method-id/” directory? That needs to be in place for the test to run and produce the JSON.
once you move that file, please send a pull request to the web platform test repo... once that lands, the JSON file will get produced when you run the test suite.
@marcoscaceres Yes I did copy/paste it into the payment-method-id directory. Now I need to send a PR to which repo as I'm getting confused with the repos. The changes are made in the wpt repo and the PR must also be send in the same right? Then I'll be testing and generating the JSON file which will go in the test-results repo. Am I going in the right direction?
The changes are made in the wpt repo and the PR must also be send in the same right?
Correct. Apologies for not being clear.
Then I'll be testing and generating the JSON file which will go in the test-results repo. Am I going in the right direction?
Yep đź‘Ť that sounds correct.
Cool I'll send a PR to the wpt repo !
Ok, first item at the top off the list crossed off! ✨
Oh, to test in Firefox, make sure you do this:
@marcoscaceres I did that and updated my PR but the file hasn't changed at all it was the same
I’ll take a look, but unfortunately it won’t be until a bit later today.
@marcoscaceres Yeah no problem! I'll also have a look tomorrow morning as it's already 4 in the morning and I have lectures tomorrow so I'll complete the other tasks tomorrow. :grinning:
Heh, was just thinking it must be super late for you :) I just tried it nonetheless, and the test is showing up. But yes, let’s finish this up tomorrow!
Spotted a typo above, it should have said to run against "/payment-method-id/" (was missing first “/“)
@marcoscaceres We need to update the link to which link exactly... Is it the link to the payment-method-id till I understood?
@yatri1609, so, in this repository, you just need to change: https://github.com/w3c/payment-method-id/blob/gh-pages/index.html#L38
To:
testSuiteURI: "https://w3c-test.org/payment-method-id/",
implementationReportURI: "https://w3c.github.io/test-results/payment-method-id/all.html",
@marcoscaceres I have already done that.
@ianbjacobs, thanks to @yatri1609. we now have an implementation report showing 3 fully conforming implementations 🎉
I'll create a new bug to track moving this to Proposed Recommendation.
@yatri1609 and @marcoscaceres,
Great work, thank you both very much! 🎉🎉🎉
closing as done via #58
The spec is currently missing an implementation report. I've asked @yatri1609 to give us a hand with this! đź‘‹
What we need to do:
[x] copy/paste the test file referenced in the spec into: https://github.com/web-platform-tests/wpt/tree/master/payment-method-id. That way we have an independent test suite.
[x] Verify that everything in the spec is being tested (I believe it is, but want one more check). This involves looking at the source of the test file - and making sure we've checked everything according to the rules of the spec.
[x] Run the tests in at least 2 browsers (e.g., Chrome and Firefox) and generate the report by following the instructions. However, instead of running against the "payment-request/" directory, it would be run against "/payment-method-id/".
[x] Finally, we need to just update the "test-suite" link at the top of the spec to point to the new test suite location. So, we need to change this line: https://github.com/w3c/payment-method-id/blob/gh-pages/index.html#L38