Skip to content

Commit 7609b00

Browse files
committed
🧹 chore: Clean up visitPaperPage logs
1 parent e926b4e commit 7609b00

File tree

1 file changed

+7
-6
lines changed

1 file changed

+7
-6
lines changed

test/browser.js

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -50,26 +50,27 @@ export const visitPaperPage = async (browser, target, options = {}) => {
5050
const p = opts.page || (await browser.pages())[0] || (await browser.newPage());
5151
await p.goto(target);
5252
const paperIsStored = new Promise((resolve, reject) => {
53-
let querySelectorTimeout;
53+
let screenshotTimeout;
5454
p.waitForSelector("meta[name='pm-complete-secret-html']").then(() => {
55-
clearTimeout(querySelectorTimeout);
55+
clearTimeout(screenshotTimeout);
5656
resolve();
5757
});
58-
querySelectorTimeout = setTimeout(async () => {
59-
console.log(`\nBackup wait for selector after 5 seconds for ${target}`);
58+
screenshotTimeout = setTimeout(async () => {
6059
const element = await p.evaluate(() => {
6160
return document.querySelector("meta[name='pm-complete-secret-html']");
6261
});
63-
console.log(`Backup element found: ${element}`);
6462
if (!element) {
6563
console.log(`No element found: taking a screenshot`);
6664
if (!fs.existsSync(`${root}/tmp`)) {
6765
console.log(`Creating tmp directory in ${root}/tmp`);
6866
fs.mkdirSync(`${root}/tmp`);
6967
}
70-
const screenshotPath = `${root}/tmp/screenshot_${Date.now()}_${target
68+
let screenshotPath = `${root}/tmp/screenshot_${Date.now()}_${target
7169
.replaceAll("https://", "")
7270
.replaceAll("/", "__")}.jpg`;
71+
screenshotPath = screenshotPath
72+
.replace(/[^a-zA-Z0-9\-_\.]/g, "")
73+
.slice(0, 100);
7374
await p.screenshot({
7475
path: screenshotPath,
7576
fullPage: true,

0 commit comments

Comments
 (0)