Skip to content
This repository was archived by the owner on Mar 25, 2025. It is now read-only.

EDS-233: Fixing python tests - #301

Merged
HamzaIbnFarooq merged 7 commits into
developfrom
hamza/EDS-233/Fix_python_tests
Feb 25, 2020
Merged

EDS-233: Fixing python tests#301
HamzaIbnFarooq merged 7 commits into
developfrom
hamza/EDS-233/Fix_python_tests

Conversation

@HamzaIbnFarooq

Copy link
Copy Markdown

Story Link

EDS-233

PR Description

After configuring python tests for edx-platform, we came to know that a few tests are failing. This PR is meant to fix all those tests.

List of currently failing tests

Type of change

Please select the options that are relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation Change

How to test?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configurations.

  • Test A
  • Test B

Checklist before merging:

  • Squased
  • Reviewd

@pep8speaks

pep8speaks commented Feb 17, 2020

Copy link
Copy Markdown

Hello @HamzaIbnFarooq! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻

Comment last updated at 2020-02-17 13:58:37 UTC

@HamzaIbnFarooq
HamzaIbnFarooq force-pushed the hamza/EDS-233/Fix_python_tests branch from 5a57a2a to 8ba8561 Compare February 17, 2020 07:05
@HamzaIbnFarooq
HamzaIbnFarooq force-pushed the hamza/EDS-233/Fix_python_tests branch from d23af45 to 55bec9e Compare February 17, 2020 07:24
@HamzaIbnFarooq

Copy link
Copy Markdown
Author

jenkins run python

@edtecheco

Copy link
Copy Markdown

Your PR has finished running tests. The following contexts failed:

  • jenkins/python

@HamzaIbnFarooq

Copy link
Copy Markdown
Author

jenkins run python

@edtecheco

Copy link
Copy Markdown

Your PR has finished running tests. The following contexts failed:

  • jenkins/python

@HamzaIbnFarooq

Copy link
Copy Markdown
Author

jenkins run python

@HamzaIbnFarooq

Copy link
Copy Markdown
Author

jenkins run python

@edtecheco

Copy link
Copy Markdown

Your PR has finished running tests. There were no failures.

@HamzaIbnFarooq
HamzaIbnFarooq merged commit 853bf15 into develop Feb 25, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants