openedx
fix: paver js_test, tests and test_js_test unit testing removed
Description
Remove the codebase specific for testing in the paver.
Useful information to include:
- Which edX user roles will this change impact? Common user roles are "Learner", "Course Author", "Developer", and "Operator".
- Include screenshots for changes to the UI (ideally, both "before" and "after" screenshots, if applicable).
- Provide links to the description of corresponding configuration changes. Remember to correctly annotate these changes.
Supporting information
Link to other information about the change, such as Jira issues, GitHub issues, or Discourse discussions. Be sure to check they are publicly readable, or if not, repeat the information here.
Testing instructions
Please provide detailed step-by-step instructions for testing this change.
Deadline
"None" if there's no rush, or provide a specific date or event (and reason) if there is one.
Other information
Include anything else that will help reviewers and consumers understand the change.
- Does this change depend on other changes elsewhere?
- Any special concerns or limitations? For example: deprecations, migrations, security, or accessibility.
- If your database migration can't be rolled back easily.
@zubairshakoorarbisoft could you please expand the PR description with a little more details about the reason for this change? And please remove the template text like "Useful information to include:" and "Please provide detailed step-by-step instructions for testing this change." if you are not using it.
@bradenmacdonald Its kind of draft PR, currently working on it. The reason is to remove paver code (only tests specific) is that now we are are running test using pytest that's why need to remove this useless codebase from this repo.
To get the more context on this task, please have a look into this issue https://github.com/openedx/edx-platform/issues/31168
CC: @iamsobanjaved
On Fri, 21 Oct 2022 at 11:53 PM, Braden MacDonald @.***> wrote:
@zubairshakoorarbisoft https://github.com/zubairshakoorarbisoft could you please expand the PR description with a little more details about the reason for this change? And please remove the template text like "Useful information to include:" and "Please provide detailed step-by-step instructions for testing this change." if you are not using it.
— Reply to this email directly, view it on GitHub https://github.com/openedx/edx-platform/pull/31180#issuecomment-1287327326, or unsubscribe https://github.com/notifications/unsubscribe-auth/ANX4P4VZFJE25D3S2JNMCSLWELRBJANCNFSM6AAAAAARJ5ZM7M . You are receiving this because you were mentioned.Message ID: @.***>
--
The content of this email and attachment(s) are confidential and intended solely for the recipient(s) specified in the message. If you received this email by error, please alert the sender by replying to this email and follow with its deletion, so that we can ensure such an error does not occur in future. If you are not the intended recipient, you are strictly prohibited to use, copy, disseminate or store this email or its attachment(s).
EdX Release Notice: This PR has been deployed to the staging environment in preparation for a release to production.
EdX Release Notice: This PR has been deployed to the production environment.