Skip to content
This repository was archived by the owner on Jan 28, 2026. It is now read-only.

Screenshot feature#23

Closed
amarilda611 wants to merge 47 commits intomainfrom
screenshot-feature
Closed

Screenshot feature#23
amarilda611 wants to merge 47 commits intomainfrom
screenshot-feature

Conversation

@amarilda611
Copy link
Copy Markdown
Contributor

No description provided.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 29, 2025

Playwright Test Metrics

Total: 4
Passed: 2
Failed: 0
Skipped: 0

Duration: 4185962 ms
Pass Rate: ** %**

ESLint (GUI tests)


Test-Flow Chart

Artifact: test-flow-chart → flowchart.png

Full run details: link

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 29, 2025

🖼️ Automated UI Preview

about.jpg\n\nhome.jpg\n\n

Copy link
Copy Markdown
Contributor

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

prettier

[prettier] reported by reviewdog 🐶


[prettier] reported by reviewdog 🐶

state: 'visible',
timeout: 15000 // 15s per selector

@github-actions
Copy link
Copy Markdown
Contributor

❌ GUI Screenshot Capture Failed

Screenshot capture failed during execution. Debugging information:

Common fixes:

  1. Verify application URL is accessible from GitHub's servers
  2. Check element selectors in screenshot.js
  3. Increase timeouts if application is slow to load

Comment thread scripts/screenshot.js
@@ -0,0 +1,36 @@
const { chromium } = require('playwright');
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[prettier] reported by reviewdog 🐶

Suggested change
const { chromium } = require('playwright');
const { chromium } = require("playwright");

Comment thread scripts/screenshot.js
Comment on lines +7 to +9
browser = await chromium.launch({
channel: 'chromium',
headless: true
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[prettier] reported by reviewdog 🐶

Suggested change
browser = await chromium.launch({
channel: 'chromium',
headless: true
browser = await chromium.launch({
channel: "chromium",
headless: true,

Comment thread scripts/screenshot.js
channel: 'chromium',
headless: true
});

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[prettier] reported by reviewdog 🐶

Suggested change

Comment thread scripts/screenshot.js

const page = await browser.newPage();
await page.setViewportSize({ width: 800, height: 600 });

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[prettier] reported by reviewdog 🐶

Suggested change

Comment thread scripts/screenshot.js
Comment on lines +16 to +18
await page.goto('https://example.com', {
waitUntil: 'networkidle',
timeout: 60000
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[prettier] reported by reviewdog 🐶

Suggested change
await page.goto('https://example.com', {
waitUntil: 'networkidle',
timeout: 60000
await page.goto("https://example.com", {
waitUntil: "networkidle",
timeout: 60000,

Comment thread scripts/screenshot.js
Comment on lines +20 to +23
await page.screenshot({
path: 'screenshots/home.jpg',
type: 'jpeg',
quality: 70
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[prettier] reported by reviewdog 🐶

Suggested change
await page.screenshot({
path: 'screenshots/home.jpg',
type: 'jpeg',
quality: 70
await page.screenshot({
path: "screenshots/home.jpg",
type: "jpeg",
quality: 70,

Comment thread scripts/screenshot.js
type: 'jpeg',
quality: 70
});

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[prettier] reported by reviewdog 🐶

Suggested change

Comment thread scripts/screenshot.js
Comment on lines +27 to +29
await page.goto('https://example.com/about', {
waitUntil: 'networkidle',
timeout: 60000
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[prettier] reported by reviewdog 🐶

Suggested change
await page.goto('https://example.com/about', {
waitUntil: 'networkidle',
timeout: 60000
await page.goto("https://example.com/about", {
waitUntil: "networkidle",
timeout: 60000,

Comment thread scripts/screenshot.js
Comment on lines +31 to +34
await page.screenshot({
path: 'screenshots/about.jpg',
type: 'jpeg',
quality: 70
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[prettier] reported by reviewdog 🐶

Suggested change
await page.screenshot({
path: 'screenshots/about.jpg',
type: 'jpeg',
quality: 70
await page.screenshot({
path: "screenshots/about.jpg",
type: "jpeg",
quality: 70,

Comment thread scripts/screenshot.js
Comment on lines +36 to +37

console.log('✅ Screenshots captured successfully!');
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[prettier] reported by reviewdog 🐶

Suggested change
console.log('✅ Screenshots captured successfully!');
console.log("✅ Screenshots captured successfully!");

Comment thread scripts/screenshot.js

console.log('✅ Screenshots captured successfully!');
} catch (error) {
console.error('❌ Screenshot failed:', error);
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[prettier] reported by reviewdog 🐶

Suggested change
console.error('❌ Screenshot failed:', error);
console.error("❌ Screenshot failed:", error);

@amarilda611
Copy link
Copy Markdown
Contributor Author

errors

@amarilda611 amarilda611 deleted the screenshot-feature branch May 29, 2025 21:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant