Skip to content

Conversation

gjulivan
Copy link
Collaborator

Pull request type


Description

@Copilot Copilot AI review requested due to automatic review settings September 10, 2025 15:09
@gjulivan gjulivan requested a review from a team as a code owner September 10, 2025 15:09
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 prepares the document viewer widget for release by adding essential testing infrastructure and release scripts. The changes align with standard Mendix widget development workflows by implementing E2E testing capabilities and ensuring proper release tooling.

  • Adds comprehensive E2E test suite with Playwright to verify document viewer functionality across different file types
  • Updates package.json with release-ready scripts and configurations for testing, publishing, and changelog management
  • Establishes test project configuration linking to the appropriate GitHub test repository

Reviewed Changes

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

File Description
packages/pluggableWidgets/document-viewer-web/package.json Adds E2E testing scripts, release tooling, test project configuration, and updates test command from placeholder to actual unit tests
packages/pluggableWidgets/document-viewer-web/e2e/DocumentViewer.spec.js Creates comprehensive E2E test suite covering document types (PDF, Word, Excel, images, text), viewer controls, and error handling scenarios
Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

@gjulivan gjulivan force-pushed the doc-viewer-release branch 2 times, most recently from b56b062 to 66909ad Compare September 11, 2025 07:21
@gjulivan gjulivan enabled auto-merge (rebase) September 11, 2025 07:35
@gjulivan gjulivan merged commit f947703 into main Sep 11, 2025
14 checks passed
@gjulivan gjulivan deleted the doc-viewer-release branch September 11, 2025 07:39
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.

3 participants