Skip to content

PAINTROID-753 Write Integration tests for WorkspacePage menu#125

Merged
juliajulie95 merged 2 commits intoCatrobat:developfrom
Amit-Matth:PAINTROID-753
Aug 21, 2025
Merged

PAINTROID-753 Write Integration tests for WorkspacePage menu#125
juliajulie95 merged 2 commits intoCatrobat:developfrom
Amit-Matth:PAINTROID-753

Conversation

@Amit-Matth
Copy link
Copy Markdown
Contributor

@Amit-Matth Amit-Matth commented Jul 18, 2025

This PR introduces integration tests covering core navigation and overflow menu actions in WorkspacePage, ensuring reliable user interactions during back navigation, fullscreen mode, and canvas management.

PAINTROID-753

New Features and Enhancements

  • Added integration tests for WorkspacePage features:
    • Overflow menu options visibility
    • "New Image" workflow with discard/save handling
    • Fullscreen toggle with back navigation
    • Unsaved changes discard dialog behavior

Refactorings and Bug Fixes

  • Added two new localization strings:
  • "Save" – for confirming saving work
  • "Discard" – for confirming discarding changes

Checklist

Your checklist for this pull request

Please review the contributing guidelines and wiki pages of this repository.

  • Include the name of the Jira ticket in the PR’s title
  • Add the link to the ticket in Jira in the description of the PR
  • Include a summary of the changes plus the relevant context
  • Choose the proper base branch (develop)
  • Confirm that the changes follow the project’s coding guidelines (Wiki)
  • Verify that the changes generate no compiler or linter warnings
  • Perform a self-review of the changes
  • Verify to commit no other files than the intentionally changed ones
  • Include reasonable and readable tests verifying the added or changed behavior
  • Confirm that new and existing tests pass locally
  • Check that the commits’ message style matches the project’s guideline
  • Verify that your changes do not have any conflicts with the base branch
  • After the PR, verify that all CI checks have passed
  • Add new information to the Wiki

@juliajulie95 juliajulie95 merged commit 9b6867d into Catrobat:develop Aug 21, 2025
1 check passed
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