From 908fc3999e06f3ccd3bb8ad0539490bbc7809748 Mon Sep 17 00:00:00 2001
From: Tobias Kunze <r@rixx.de>
Date: Mon, 14 Oct 2019 05:52:33 +0200
Subject: [PATCH] Sort databases on homepage by argument order - #591

Closes #585 - thanks, @rixx!
---
 datasette/app.py         | 2 +-
 datasette/views/index.py | 2 --
 tests/test_html.py       | 4 ++--
 3 files changed, 3 insertions(+), 5 deletions(-)

diff --git a/datasette/app.py b/datasette/app.py
index 41a4eb37..935b1730 100644
--- a/datasette/app.py
+++ b/datasette/app.py
@@ -159,7 +159,7 @@ class Datasette:
             self.files = [MEMORY]
         elif memory:
             self.files = (MEMORY,) + self.files
-        self.databases = {}
+        self.databases = collections.OrderedDict()
         self.inspect_data = inspect_data
         for file in self.files:
             path = file
diff --git a/datasette/views/index.py b/datasette/views/index.py
index fddb04d9..f2e5f774 100644
--- a/datasette/views/index.py
+++ b/datasette/views/index.py
@@ -97,8 +97,6 @@ class IndexView(BaseView):
                 }
             )
 
-        databases.sort(key=lambda database: database["name"])
-
         if as_format:
             headers = {}
             if self.ds.cors:
diff --git a/tests/test_html.py b/tests/test_html.py
index 0a6df984..ec7765f6 100644
--- a/tests/test_html.py
+++ b/tests/test_html.py
@@ -26,11 +26,11 @@ def test_homepage(app_client_two_attached_databases):
     )
     # Should be two attached databases
     assert [
-        {"href": "/extra_database", "text": "extra_database"},
         {"href": "/fixtures", "text": "fixtures"},
+        {"href": "/extra_database", "text": "extra_database"},
     ] == [{"href": a["href"], "text": a.text.strip()} for a in soup.select("h2 a")]
     # The first attached database should show count text and attached tables
-    h2 = soup.select("h2")[0]
+    h2 = soup.select("h2")[1]
     assert "extra_database" == h2.text.strip()
     counts_p, links_p = h2.find_all_next("p")[:2]
     assert (