oppia icon indicating copy to clipboard operation
oppia copied to clipboard

[GSoC'24] M1.10, Fix a part of #20374: Acceptance test coverage for 2 Topics Manager's CUJs

Open Akhilesh-max opened this issue 8 months ago • 14 comments

Overview

  1. This PR fixes or fixes part of #20374 .

  2. This PR does the following: This PR provides acceptance test coverage for the CUJs:

    2.9: create-and-delete-questions-in-skill-editor.spec.ts

    2.10: assign-unassign-and-merge-skills.spec.ts

Essential Checklist

Please follow the instructions for making a code change.

  • [X] I have linked the issue that this PR fixes in the "Development" section of the sidebar.
  • [X] I have checked the "Files Changed" tab and confirmed that the changes are what I want to make.
  • [X] I have written tests for my code.
  • [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 assigned the correct reviewers to this PR (or will leave a comment with the phrase "@{{reviewer_username}} PTAL" if I can't assign them directly).

Proof that changes are correct

NA

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.

Akhilesh-max avatar Jun 26 '24 23:06 Akhilesh-max