Implement part of #17712 : Acceptance tests for Exploration Creator Section(CUJ 10).

rahat2134 commented 2 weeks ago

Overview

This PR fixes part of #17712 .
This PR does the following: Adds acceptance test for exploration editor section. Points: 10- User can Publish the latest changes, User can draft the latest changes)
Essential Checklist

[X] The PR title starts with "Fix #bugnum: " or "Fix part of #bugnum: ...", followed by a short, clear summary of the changes.
[X] I have followed the instructions for making a code change.
[X] I have assigned the correct reviewers to this PR (or will leave a comment with the phrase "@{{reviewer_username}} PTAL" if I don't have permissions to assign reviewers directly).
[X] The linter/Karma presubmit checks pass on my local machine, and my PR follows the coding style guide).
[X] "Allow edits from maintainers" is checked. (See here for instructions on how to enable it.)

Proof that changes are correct

PR Pointers

Never force push! If you do, your PR will be closed.
To reply to reviewers, follow these instructions: https://github.com/oppia/oppia/wiki/Make-a-pull-request#step-5-address-review-comments-until-all-reviewers-approve
Some e2e tests are flaky, and can fail for reasons unrelated to your PR. We are working on fixing this, but in the meantime, if you need to restart the tests, please check the "If your build fails" wiki page.
See the Code Owner's wiki page for what code owners will expect.

oppiabot[bot] commented 2 weeks ago

Hi @rahat2134 please assign the required reviewer(s) for this PR. Thanks!

rahat2134 commented 2 weeks ago

@seanlip PTAL. Although this has already been reviewed by you. But for the sake of convenience take a look again. Modification - • Add a wait for autosave pop to be visible and then hidden (for desktop) when we make any change. • Have to add external wait For of 2 seconds because in mobile viewport we don't have Auto saving.... pop up. There is no UI change. So, At last, I have to use waitForTimeout

oppiabot[bot] commented 2 weeks ago

Hi @rahat2134, it looks like some changes were requested on this pull request by @StephenYu2018. PTAL. Thanks!

rahat2134 commented 2 weeks ago

@StephenYu2018 PTAL

oppiabot[bot] commented 2 weeks ago

Unassigning @StephenYu2018 since they have already approved the PR.

oppiabot[bot] commented 2 weeks ago

Hi @rahat2134. Due to recent changes in the "develop" branch, this PR now has a merge conflict. Please follow this link if you need help resolving the conflict, so that the PR can be merged. Thanks!

rahat2134 commented 2 weeks ago

Changing PR to draft PR till #20231 got merge

oppia / oppia