GitHub PullRequestReviewCommentEvent at 2014-03-25 14:37:58
Repository: edx-platform
Actor: valera-rozuvan
Payload: {"comment":{"url":"https://api.github.com/repos/edx/edx-platform/pulls/comments/10932823","id":10932823,"diff_hunk":"@@ -477,3 +491,40 @@ def test_source_is_empty(self):\n \n         self.assertNotIn('source', fields)\n         self.assertFalse(self.item_descriptor.download_video)\n+\n+\n+class VideoDescriptorTest(unittest.TestCase):\n+    \"\"\"\n+    Tests for video descriptor that requires access to django settings.\n+    \"\"\"\n+\n+    def setUp(self):\n+        system = get_test_descriptor_system()\n+        location = Location('i4x://org/course/video/name')\n+        self.descriptor = system.construct_xblock_from_class(\n+            VideoDescriptor,\n+            scope_ids=ScopeIds(None, None, location, location),\n+            field_data=DictFieldData({}),\n+        )\n+        self.descriptor.runtime.handler_url = MagicMock()\n+\n+    def test_get_context(self):\n+        \"\"\"\"\n+        Test get_context.\n+\n+        This test is located here and not in xmodule.tests because get_context calls editable_metadata_fields.\n+        Which, in turn, uses settings.LANGUAGES from django setttings.\n+        \"\"\"\n+        correct_tabs = [\n+            {\n+                'name': \"Basic\",\n+                'template': \"video/transcripts.html\",\n+                'current': True\n+            },\n+            {\n+                'name': 'Advanced',\n+                'template': 'tabs/metadata-edit-tab.html'","path":"lms/djangoapps/courseware/tests/test_video_mongo.py","position":285,"original_position":285,"commit_id":"6dc68e1b138b23a8d4969519f13cb681657d075f","original_commit_id":"6dc68e1b138b23a8d4969519f13cb681657d075f","user":{"login":"valera-rozuvan","id":2273090,"avatar_url":"https://avatars.githubusercontent.com/u/2273090?","gravatar_id":"937a84db1c29347e9c165db225c1ed79","url":"https://api.github.com/users/valera-rozuvan","html_url":"https://github.com/valera-rozuvan","followers_url":"https://api.github.com/users/valera-rozuvan/followers","following_url":"https://api.github.com/users/valera-rozuvan/following{/other_user}","gists_url":"https://api.github.com/users/valera-rozuvan/gists{/gist_id}","starred_url":"https://api.github.com/users/valera-rozuvan/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/valera-rozuvan/subscriptions","organizations_url":"https://api.github.com/users/valera-rozuvan/orgs","repos_url":"https://api.github.com/users/valera-rozuvan/repos","events_url":"https://api.github.com/users/valera-rozuvan/events{/privacy}","received_events_url":"https://api.github.com/users/valera-rozuvan/received_events","type":"User","site_admin":false},"body":"@auraz Should a comma be added a the end?","created_at":"2014-03-25T14:37:58Z","updated_at":"2014-03-25T14:37:58Z","html_url":"https://github.com/edx/edx-platform/pull/2899#discussion_r10932823","pull_request_url":"https://api.github.com/repos/edx/edx-platform/pulls/2899","_links":{"self":{"href":"https://api.github.com/repos/edx/edx-platform/pulls/comments/10932823"},"html":{"href":"https://github.com/edx/edx-platform/pull/2899#discussion_r10932823"},"pull_request":{"href":"https://api.github.com/repos/edx/edx-platform/pulls/2899"}}}}