Benature/omnivore
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

rename jsdom to linkedom

Browse Source
This commit is contained in:
Hongbo Wu
2022-07-14 10:59:44 +08:00
parent 2e645b29b2
commit f0f7aa5a6d
4 changed files with 8 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
packages/readabilityjs/Readability.js
View File

@ -2879,7 +2879,6 @@ Readability.prototype = {
* 4. Replace the current DOM tree with the new one.
* 5. Read peacefully.
*
* @return void
**/
parse: async function() {
// Avoid parsing too large documents, as per configuration option

11
packages/readabilityjs/test/generate-testcase.js
View File

@ -6,7 +6,6 @@ var prettyPrint = require("./utils").prettyPrint;
var htmltidy = require("htmltidy2").tidy;
var { Readability, isProbablyReaderable } = require("../index");
var JSDOMParser = require("../JSDOMParser");
const { generate: generateRandomUA } = require("modern-random-ua/random_ua");
const puppeteer = require('puppeteer');
const { parseHTML } = require("linkedom");
@ -226,12 +225,12 @@ async function runReadability(source, destPath, metadataDestPath) {
var uri = "http://fakehost/test/page.html";
var myReader, result, readerable;
try {
// Use jsdom for isProbablyReaderable because it supports querySelectorAll
var jsdom = parseHTML(source).document;
readerable = isProbablyReaderable(jsdom);
// Use linkedom for isProbablyReaderable because it supports querySelectorAll
var dom = parseHTML(source).document;
readerable = isProbablyReaderable(dom);
// We pass `caption` as a class to check that passing in extra classes works,
// given that it appears in some of the test documents.
myReader = new Readability(jsdom, { classesToPreserve: ["caption"], url: uri });
myReader = new Readability(dom, { classesToPreserve: ["caption"], url: uri });
result = await myReader.parse();
} catch (ex) {
console.error(ex);
@ -274,7 +273,7 @@ if (process.argv.length < 3) {
if (process.argv[2] === "all") {
fs.readdir(testcaseRoot, function (err, files) {
if (err) {
console.error("error reading testcaseses");
console.error("error reading testcases");
return;
}

2
packages/readabilityjs/test/test-jsdomparser.js → packages/readabilityjs/test/test-linkedomparser.js
View File

@ -10,7 +10,7 @@ var BASETESTCASE = '<html><body><p>Some text and <a class="someclass" href="#">a
var baseDoc = new JSDOMParser().parse(BASETESTCASE, "http://fakehost/");
describe("Test JSDOM functionality", function() {
describe("Test linkedom functionality", function() {
function nodeExpect(actual, expected) {
try {
expect(actual).eql(expected);

4
packages/readabilityjs/test/test-readability.js
View File

@ -326,8 +326,8 @@ describe("Test pages", function() {
describe(testPage.dir, function() {
var uri = "http://fakehost/test/page.html";
runTestsWithItems("jsdom", function(source) {
var doc =parseHTML(source).document;
runTestsWithItems("linkedom", function(source) {
var doc = parseHTML(source).document;
removeCommentNodesRecursively(doc);
return doc;
}, testPage.source, testPage.expectedContent, testPage.expectedMetadata, uri);