From a04690d051ac884aa8bd24cf62bcca9181c77fde Mon Sep 17 00:00:00 2001 From: Calen Pennington Date: Fri, 25 Sep 2020 11:00:34 -0400 Subject: [PATCH] Increase the mongo query count by one because edxnotes has been turned back on, and it loads the course from the modulestore as part of get_html --- lms/djangoapps/courseware/testutils.py | 6 +++--- lms/djangoapps/lti_provider/tests/test_views.py | 2 +- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/lms/djangoapps/courseware/testutils.py b/lms/djangoapps/courseware/testutils.py index 629cc41944..a7613585eb 100644 --- a/lms/djangoapps/courseware/testutils.py +++ b/lms/djangoapps/courseware/testutils.py @@ -155,9 +155,9 @@ class RenderXBlockTestMixin(six.with_metaclass(ABCMeta, object)): return response @ddt.data( - ('vertical_block', ModuleStoreEnum.Type.mongo, 12), + ('vertical_block', ModuleStoreEnum.Type.mongo, 13), ('vertical_block', ModuleStoreEnum.Type.split, 6), - ('html_block', ModuleStoreEnum.Type.mongo, 13), + ('html_block', ModuleStoreEnum.Type.mongo, 14), ('html_block', ModuleStoreEnum.Type.split, 6), ) @ddt.unpack @@ -214,7 +214,7 @@ class RenderXBlockTestMixin(six.with_metaclass(ABCMeta, object)): Helper method used by test_success_enrolled_staff because one test class using this mixin has an increased number of mongo (only) queries. """ - return 8 + return 9 def test_success_unenrolled_staff(self): self.setup_course() diff --git a/lms/djangoapps/lti_provider/tests/test_views.py b/lms/djangoapps/lti_provider/tests/test_views.py index 79fd816985..ac740898db 100644 --- a/lms/djangoapps/lti_provider/tests/test_views.py +++ b/lms/djangoapps/lti_provider/tests/test_views.py @@ -241,4 +241,4 @@ class LtiLaunchTestRender(LtiTestMixin, RenderXBlockTestMixin, ModuleStoreTestCa # (5)-(8) calls related to the inherited user_partitions field. # (9) edx_notes descriptor call to get_course """ - return 8 + return 9