Added logging to test output.

pull/23/head
Nicolas Perriault 9 years ago
parent cfe299e878
commit 99f338a03a

@ -18,7 +18,7 @@
* This code is heavily based on Arc90's readability.js (1.7.1) script
* available at: http://code.google.com/p/arc90labs-readability
*/
var root = this;
var Readability = function(uri, doc) {
var ENABLE_LOGGING = false;
@ -46,8 +46,13 @@ var Readability = function(uri, doc) {
// Control whether log messages are sent to the console
if (ENABLE_LOGGING) {
var logger;
this.log = function (msg) {
dump("Reader: (Readability) " + msg + "\n");
if ("dump" in root) {
dump("Reader: (Readability) " + msg + "\n");
} else if ("console" in root) {
console.log("Reader: (Readability) " + msg + "\n");
}
};
} else {
this.log = function () {};

@ -21,6 +21,6 @@
"chai-as-promised": "^4.3.*",
"html": "0.0.*",
"mocha": "^2.2.*",
"readable-proxy": "1.2.*"
"readable-proxy": "1.3.*"
}
}

@ -30,6 +30,14 @@ describe("Test page", function() {
return scrape("file://" + testPage.source).catch(function(err) {
throw err;
}).then(function(result) {
// print Readability log messages
(result.consoleLogs || [])
.filter(function(logMessage) {
return logMessage.indexOf("Reader: (Readability)") === 0;
})
.forEach(function(logMessage) {
console.log("[LOG]", logMessage);
});
// normalize html
return prettyPrint(result.content);
}).should.eventually.become(prettyPrint(expected));

Loading…
Cancel
Save