Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[WV-953] - Wrote automated test script for checking Followers Count and Endorsers Count for topics Pro-choice and Pro-life #4262

Merged
merged 3 commits into from
Feb 28, 2025

Conversation

mythrienjam
Copy link
Contributor

@mythrienjam mythrienjam commented Feb 25, 2025

What github.com/wevote/WebApp/issues does this fix?

WV-953

Changes included this pull request?

1) Updated OneValue.jsx
Added id’s for tabs “All Endorsers” and “For This Election” here

2) Updated topics.page.js

3) Added VerifyCount.js, - Here I wrote 4 tests

 i) For checking Followers Count for Pro-choice topic
 ii) For Checking endorsers count for Pro-choice - This test should be run after the code in OneValue.jsx is merged
 iii) For checking Followers count for Pro-life 
 iv) For checking endorsers count for Pro-life  - This test should be run after the code in OneValue.jsx is merged since I have added id’s for tabs “All Endorsers” and “For This Election” tabs in  OneValue.jsx and I have used those id's in this test

Copy link
Contributor

@charanya-QA charanya-QA left a comment

Choose a reason for hiding this comment

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

The file OneValue.jsx already has IDs for "Forthis Election" and "Endorsers" button. Its not reflecting on the website because website is not updated with the latest code. So you can use id = "forThisElection" and id = "allEndorsers" on the line 267 and 277 respectively. Please change the test scripts accordingly and revert onevalue.jsx changes . Thanks!

@mythrienjam
Copy link
Contributor Author

Since I have latest code now on my machine, I am able to see id = "forThisElection" and id = "allEndorsers"

Let me work on this complete it as soon as possible

@mythrienjam
Copy link
Contributor Author

mythrienjam commented Feb 28, 2025

Since id's were added already , Deleted my changes in OneValue.jsx
Changed my code accordingly.

i) Test for checking Followers Count for Pro-choice topic
ii) Test for Checking endorsers count for Pro-choice - It looks like ID's are not yet available on quality.wevote.us
iii) Test for checking Followers count for Pro-life
iv) Test foror checking endorsers count for Pro-life - It looks like ID's are not yet available on quality.wevote.us

@charanya-QA
Copy link
Contributor

Thanks @mythrienjam

@charanya-QA charanya-QA merged commit 5a92f6d into wevote:develop Feb 28, 2025
4 checks 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