make search term optional (bug 619052)

Author: cvan

apps/search/templates/search/results.html

@@ -73,15 +73,17 @@ <h3>{{ title }}</h3>
       <h2>{{ _('Filter Results') }}</h2>
       <ul class="facets island pjax-trigger">
         {{ facet(_('Category'), 'category-facets', categories) }}
-        <li id="compat-facets" class="facet">
-          <h3>{{ _('Works with') }}</h3>
-          {% if versions %}
-            {{ facet_links(versions) }}
-          {% endif %}
-          {% if platforms %}
-            {{ facet_links(platforms) }}
-          {% endif %}
-        </li>
+        {% if versions or platforms %}
+          <li id="compat-facets" class="facet">
+            <h3>{{ _('Works with') }}</h3>
+            {% if versions %}
+              {{ facet_links(versions) }}
+            {% endif %}
+            {% if platforms %}
+              {{ facet_links(platforms) }}
+            {% endif %}
+          </li>
+        {% endif %}
         {{ facet(_('Tag'), 'tag-facets', tags) }}
       </ul>
       {{ num_results() }}

apps/search/tests/test_views.py

@@ -476,6 +476,9 @@ def test_mobile_get(self):
         eq_(r.status_code, 200)
         self.assertTemplateUsed(r, 'search/mobile/results.html')
 
+    def test_no_compat_facets(self):
+        assert not pq(self.client.get(self.url).content)('#compat-facets')
+
 
 class TestAjaxSearch(amo.tests.ESTestCase):