Fixes encoding of special characters.

Solution was taken from existing PR in original spraypaint repository:
graphiti-api#104

Author: mcarreira-esab

src/util/parameterize.ts

@@ -34,15 +34,9 @@ const parameterize = (obj: any, prefix?: string): string => {
   return str.join("&")
 }
 
-// IE does not encode by default like other browsers
 const maybeEncode = (value: string): string => {
-  var isBrowser =
-    typeof window !== "undefined" &&
-    typeof window.navigator.userAgent !== "undefined"
-  const isIE = isBrowser && window.navigator.userAgent.match(/(MSIE|Trident)/)
-  const isEncoded = typeof value === "string" && value.indexOf("%") !== -1
-  const shouldEncode = isBrowser && isIE && !isEncoded
-  return shouldEncode ? encodeURIComponent(value) : value
+  var isEncoded = typeof value === "string" && decodeURI(value) !== value
+  return isEncoded ? value : encodeURIComponent(value)
 }
 
 export { parameterize as default }

test/integration/finders.test.ts

@@ -574,7 +574,7 @@ describe("Model finders", () => {
 
   describe("#includes", () => {
     beforeEach(() => {
-      fetchMock.get("http://example.com/api/v1/people?include=a.b,a.c.d", {
+      fetchMock.get("http://example.com/api/v1/people?include=a.b%2Ca.c.d", {
         data: [
           {
             id: "2",

test/integration/relations.test.ts

@@ -59,7 +59,7 @@ describe("Relations", () => {
   describe("#find()", () => {
     beforeEach(() => {
       fetchMock.get(
-        "http://example.com/api/v1/authors/1?include=books,multi_words",
+        "http://example.com/api/v1/authors/1?include=books%2Cmulti_words",
         generateMockResponse("authors")
       )
     })
@@ -103,7 +103,7 @@ describe("Relations", () => {
   describe("when keyCase is snake_case", () => {
     beforeEach(() => {
       fetchMock.get(
-        "http://example.com/api/v1/non_fiction_authors/1?include=books,multi_words",
+        "http://example.com/api/v1/non_fiction_authors/1?include=books%2Cmulti_words",
         generateMockResponse("non_fiction_authors")
       )
     })

test/unit/scope.test.ts

@@ -342,7 +342,7 @@ describe("Scope", () => {
         .stats({ total: "count" })
         .includes({ a: ["b", { c: "d" }] })
       expect(scope.toQueryParams()).to.eq(
-        "page[number]=2&page[size]=10&filter[foo_bar]=baz&sort=foo,-bar&fields[people]=name,age&stats[total]=count&include=a.b,a.c.d"
+        "page[number]=2&page[size]=10&filter[foo_bar]=baz&sort=foo,-bar&fields[people]=name,age&stats[total]=count&include=a.b%2Ca.c.d"
       )
     })
 
@@ -376,7 +376,7 @@ describe("Scope", () => {
       })
 
       it("casts arrays correctly", () => {
-        scope = scope.extraParams({ foo: "bar,baz" })
+        scope = scope.extraParams({ foo: ["bar", "baz"] })
         expect(<string>scope.toQueryParams()).to.eq("foo=bar,baz")
       })