Skip to content

Added invalid review with rating only#44

Merged
TAMILKUMAR0027 merged 1 commit into
mainfrom
Jothika
Jun 14, 2026
Merged

Added invalid review with rating only#44
TAMILKUMAR0027 merged 1 commit into
mainfrom
Jothika

Conversation

@raviravi31799-crypto

Copy link
Copy Markdown
Collaborator

Pull Request Details

PR Information

  • PR Number: Added checkout page #8
  • Author: JOTHIKA R
  • Created Date: 14-06-2026
  • Branch Name: Jothika
  • Target Branch: main

Team Information


Related Issue

Fixes #

Closes #


Summary

Added invalid test cases for the Add Review functionality by validating scenarios where only the star rating is selected and mandatory review fields are left empty.


Detailed Changes

Added

  • Invalid Add Review test scenarios.
  • Test data for negative review validation in DataProvider.xlsx.

Modified

  • Add Review test cases.
  • Excel data provider entries for invalid scenarios.

Removed

  • None.

Refactored

  • None.

Type of Change

  • Test Automation
  • Bug Fix
  • New Feature
  • Enhancement
  • Refactoring
  • Performance Improvement
  • CI/CD Changes
  • Documentation Update
  • Dependency Upgrade

Testing Performed

Pytest Execution

  • All tests passed
  • Smoke suite passed
  • Sanity suite passed
  • Regression suite passed
  • E2E suite passed
  • Negative test scenarios verified

Browsers Tested

  • Chrome
  • Firefox
  • Headless Chrome
  • Headless Firefox

Evidence

  • Executed Add Review negative scenarios locally using Pytest.
  • Verified warning messages for invalid review submission.

Configuration Changes

  • No configuration changes
  • config.ini modified
  • pytest.ini modified
  • requirements.txt updated
  • Environment variables added

Impact Analysis

Affected Modules:

  • Add Review Page
  • Add Review Page Action
  • DataProvider.xlsx

Potential Risks:

  • Low

Rollback Plan:

  • Revert the Add Review invalid test case changes and restore previous test data.

Checklist

Code Quality

  • Code follows project standards
  • No hardcoded values
  • Logging added where required
  • Exception handling implemented
  • Unused imports removed
  • PEP8 guidelines followed

Testing

  • Test cases added/updated
  • Existing tests are passing
  • Local execution verified

Documentation

  • README updated
  • Comments added where necessary
  • Documentation updated

CI/CD

  • GitHub Actions passed
  • Build succeeded
  • No merge conflicts

Reviewer Notes

Special instructions for reviewers:

  • Please verify the warning messages returned when only the star rating is selected and the mandatory review fields are left empty.

Final Approval

@14-Prasanna 14-Prasanna left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

okay doner

@Rishwanth-Adhishwar Rishwanth-Adhishwar left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Viewed Your Feature, It doesn,t affect other modules, and it is ready to merge

@samihamuhabathulla-jpg samihamuhabathulla-jpg left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

good

@TAMILKUMAR0027 TAMILKUMAR0027 left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Approved

@TAMILKUMAR0027 TAMILKUMAR0027 merged commit 43d66b4 into main Jun 14, 2026
1 check failed
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.

5 participants