test/url: Add unit tests for Plus Sign handling

See Mikhus#49
besfahbod · Mar 19, 2020 · b3803c5 · b3803c5
1 parent e28a567
commit b3803c5
Showing 1 changed file with 10 additions and 2 deletions.
diff --git a/test/url.js b/test/url.js
@@ -161,8 +161,16 @@ describe('Url props interface', function () {
 });
 
 describe('Path url encoding', function () {
-    it('should correctly encode whitespace with %20', function () {
+    it('should correctly encode whitespace as %20', function () {
         const u = new Url('http://localhost/path with space');
-        assert.equal(u.toString().toLowerCase(),'http://localhost/path%20with%20space');
+        assert.equal(u.toString(),'http://localhost/path%20with%20space');
+    // TODO: Fix https://github.com/Mikhus/domurl/issues/49
+    xit('should correctly encode Plus Sign (+) to %2b in path.', function () {
+        const u = new Url('http://localhost/path+with+plus');
+        assert.equal(u.toString(), 'http://localhost/path%2bwith%2bplus');
+    });
+    xit('should preserve Plus Sign (+) in path.', function () {
+        const u = new Url('http://localhost/path+with+plus');
+        assert.equal(u.toString(), 'http://localhost/path%2bwith%2bplus');
     });
 });