diff --git a/common/static/coffee/spec/discussion/content_spec.coffee b/common/static/coffee/spec/discussion/content_spec.coffee index b6075fe05a..10c2ec33d7 100644 --- a/common/static/coffee/spec/discussion/content_spec.coffee +++ b/common/static/coffee/spec/discussion/content_spec.coffee @@ -1,9 +1,6 @@ describe 'All Content', -> beforeEach -> - # TODO: figure out a better way of handling this - # It is set up in main.coffee DiscussionApp.start - window.$$course_id = 'edX/999/test' - window.user = new DiscussionUser {id: '567'} + DiscussionSpecHelper.setUpGlobals() describe 'Content', -> beforeEach -> diff --git a/common/static/coffee/spec/discussion/discussion_spec_helper.coffee b/common/static/coffee/spec/discussion/discussion_spec_helper.coffee new file mode 100644 index 0000000000..fc7286e92c --- /dev/null +++ b/common/static/coffee/spec/discussion/discussion_spec_helper.coffee @@ -0,0 +1,6 @@ +class @DiscussionSpecHelper + # This is sad. We should avoid dependence on global vars. + @setUpGlobals = -> + DiscussionUtil.loadRoles({"Moderator": [], "Administrator": [], "Community TA": []}) + window.$$course_id = "edX/999/test" + window.user = new DiscussionUser({id: "567", upvoted_ids: []}) diff --git a/common/static/coffee/spec/discussion/view/discussion_content_view_spec.coffee b/common/static/coffee/spec/discussion/view/discussion_content_view_spec.coffee index 2a2f3fa7d3..06e8db3dc7 100644 --- a/common/static/coffee/spec/discussion/view/discussion_content_view_spec.coffee +++ b/common/static/coffee/spec/discussion/view/discussion_content_view_spec.coffee @@ -1,6 +1,6 @@ describe "DiscussionContentView", -> beforeEach -> - + DiscussionSpecHelper.setUpGlobals() setFixtures( """
@@ -36,7 +36,6 @@ describe "DiscussionContentView", -> @thread = new Thread(@threadData) @view = new DiscussionContentView({ model: @thread }) @view.setElement($('.discussion-post')) - window.user = new DiscussionUser({id: '567', upvoted_ids: []}) it 'defines the tag', -> expect($('#jasmine-fixtures')).toExist diff --git a/common/static/coffee/spec/discussion/view/discussion_thread_list_view_spec.coffee b/common/static/coffee/spec/discussion/view/discussion_thread_list_view_spec.coffee index 7c7e82abc4..324d336e8e 100644 --- a/common/static/coffee/spec/discussion/view/discussion_thread_list_view_spec.coffee +++ b/common/static/coffee/spec/discussion/view/discussion_thread_list_view_spec.coffee @@ -1,61 +1,140 @@ describe "DiscussionThreadListView", -> beforeEach -> - + DiscussionSpecHelper.setUpGlobals() setFixtures """ diff --git a/lms/templates/discussion/_underscore_templates.html b/lms/templates/discussion/_underscore_templates.html index a1c3585bee..d6dc58ffca 100644 --- a/lms/templates/discussion/_underscore_templates.html +++ b/lms/templates/discussion/_underscore_templates.html @@ -205,16 +205,67 @@ +