Archives/mercury-parser
2
0
Fork 0
mirror of https://github.com/postlight/mercury-parser synced 2024-11-17 03:25:31 +00:00
Code Issues Projects Releases Wiki Activity

fix: title wasn't cleaning html tags

Browse Source
This commit is contained in:
Adam Pash 2016-09-01 13:45:00 -04:00
parent 956fd678f7
commit d3b791d516
2 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
src/extractor/generic/title/utils/clean-title.js
View File

@ -18,5 +18,6 @@ export default function cleanTitle(title, url, $) {
} }
} }
return title.trim() // strip any html tags in the title text
return stripTags(title, $).trim()
} }

2
src/extractor/generic/title/utils/clean-title.test.js
View File

@ -20,8 +20,8 @@ describe('cleanTitle(title, $)', () => {
}) })
it('removes HTML tags from titles', () => { it('removes HTML tags from titles', () => {
const title = "Too Short"
const $ = cheerio.load(HTML.docWithTagsInH1.before) const $ = cheerio.load(HTML.docWithTagsInH1.before)
const title = $('h1').html()
assert.equal(cleanTitle(title, '', $), HTML.docWithTagsInH1.after) assert.equal(cleanTitle(title, '', $), HTML.docWithTagsInH1.after)
}) })