Skip to content

Completed addReview feature and added AddOns#49

Open
raviravi31799-crypto wants to merge 2 commits into
mainfrom
Jothika
Open

Completed addReview feature and added AddOns#49
raviravi31799-crypto wants to merge 2 commits into
mainfrom
Jothika

Conversation

@raviravi31799-crypto

Copy link
Copy Markdown
Collaborator

Pull Request Details

PR Information


Team Information


Related Issue

Fixes #

Closes #


Summary

Implemented AddOns feature components functionality and completed all test cases for the Add Review feature.


Detailed Changes

Added

  • AddOns page object and action classes.
  • Test cases for left drawer and right drawer functionality.
  • Add Review negative test scenarios.

Modified

  • Existing page objects and action methods.
  • Data provider Excel sheets.
  • Test scripts and logging statements.

Removed

  • None

Refactored

  • Improved page action methods and locator usage.

Type of Change

  • Bug Fix
  • New Feature
  • Enhancement
  • Refactoring
  • Performance Improvement
  • Test Automation
  • 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

Attached execution logs and Pytest reports.


Configuration Changes

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

Impact Analysis

Affected Modules:

  • AddOns
  • Add Review
  • Data Provider

Potential Risks:

  • Low

Rollback Plan:


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 AddOns left and right drawer functionality and Add Review positive and negative scenarios.

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 done

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.

Delete this file

@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.

Everything is Fine,While pushing Close Your Excel To avoid This File:data_provider/~$DataProvider.xlsx

@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

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