-
Notifications
You must be signed in to change notification settings - Fork 3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ISO 2022-jp encoding and decoding tests #8071
Conversation
added timeout meta removed tabs and line-end spaces removed extraneous text pointed to /common/blank.html
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's get 'em in!
@@ -0,0 +1 @@ | |||
Content-Type: text/html; charset=csiso2022jp |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you add a trailing newline here and the other .headers files?
It looks like that was done for the shift_jis tests, and makes cat
ing the files more pleasant
} | ||
} | ||
|
||
function encode(input, expected, desc) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Used in 5 test files; would be nice to factor it out but we can do that later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah I think there's a general project to be done where you diff files in these encoding tests against each other and factor out large chunks, or use the .sub.html infrastructure to just make them into one test and then iframe N copies of them.
Build PASSEDStarted: 2017-11-06 04:02:25 Failing Jobs
View more information about this build on: |
I was inspired to do another round of shepherding @r12a's tests through the process. This is an updated version of #3199 with fixes to make lint pass and JavaScript prettification.
It reportedly passes in Firefox 57, per #3199 (comment), so probably @hsivonen is happy with them at least, and we should merge them. Maybe @hsivonen or @inexorabletash can approve this so that, once CI passes, I can merge them.
Closes #3199.