Skip to content

Commit 864d649

Browse files
dunglasdunglas
committed
Fix prettier config and CS
1 parent 0ac2c88 commit 864d649

11 files changed

Lines changed: 888 additions & 714 deletions

‎.eslintrc.js‎

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -14,9 +14,11 @@ module.exports = {
1414
plugins: ['import', 'flowtype', 'prettier'],
1515

1616
extends: [
17-
"prettier",
18-
"prettier/flowtype",
19-
"prettier/react"
17+
"plugin:prettier/recommended",
2018
],
2119

20+
rules: {
21+
"prettier/prettier": "error"
22+
},
23+
2224
};

‎src/Api.js‎

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,11 @@
11
// @flow
22

3-
import Resource from './Resource';
3+
import Resource from "./Resource";
44

55
type ApiOptions = {
66
title?: string,
7-
resources?: Map<string, Resource>,
8-
}
7+
resources?: Map<string, Resource>
8+
};
99

1010
/**
1111
* @property {string} entrypoint - The URL of the API's entrypoint
@@ -20,12 +20,12 @@ export default class Api {
2020
constructor(entrypoint: string, options: ApiOptions = {}) {
2121
this.entrypoint = entrypoint;
2222

23-
Object.keys(options).forEach((key) => {
23+
Object.keys(options).forEach(key => {
2424
Object.defineProperty(this, key, {
2525
readable: true,
2626
writable: true,
2727
enumerable: true,
28-
value: options[key],
28+
value: options[key]
2929
});
3030
});
3131
}

‎src/Field.js‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -7,8 +7,8 @@ type FieldOptions = {
77
required?: boolean,
88
description?: string,
99
maxCardinality?: number,
10-
deprecated?: boolean,
11-
}
10+
deprecated?: boolean
11+
};
1212

1313
/**
1414
* @property {string} name - The name of this field
@@ -23,12 +23,12 @@ export default class Field {
2323
constructor(name: string, options: FieldOptions = {}) {
2424
this.name = name;
2525

26-
Object.keys(options).forEach((key) => {
26+
Object.keys(options).forEach(key => {
2727
Object.defineProperty(this, key, {
2828
readable: true,
2929
writable: true,
3030
enumerable: true,
31-
value: options[key],
31+
value: options[key]
3232
});
3333
});
3434
}

‎src/Operation.js‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -4,8 +4,8 @@ type OperationOptions = {
44
method?: string,
55
returns?: string,
66
types?: Array<string>,
7-
deprecated?: boolean,
8-
}
7+
deprecated?: boolean
8+
};
99

1010
/**
1111
* @property {string} name - The name of this operation
@@ -20,12 +20,12 @@ export default class Operation {
2020
constructor(name: string, options: OperationOptions = {}) {
2121
this.name = name;
2222

23-
Object.keys(options).forEach((key) => {
23+
Object.keys(options).forEach(key => {
2424
Object.defineProperty(this, key, {
2525
readable: true,
2626
writable: true,
2727
enumerable: true,
28-
value: options[key],
28+
value: options[key]
2929
});
3030
});
3131
}

‎src/Resource.js‎

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,15 +1,15 @@
11
// @flow
22

3-
import Field from './Field';
4-
import Operation from './Operation';
3+
import Field from "./Field";
4+
import Operation from "./Operation";
55

66
type ResourceOptions = {
77
id?: string,
88
title?: string,
99
deprecated?: boolean,
1010
readableFields?: Field[],
1111
writableFields?: Field[],
12-
operations?: Operation[],
12+
operations?: Operation[]
1313
};
1414

1515
/**
@@ -29,12 +29,12 @@ export default class Resource {
2929
this.name = name;
3030
this.url = url;
3131

32-
Object.keys(options).forEach((key) => {
32+
Object.keys(options).forEach(key => {
3333
Object.defineProperty(this, key, {
3434
readable: true,
3535
writable: true,
3636
enumerable: true,
37-
value: options[key],
37+
value: options[key]
3838
});
3939
});
4040
}

‎src/hydra/fetchJsonLd.js‎

Lines changed: 26 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -6,30 +6,39 @@
66
* @return {Promise.<object>} An object with a response key (the original HTTP response) and an optional body key (the parsed JSON-LD body)
77
*/
88
export default function fetchJsonLd(url, options = {}) {
9-
const jsonLdMimeType = 'application/ld+json';
9+
const jsonLdMimeType = "application/ld+json";
1010

11-
if ('undefined' === typeof options.headers) {
11+
if ("undefined" === typeof options.headers) {
1212
options.headers = new Headers();
1313
}
1414

15-
if (null === options.headers.get('Accept')) {
16-
options.headers.set('Accept', jsonLdMimeType);
15+
if (null === options.headers.get("Accept")) {
16+
options.headers.set("Accept", jsonLdMimeType);
1717
}
1818

19-
if ('undefined' !== options.body && !(typeof FormData !== 'undefined' && options.body instanceof FormData) && null === options.headers.get('Content-Type')) {
20-
options.headers.set('Content-Type', jsonLdMimeType);
19+
if (
20+
"undefined" !== options.body &&
21+
!(typeof FormData !== "undefined" && options.body instanceof FormData) &&
22+
null === options.headers.get("Content-Type")
23+
) {
24+
options.headers.set("Content-Type", jsonLdMimeType);
2125
}
2226

23-
return fetch(url, options)
24-
.then(response => {
25-
const { headers, status } = response;
26-
if (204 === status) {
27-
return Promise.resolve({ response });
28-
}
29-
if (500 <= status || !headers.has('Content-Type') || !headers.get('Content-Type').includes(jsonLdMimeType)) {
30-
return Promise.reject({ response });
31-
}
27+
return fetch(url, options).then(response => {
28+
const { headers, status } = response;
29+
if (204 === status) {
30+
return Promise.resolve({ response });
31+
}
32+
if (
33+
500 <= status ||
34+
!headers.has("Content-Type") ||
35+
!headers.get("Content-Type").includes(jsonLdMimeType)
36+
) {
37+
return Promise.reject({ response });
38+
}
3239

33-
return Promise.resolve(response.json().then(body => ({ response, body, document: body })));
34-
});
40+
return Promise.resolve(
41+
response.json().then(body => ({ response, body, document: body }))
42+
);
43+
});
3544
}

‎src/hydra/fetchJsonLd.test.js‎

Lines changed: 43 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -1,52 +1,72 @@
1-
import fetchJsonLd from './fetchJsonLd';
1+
import fetchJsonLd from "./fetchJsonLd";
22

3-
test('fetch a JSON-LD document', () => {
4-
fetch.mockResponseOnce(`{
3+
test("fetch a JSON-LD document", () => {
4+
fetch.mockResponseOnce(
5+
`{
56
"@context": "http://json-ld.org/contexts/person.jsonld",
67
"@id": "http://dbpedia.org/resource/John_Lennon",
78
"name": "John Lennon",
89
"born": "1940-10-09",
910
"spouse": "http://dbpedia.org/resource/Cynthia_Lennon"
10-
}`, {status: 200, statusText: 'OK', headers: new Headers({'Content-Type': 'application/ld+json'})});
11-
12-
return fetchJsonLd('/foo.jsonld').then(data => {
13-
expect(data.response.ok).toBe(true);
14-
expect(data.body.name).toBe('John Lennon');
11+
}`,
12+
{
13+
status: 200,
14+
statusText: "OK",
15+
headers: new Headers({ "Content-Type": "application/ld+json" })
1516
}
1617
);
18+
19+
return fetchJsonLd("/foo.jsonld").then(data => {
20+
expect(data.response.ok).toBe(true);
21+
expect(data.body.name).toBe("John Lennon");
22+
});
1723
});
1824

19-
test('fetch a non JSON-LD document', () => {
20-
fetch.mockResponseOnce(`<body>Hello</body>`, {status: 200, statusText: 'OK', headers: new Headers({'Content-Type': 'text/html'})});
25+
test("fetch a non JSON-LD document", () => {
26+
fetch.mockResponseOnce(`<body>Hello</body>`, {
27+
status: 200,
28+
statusText: "OK",
29+
headers: new Headers({ "Content-Type": "text/html" })
30+
});
2131

22-
return fetchJsonLd('/foo.jsonld').catch(data => {
23-
expect(data.response.ok).toBe(true);
24-
expect(typeof data.body).toBe('undefined');
25-
}
26-
);
32+
return fetchJsonLd("/foo.jsonld").catch(data => {
33+
expect(data.response.ok).toBe(true);
34+
expect(typeof data.body).toBe("undefined");
35+
});
2736
});
2837

29-
test('fetch an error', () => {
30-
fetch.mockResponseOnce(`{
38+
test("fetch an error", () => {
39+
fetch.mockResponseOnce(
40+
`{
3141
"@context": "http://json-ld.org/contexts/person.jsonld",
3242
"@id": "http://dbpedia.org/resource/John_Lennon",
3343
"name": "John Lennon",
3444
"born": "1940-10-09",
3545
"spouse": "http://dbpedia.org/resource/Cynthia_Lennon"
36-
}`, {status: 400, statusText: 'Bad Request', headers: new Headers({'Content-Type': 'application/ld+json'})});
46+
}`,
47+
{
48+
status: 400,
49+
statusText: "Bad Request",
50+
headers: new Headers({ "Content-Type": "application/ld+json" })
51+
}
52+
);
3753

38-
return fetchJsonLd('/foo.jsonld').catch(({response}) => {
54+
return fetchJsonLd("/foo.jsonld").catch(({ response }) => {
3955
response.json().then(body => {
4056
expect(response.ok).toBe(false);
41-
expect(body.born).toBe('1940-10-09');
57+
expect(body.born).toBe("1940-10-09");
4258
});
4359
});
4460
});
4561

46-
test('fetch an empty document', () => {
47-
fetch.mockResponseOnce('', {status: 204, statusText: 'No Content', headers: new Headers({'Content-Type': 'text/html'})});
62+
test("fetch an empty document", () => {
63+
fetch.mockResponseOnce("", {
64+
status: 204,
65+
statusText: "No Content",
66+
headers: new Headers({ "Content-Type": "text/html" })
67+
});
4868

49-
return fetchJsonLd('/foo.jsonld').then(data => {
69+
return fetchJsonLd("/foo.jsonld").then(data => {
5070
expect(data.response.ok).toBe(true);
5171
expect(data.body).toBe(undefined);
5272
});

‎src/hydra/index.js‎

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,2 +1,4 @@
1-
export fetchJsonLd from './fetchJsonLd';
2-
export parseHydraDocumentation, { getDocumentationUrlFromHeaders } from './parseHydraDocumentation';
1+
export fetchJsonLd from "./fetchJsonLd";
2+
export parseHydraDocumentation, {
3+
getDocumentationUrlFromHeaders
4+
} from "./parseHydraDocumentation";

0 commit comments

Comments
 (0)