Skip to content

Playwright script for TC29, 105 and 113#692

Merged
maharrshi merged 4 commits intodevelopfrom
test/automate/project-filters
Jul 23, 2025
Merged

Playwright script for TC29, 105 and 113#692
maharrshi merged 4 commits intodevelopfrom
test/automate/project-filters

Conversation

@maharrshi
Copy link
Copy Markdown
Collaborator

@maharrshi maharrshi commented Jul 22, 2025

Description

Relevant Technical Choices

Testing Instructions

Additional Information:

Screenshot/Screencast

Checklist

  • I have carefully reviewed the code before submitting it for review.
  • This code is adequately covered by unit tests to validate its functionality.
  • I have conducted thorough testing to ensure it functions as intended.
  • A member of the QA team has reviewed and tested this PR (To be checked by QA or code reviewer)

Fixes #

@maharrshi maharrshi requested a review from Copilot July 22, 2025 14:12
Copy link
Copy Markdown
Contributor

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 implements two test cases (TC105 and TC113) for the project management module. TC105 tests the creation and deletion of private views, while TC113 tests multiple filter functionality with both positive and negative scenarios.

  • Enhanced error handling in API requests with detailed error body logging
  • Added comprehensive filter functionality for project listings
  • Implemented private view creation and deletion capabilities
  • Updated test data structure to support multiple project payloads per test case

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 7 comments.

Show a summary per file
File Description
tests/e2e/utils/api/projectRequests.js Enhanced error handling with detailed error body logging for failed API requests
tests/e2e/specs/manager/project.spec.js Added two new test cases (TC105, TC113) for private view management and project filtering
tests/e2e/pageObjects/projectPage.js Extended page object with filter controls, view management, and utility methods
tests/e2e/helpers/timesheetHelper.js Refactored to support multiple project payloads per test case with improved error handling
tests/e2e/data/manager/team.js Updated employee list for TC53 test case
tests/e2e/data/manager/project.js Added test data for TC105 and TC113 with multiple project configurations

maharrshi and others added 3 commits July 22, 2025 19:48
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@maharrshi maharrshi marked this pull request as ready for review July 23, 2025 06:25
@maharrshi maharrshi requested a review from jayson31 July 23, 2025 06:26
@maharrshi maharrshi changed the title test cases: TC105, TC113 Playwright script for TC29, 105 and 113 Jul 23, 2025
Copy link
Copy Markdown
Collaborator

@jayson31 jayson31 left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@maharrshi maharrshi merged commit a4b570f into develop Jul 23, 2025
5 of 9 checks passed
@maharrshi maharrshi deleted the test/automate/project-filters branch July 23, 2025 07:54
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