From e1751c065cb556284e85560a39d3ba58e8d85b66 Mon Sep 17 00:00:00 2001
From: Alanna Scott <alanna.scott@airbnb.com>
Date: Tue, 13 Jun 2017 09:44:26 -0700
Subject: [PATCH] add new slice test (#2939)

* sort explicitly on label

* add simple test for /slicemodelview/add endpoint

* make comments and method names more clear

* fix test name

* be more explicit, test status_code
---
 superset/views/core.py |  2 +-
 tests/core_tests.py    | 11 +++++++++--
 2 files changed, 10 insertions(+), 3 deletions(-)

diff --git a/superset/views/core.py b/superset/views/core.py
index b85f01306c..c96308fd44 100755
--- a/superset/views/core.py
+++ b/superset/views/core.py
@@ -380,7 +380,7 @@ class SliceModelView(SupersetModelView, DeleteMixin):  # noqa
         return self.render_template(
             "superset/add_slice.html",
             bootstrap_data=json.dumps({
-                'datasources': sorted(datasources),
+                'datasources': sorted(datasources, key=lambda d: d["label"]),
             }),
         )
 
diff --git a/tests/core_tests.py b/tests/core_tests.py
index 437511d652..5155d6e121 100644
--- a/tests/core_tests.py
+++ b/tests/core_tests.py
@@ -216,17 +216,24 @@ class CoreTests(SupersetTestCase):
             logging.info("[{name}]/[{method}]: {url}".format(**locals()))
             self.client.get(url)
 
-    def test_add_slice(self):
+    def test_tablemodelview_list(self):
         self.login(username='admin')
 
-        # Click on the + to add a slice
         url = '/tablemodelview/list/'
         resp = self.get_resp(url)
 
+        # assert that a table is listed
         table = db.session.query(SqlaTable).first()
         assert table.name in resp
         assert '/superset/explore/table/{}'.format(table.id) in resp
 
+    def test_add_slice(self):
+        self.login(username='admin')
+        # assert that /slicemodelview/add responds with 200
+        url = '/slicemodelview/add'
+        resp = self.client.get(url)
+        self.assertEqual(resp.status_code, 200)
+
     def test_slices_V2(self):
         # Add explore-v2-beta role to admin user
         # Test all slice urls as user with with explore-v2-beta role