From 5c8daf414463b3744c8445460ddf9f440f623587 Mon Sep 17 00:00:00 2001
From: Greg Price <gprice@edx.org>
Date: Thu, 4 Sep 2014 17:13:30 -0400
Subject: [PATCH] Improve thread list view tests

An obsolete test case was passing because the assertions were too weak,
so the assertions are improved and the test case removed.
---
 .../view/discussion_thread_list_view_spec.coffee       | 10 +---------
 1 file changed, 1 insertion(+), 9 deletions(-)

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 3f434dd2fd..1b2940e54d 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
@@ -475,6 +475,7 @@ describe "DiscussionThreadListView", ->
         testSelectionRequest = (callback, itemText) ->
           setupAjax(callback)
           $(".forum-nav-browse-title:contains(#{itemText})").click()
+          expect($.ajax).toHaveBeenCalled()
 
         it "should get all discussions", ->
           testSelectionRequest(
@@ -482,15 +483,6 @@ describe "DiscussionThreadListView", ->
             "All"
           )
 
-        it "should get flagged threads", ->
-          testSelectionRequest(
-            (params) ->
-              expect(params.url.path()).toEqual(DiscussionUtil.urlFor("search"))
-              expect(params.data.flagged).toEqual(true)
-            ,
-            "Flagged"
-          )
-
         it "should get followed threads", ->
           testSelectionRequest(
             (params) ->