Merge pull request #10941 from julienw/890460-fixedheader-followup

Bug 890460 - [SMS] After deleting all the messages in particular thread,...
mozilla-b2g · Jul 12, 2013 · accb67f · accb67f
2 parents 8a7caf9 + 0dbbeba
commit accb67f
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 0 deletions.
diff --git a/apps/sms/js/thread_list_ui.js b/apps/sms/js/thread_list_ui.js
@@ -339,6 +339,7 @@ var ThreadListUI = {
       });
     } else {
       ThreadListUI.setEmpty(true);
+      FixedHeader.refresh();
 
       // Callback if exist
       if (renderCallback) {

diff --git a/apps/sms/test/unit/thread_list_ui_test.js b/apps/sms/test/unit/thread_list_ui_test.js
@@ -514,4 +514,18 @@ suite('thread_list_ui', function() {
       });
     });
   });
+
+  suite('renderThreads', function() {
+    setup(function() {
+      this.sinon.spy(FixedHeader, 'refresh');
+      this.sinon.spy(ThreadListUI, 'setEmpty');
+    });
+
+    test('Rendering an empty screen', function() {
+      ThreadListUI.renderThreads([]);
+      assert.ok(FixedHeader.refresh.called);
+      assert.ok(ThreadListUI.setEmpty.called);
+      assert.isTrue(ThreadListUI.setEmpty.args[0][0]);
+    });
+  });
 });