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

Added notes to run single e2e and unit tests. #5116

Merged
merged 2 commits into from Jun 17, 2018

Conversation

DubeySandeep
Copy link
Member

Explanation

Added notes to run single e2e and unit tests.

Checklist

  • The PR title starts with "Fix #bugnum: ", followed by a short, clear summary of the changes.
  • The PR explanation includes the words "Fixes #bugnum: ...".
  • The linter/Karma presubmit checks have passed.
    • These should run automatically, but if not, you can manually trigger them locally using python scripts/pre_commit_linter.py and bash scripts/run_frontend_tests.sh.
  • The PR is made from a branch that's not called "develop".
  • The PR follows the style guide.
  • The PR is assigned to an appropriate reviewer.
    • If you're a new contributor, please ask on Gitter for someone to assign a reviewer.
    • If you're not sure who the appropriate reviewer is, please assign to the issue's "owner" -- see the "talk-to" label on the issue.

feconf.py Outdated
@@ -520,23 +520,23 @@ def get_empty_ratings():
DEMO_EXPLORATIONS = {
u'0': 'welcome.yaml',
u'1': 'multiples.yaml',
u'2': 'binary_search',
u'2': 'binary_search.yaml',
Copy link
Member

Choose a reason for hiding this comment

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

Why this change? I thought exploration files with assets were referred to without the yaml suffix.

Copy link
Member Author

Choose a reason for hiding this comment

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

Oh! Thanks for the info, this wasn't known to me and I found admin page was looking bit weird so did it. I'll revert this changes. Thanks!

Copy link
Member Author

Choose a reason for hiding this comment

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

Done!

@codecov-io
Copy link

codecov-io commented Jun 17, 2018

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff            @@
##           develop    #5116   +/-   ##
========================================
  Coverage    46.12%   46.12%           
========================================
  Files          423      423           
  Lines        24562    24562           
  Branches      3919     3919           
========================================
  Hits         11328    11328           
  Misses       13234    13234

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 9f3fbeb...1812b28. Read the comment docs.

Copy link
Member

@seanlip seanlip left a comment

Choose a reason for hiding this comment

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

Thanks!

@DubeySandeep DubeySandeep merged commit d41671a into oppia:develop Jun 17, 2018
@DubeySandeep DubeySandeep deleted the social-commite-work branch June 17, 2018 07:49
hoangviet1993 pushed a commit to hoangviet1993/oppia that referenced this pull request Jun 20, 2018
* Added notes to run single e2e and unit tests
giritheja added a commit to giritheja/oppia that referenced this pull request Jun 20, 2018
* upstream/develop: (78 commits)
  Create Topic Editor (Part 3): Create the editor UI (oppia#5112)
  Fix part of oppia#3968: ParamMetadataObjectFactory domain object (oppia#5082)
  Update CHANGELOG for v2.6.5 (oppia#5123)
  fix backend load test (oppia#5120)
  Improving the dev workflow: Add restrict check for directives (oppia#5068)
  Added notes to run single e2e and unit tests. (oppia#5116)
  Add node path to stylelint cmd args (oppia#5117)
  Fix oppia#5111: remove notification preferences card from non-editors.
  Fix oppia#5009: Add wait for suggestions e2e test (oppia#5109)
  Create Topic Editor (Part 2): Created Topics and Skills dashboard (oppia#5094)
  Milestone 1.2: Lesson translation dashboard. (oppia#5099)
  Add angular-ui-tree dependency. (oppia#5110)
  Number with Units Interaction: Milestone 1.3 (oppia#5028)
  Fix oppia#5083: Audio needs update span disable for guest users. (oppia#5091)
  Fix oppia#5044: Added a named constant for revert commit command (oppia#5102)
  Create Topic Editor (Part 1): Added frontend domain objects for topics. (oppia#5090)
  Added try again button. (oppia#5087)
  Fix oppia#5088: Removed refresh option from the create view (oppia#5100)
  Added migration function for Textangular - Milestone 1 (part 2) (oppia#4985)
  Routine translations update. (oppia#5101)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants