Skip to content

Commit

Permalink
test: clean up comments in test-url-format
Browse files Browse the repository at this point in the history
test-url-format has the max-len rule disabled by a comment but doesn't
have any lines that violate the max-len lint rule. Remove the comment.

Reformat other comments for capitalization, punctuation, and updated
URLs.

PR-URL: #11679
Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Daijiro Wachi <daijiro.wachi@gmail.com>
  • Loading branch information
Trott committed Mar 6, 2017
1 parent a2ae089 commit 60c8115
Showing 1 changed file with 5 additions and 6 deletions.
11 changes: 5 additions & 6 deletions test/parallel/test-url-format.js
Original file line number Diff line number Diff line change
@@ -1,11 +1,10 @@
/* eslint-disable max-len */
'use strict';
require('../common');
const assert = require('assert');
const url = require('url');

// some extra formatting tests, just to verify
// that it'll format slightly wonky content to a valid url.
// Formatting tests to verify that it'll format slightly wonky content to a
// valid URL.
const formatTests = {
'http://example.com?': {
href: 'http://example.com/?',
Expand Down Expand Up @@ -133,7 +132,7 @@ const formatTests = {
protocol: 'dot.test:',
pathname: '/bar'
},
// ipv6 support
// IPv6 support
'coap:u:p@[::1]:61616/.well-known/r?n=Temperature': {
href: 'coap:u:p@[::1]:61616/.well-known/r?n=Temperature',
protocol: 'coap:',
Expand All @@ -150,9 +149,9 @@ const formatTests = {
pathname: '/s/stopButton'
},

// encode context-specific delimiters in path and query, but do not touch
// Encode context-specific delimiters in path and query, but do not touch
// other non-delimiter chars like `%`.
// <https://github.com/joyent/node/issues/4082>
// <https://github.com/nodejs/node-v0.x-archive/issues/4082>

// `#`,`?` in path
'/path/to/%%23%3F+=&.txt?foo=theA1#bar': {
Expand Down

0 comments on commit 60c8115

Please sign in to comment.