Skip to content

Commit ef3c6ed

Browse files
committed
fix: linter
1 parent 7fb353b commit ef3c6ed

File tree

4 files changed

+10
-10
lines changed

4 files changed

+10
-10
lines changed

packages/metascraper-audio/test/index.js

+5-5
Original file line numberDiff line numberDiff line change
@@ -6,35 +6,35 @@ const metascraper = require('metascraper')([require('metascraper-audio')()])
66

77
describe('metascraper-audio', () => {
88
it('og:audio', async () => {
9-
const html = `<meta property="og:audio" content="https://browserless.js.org/static/demo.mp3">`
9+
const html = '<meta property="og:audio" content="https://browserless.js.org/static/demo.mp3">'
1010
const url = 'https://browserless.js.org'
1111
const metadata = await metascraper({ html, url })
1212
snapshot(metadata)
1313
})
1414

1515
it('og:audio:secure_url', async () => {
16-
const html = `<meta property="og:audio:secure_url" content="https://browserless.js.org/static/demo.mp3">`
16+
const html = '<meta property="og:audio:secure_url" content="https://browserless.js.org/static/demo.mp3">'
1717
const url = 'https://browserless.js.org'
1818
const metadata = await metascraper({ html, url })
1919
snapshot(metadata)
2020
})
2121

2222
it('twitter:player:stream', async () => {
23-
const html = `<meta property="twitter:player:stream" content="https://browserless.js.org/static/demo.mp3">`
23+
const html = '<meta property="twitter:player:stream" content="https://browserless.js.org/static/demo.mp3">'
2424
const url = 'https://browserless.js.org'
2525
const metadata = await metascraper({ html, url })
2626
snapshot(metadata)
2727
})
2828

2929
it('audio:src', async () => {
30-
const html = `<audio src="https://browserless.js.org/static/demo.mp3">`
30+
const html = '<audio src="https://browserless.js.org/static/demo.mp3">'
3131
const url = 'https://browserless.js.org'
3232
const metadata = await metascraper({ html, url })
3333
snapshot(metadata)
3434
})
3535

3636
it('audio:source:src', async () => {
37-
const html = `<audio><source src="https://browserless.js.org/static/demo.mp3"></source></audio>`
37+
const html = '<audio><source src="https://browserless.js.org/static/demo.mp3"></source></audio>'
3838
const url = 'https://browserless.js.org'
3939
const metadata = await metascraper({ html, url })
4040
snapshot(metadata)

packages/metascraper-helpers/test/index.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -192,15 +192,15 @@ describe('metascraper-helpers', () => {
192192
const url =
193193
'https://www.theverge.com/2017/11/16/16667366/tesla-semi-truck-announced-price-release-date-electric-self-driving'
194194
const $ = cheerio.load(
195-
`<script type="application/ld+json">{"@context":"http://schema.org","@type":"NewsArticle","mainEntityOfPage":"https://www.theverge.com/2017/11/16/16667366/tesla-semi-truck-announced-price-release-date-electric-self-driving","headline":"This is the Tesla Semi truck","description":"500 miles of range and more aerodynamic than a supercar","speakable":{"@type":"SpeakableSpecification","xpath":["/html/head/title","/html/head/meta[@name='description']/@content"]},"datePublished":"2017-11-16T23:47:07-05:00","dateModified":"2017-11-16T23:47:07-05:00","author":{"@type":"Person","name":"Zac Estrada"},"publisher":{"@type":"Organization","name":"The Verge","logo":{"@type":"ImageObject","url":"https://cdn.vox-cdn.com/uploads/chorus_asset/file/13668586/google_amp.0.png","width":600,"height":60}},"about":{"@type":"Event","name":"Tesla Semi Truck Event 2017","startDate":"2017-11-17T04:00:00+00:00","location":{"@type":"Place","name":"Tesla Motors factory","address":"Hawthorne, California, USA"}},"image":[{"@type":"ImageObject","url":"https://cdn.vox-cdn.com/thumbor/k8ssXKPAuRwxa1pKew982ZMgv0o=/1400x1400/filters:format(jpeg)/cdn.vox-cdn.com/uploads/chorus_asset/file/9699573/Semi_Front_Profile.jpg","width":1400,"height":1400},{"@type":"ImageObject","url":"https://cdn.vox-cdn.com/thumbor/l6nkV8CkJIdUrJIzHFWUFc1zLRM=/1400x1050/filters:format(jpeg)/cdn.vox-cdn.com/uploads/chorus_asset/file/9699573/Semi_Front_Profile.jpg","width":1400,"height":1050},{"@type":"ImageObject","url":"https://cdn.vox-cdn.com/thumbor/5Sqo6J73lBi1hwzEiKCQy6FLx3I=/1400x788/filters:format(jpeg)/cdn.vox-cdn.com/uploads/chorus_asset/file/9699573/Semi_Front_Profile.jpg","width":1400,"height":788}]}</script>`
195+
'<script type="application/ld+json">{"@context":"http://schema.org","@type":"NewsArticle","mainEntityOfPage":"https://www.theverge.com/2017/11/16/16667366/tesla-semi-truck-announced-price-release-date-electric-self-driving","headline":"This is the Tesla Semi truck","description":"500 miles of range and more aerodynamic than a supercar","speakable":{"@type":"SpeakableSpecification","xpath":["/html/head/title","/html/head/meta[@name=\'description\']/@content"]},"datePublished":"2017-11-16T23:47:07-05:00","dateModified":"2017-11-16T23:47:07-05:00","author":{"@type":"Person","name":"Zac Estrada"},"publisher":{"@type":"Organization","name":"The Verge","logo":{"@type":"ImageObject","url":"https://cdn.vox-cdn.com/uploads/chorus_asset/file/13668586/google_amp.0.png","width":600,"height":60}},"about":{"@type":"Event","name":"Tesla Semi Truck Event 2017","startDate":"2017-11-17T04:00:00+00:00","location":{"@type":"Place","name":"Tesla Motors factory","address":"Hawthorne, California, USA"}},"image":[{"@type":"ImageObject","url":"https://cdn.vox-cdn.com/thumbor/k8ssXKPAuRwxa1pKew982ZMgv0o=/1400x1400/filters:format(jpeg)/cdn.vox-cdn.com/uploads/chorus_asset/file/9699573/Semi_Front_Profile.jpg","width":1400,"height":1400},{"@type":"ImageObject","url":"https://cdn.vox-cdn.com/thumbor/l6nkV8CkJIdUrJIzHFWUFc1zLRM=/1400x1050/filters:format(jpeg)/cdn.vox-cdn.com/uploads/chorus_asset/file/9699573/Semi_Front_Profile.jpg","width":1400,"height":1050},{"@type":"ImageObject","url":"https://cdn.vox-cdn.com/thumbor/5Sqo6J73lBi1hwzEiKCQy6FLx3I=/1400x788/filters:format(jpeg)/cdn.vox-cdn.com/uploads/chorus_asset/file/9699573/Semi_Front_Profile.jpg","width":1400,"height":788}]}</script>'
196196
)
197197
snapshot(jsonld(url, $))
198198
})
199199
it('ensure to return first value', () => {
200200
const url =
201201
'https://www.theverge.com/2017/11/16/16667366/tesla-semi-truck-announced-price-release-date-electric-self-driving'
202202
const $ = cheerio.load(
203-
`<script type="application/ld+json">[{"@context":"http://schema.org","@type":"NewsArticle","mainEntityOfPage":"https://www.theverge.com/2017/11/16/16667366/tesla-semi-truck-announced-price-release-date-electric-self-driving","headline":"This is the Tesla Semi truck","description":"500 miles of range and more aerodynamic than a supercar","speakable":{"@type":"SpeakableSpecification","xpath":["/html/head/title","/html/head/meta[@name='description']/@content"]},"datePublished":"2017-11-16T23:47:07-05:00","dateModified":"2017-11-16T23:47:07-05:00","author":{"@type":"Person","name":"Zac Estrada"},"publisher":{"@type":"Organization","name":"The Verge","logo":{"@type":"ImageObject","url":"https://cdn.vox-cdn.com/uploads/chorus_asset/file/13668586/google_amp.0.png","width":600,"height":60}},"about":{"@type":"Event","name":"Tesla Semi Truck Event 2017","startDate":"2017-11-17T04:00:00+00:00","location":{"@type":"Place","name":"Tesla Motors factory","address":"Hawthorne, California, USA"}},"image":[{"@type":"ImageObject","url":"https://cdn.vox-cdn.com/thumbor/k8ssXKPAuRwxa1pKew982ZMgv0o=/1400x1400/filters:format(jpeg)/cdn.vox-cdn.com/uploads/chorus_asset/file/9699573/Semi_Front_Profile.jpg","width":1400,"height":1400},{"@type":"ImageObject","url":"https://cdn.vox-cdn.com/thumbor/l6nkV8CkJIdUrJIzHFWUFc1zLRM=/1400x1050/filters:format(jpeg)/cdn.vox-cdn.com/uploads/chorus_asset/file/9699573/Semi_Front_Profile.jpg","width":1400,"height":1050},{"@type":"ImageObject","url":"https://cdn.vox-cdn.com/thumbor/5Sqo6J73lBi1hwzEiKCQy6FLx3I=/1400x788/filters:format(jpeg)/cdn.vox-cdn.com/uploads/chorus_asset/file/9699573/Semi_Front_Profile.jpg","width":1400,"height":788}]}]</script>`
203+
'<script type="application/ld+json">[{"@context":"http://schema.org","@type":"NewsArticle","mainEntityOfPage":"https://www.theverge.com/2017/11/16/16667366/tesla-semi-truck-announced-price-release-date-electric-self-driving","headline":"This is the Tesla Semi truck","description":"500 miles of range and more aerodynamic than a supercar","speakable":{"@type":"SpeakableSpecification","xpath":["/html/head/title","/html/head/meta[@name=\'description\']/@content"]},"datePublished":"2017-11-16T23:47:07-05:00","dateModified":"2017-11-16T23:47:07-05:00","author":{"@type":"Person","name":"Zac Estrada"},"publisher":{"@type":"Organization","name":"The Verge","logo":{"@type":"ImageObject","url":"https://cdn.vox-cdn.com/uploads/chorus_asset/file/13668586/google_amp.0.png","width":600,"height":60}},"about":{"@type":"Event","name":"Tesla Semi Truck Event 2017","startDate":"2017-11-17T04:00:00+00:00","location":{"@type":"Place","name":"Tesla Motors factory","address":"Hawthorne, California, USA"}},"image":[{"@type":"ImageObject","url":"https://cdn.vox-cdn.com/thumbor/k8ssXKPAuRwxa1pKew982ZMgv0o=/1400x1400/filters:format(jpeg)/cdn.vox-cdn.com/uploads/chorus_asset/file/9699573/Semi_Front_Profile.jpg","width":1400,"height":1400},{"@type":"ImageObject","url":"https://cdn.vox-cdn.com/thumbor/l6nkV8CkJIdUrJIzHFWUFc1zLRM=/1400x1050/filters:format(jpeg)/cdn.vox-cdn.com/uploads/chorus_asset/file/9699573/Semi_Front_Profile.jpg","width":1400,"height":1050},{"@type":"ImageObject","url":"https://cdn.vox-cdn.com/thumbor/5Sqo6J73lBi1hwzEiKCQy6FLx3I=/1400x788/filters:format(jpeg)/cdn.vox-cdn.com/uploads/chorus_asset/file/9699573/Semi_Front_Profile.jpg","width":1400,"height":788}]}]</script>'
204204
)
205205
const json = jsonld(url, $)
206206
should(typeof json).be.equal('object')

packages/metascraper-media-provider/src/get-media/provider/generic.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,7 @@ module.exports = ({ tunnel, onError, userAgent, cacheDir }) => {
6868
data = await getInfo(url, flags)
6969
} catch (rawError) {
7070
const err = makeError({ rawError, url, flags })
71-
debug(`getInfo:err`, err.message)
71+
debug('getInfo:err', err.message)
7272
retry.incr()
7373
onError(err, url)
7474
if (!(tunnel && isTwitterRateLimit(url, err))) return data

packages/metascraper-media-provider/src/get-media/provider/twitter.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ const createGuestToken = ({ userAgent, tunnel }) => {
4343
)
4444
token = get(body, 'guest_token')
4545
} catch (err) {
46-
debug(`guestToken:err`, err.message)
46+
debug('guestToken:err', err.message)
4747
retry.incr()
4848
}
4949
} while (!token)
@@ -98,7 +98,7 @@ const createGetTwitterVideo = ({ userAgent, getGuestToken }) => {
9898
}
9999
} catch (err) {
100100
guestToken = getGuestToken()
101-
debug(`getTwitterInfo rotating token`)
101+
debug('getTwitterInfo rotating token')
102102
}
103103
} while (!data)
104104

0 commit comments

Comments
 (0)