Merge pull request #2 from parro-it/master

added datauri case and test
jprichardson · Aug 27, 2015 · 4afc66b · 4afc66b
2 parents 09b3de8 + 97923f7
commit 4afc66b
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 1 deletion.
diff --git a/lib/__tests__/args.test.js b/lib/__tests__/args.test.js
@@ -35,5 +35,16 @@ describe('args', function () {
         assert.strictEqual(url, 'file:///tmp/index.html')
       })
     })
+
+    describe('> when data uri', function () {
+      it('should create a url with encoded args in hash', function () {
+        var uri = 'data:text/plain;charset=utf-8;base64,dGhpcyBpcyBhIHRlc3QK'
+        var url = args.urlWithArgs(uri, {doThat: 'ok'})
+        assert.strictEqual(url, uri + '#%7B%22doThat%22%3A%22ok%22%7D')
+
+        url = args.urlWithArgs(uri)
+        assert.strictEqual(url, uri)
+      })
+    })
   })
 })
diff --git a/lib/args.js b/lib/args.js
@@ -15,7 +15,7 @@ function urlWithArgs (urlOrFile, args) {
   args = encode(args)
 
   var u
-  if (urlOrFile.indexOf('http') === 0) {
+  if (urlOrFile.indexOf('http') === 0 || urlOrFile.indexOf('data') === 0) {
     var urlData = url.parse(urlOrFile)
     var hash = urlData.hash || args ? args : undefined
     u = url.format(assign(urlData, { hash: hash }))