Skip to content

cad reporting browserstack-report action #61

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 69 commits into
base: master
Choose a base branch
from

Conversation

shandli123
Copy link
Collaborator

@shandli123 shandli123 commented May 13, 2025

@shandli123 shandli123 requested a review from a team as a code owner May 13, 2025 16:05
@shandli123 shandli123 requested a review from Copilot May 15, 2025 09:55
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds a new GitHub Action for fetching and displaying BrowserStack reports, including modules for input validation, report retrieval and processing, and test coverage for these functionalities.

  • Added unit tests for input validation and action input handling.
  • Introduced utilities for file upload and timeout management and services for report fetching and processing.
  • Updated GitHub Action metadata and configuration files (action.yml, package.json, constants) to support the new functionality.

Reviewed Changes

Copilot reviewed 17 out of 17 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
test/actionInput/inputValidator.test.js Unit tests covering input validation for build info generation and timeout validation.
test/actionInput/index.test.js Tests for validating action inputs and error scenarios.
src/utils/UploadFileForArtifact.js Utility to create an artifact directory and save report HTML files.
src/utils/TimeoutManager.js Utility to track timeouts for long-running operations.
src/services/ReportService.js Service module implementing report fetching with polling and error handling.
src/services/ReportProcessor.js Service module for processing reports into GitHub Actions summary and uploading artifacts.
src/main.js The main entrypoint coordinating input fetching, report polling, and processing.
src/actionInput/inputValidator.js Module for validating and producing build info from GitHub event metadata.
src/actionInput/index.js Combines input fetching and validation logic for the action.
package.json Updated project configuration for dependencies, build, and test scripts.
config/constants.js Centralized configuration constants used across the action.
action.yml GitHub Action metadata file with input definitions and runtime configuration.
README.md, .gitignore, .eslintrc.js Documentation and configuration files for project consistency.

@Amark19
Copy link

Amark19 commented May 19, 2025

LGTM !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants