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

Fix broken backend test. #3208

Merged
merged 1 commit into from
Mar 16, 2017
Merged

Fix broken backend test. #3208

merged 1 commit into from
Mar 16, 2017

Conversation

seanlip
Copy link
Member

@seanlip seanlip commented Mar 16, 2017

No description provided.

@seanlip
Copy link
Member Author

seanlip commented Mar 16, 2017

Hi @tjiang11 -- I figured out the backend test issue. It's actually quite amusing, and now I know why there was a discrepancy :) PTAL?

@codecov-io
Copy link

Codecov Report

Merging #3208 into develop will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop    #3208   +/-   ##
========================================
  Coverage    45.82%   45.82%           
========================================
  Files          230      230           
  Lines        18140    18140           
  Branches      2933     2933           
========================================
  Hits          8313     8313           
  Misses        9827     9827

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4d7be26...bcedf91. Read the comment docs.

Copy link
Contributor

@wxyxinyu wxyxinyu left a comment

Choose a reason for hiding this comment

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

LGTM

@seanlip seanlip merged commit ad7dcb3 into develop Mar 16, 2017
@seanlip seanlip deleted the test-problem branch March 16, 2017 06:06
giritheja added a commit to giritheja/oppia that referenced this pull request Mar 19, 2017
* upstream/develop: (41 commits)
  Fix naming error in PR 3202. (oppia#3220)
  Add e2e tests for suggestion acceptance on explorations (oppia#3202)
  Fix oppia#3180: Fixing issues with the teachOppiaModal. (oppia#3181)
  Fix oppia#2394: Add useful docstrings to core/storage/job/gae_models.py  (oppia#3195)
  Fixes oppia#3192: shifted back css of oppia-sidebar-menu-open back to oppia.css (oppia#3211)
  Add test (oppia#3217)
  Add a check that user contributions for logged in users exist on the home page redirect check (oppia#3213)
  Fix part of oppia#2863: Create front-end fallback domain object (oppia#3207)
  Fix broken backend test. (oppia#3208)
  Fix oppia#3162: set expiration explicitly for files in the build directory. (oppia#3203)
  Add exception for updating user first contribution time for migration bot (oppia#3206)
  Fix oppia#3101: Discard-draft Tooltip (oppia#3205)
  added back formbuilder js in admin.html
  retained formbuilder.js in base.html
  Fix oppia#3099: Clarify error message for Item Selection and Multiple Choice interaction (oppia#3184)
  Resize image region code refactor
  Bump the version number to v2.4.1 and update the changelog
  Add new contributors to AUTHORS, CONTRIBUTORS and about.html
  Fix part of oppia#2863: Create front-end domain object for Interactions (oppia#3049)
  Fix part of oppia#2394: Added docstrings in cron.py (oppia#3182)
  ...
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.

4 participants