Merge pull request #166 from pearkes/sort_regions

PR #165 with tests
petems · May 20, 2015 · 50fe615 · 50fe615
2 parents 58b0e10 + 12ed073
commit 50fe615
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/lib/tugboat/middleware/list_regions.rb b/lib/tugboat/middleware/list_regions.rb
@@ -3,7 +3,7 @@ module Middleware
     class ListRegions < Base
       def call(env)
         ocean = env["ocean"]
-        regions = ocean.regions.list.regions
+        regions = ocean.regions.list.regions.sort_by(&:name)
 
         say "Regions:"
         regions.each do |region|

diff --git a/spec/fixtures/show_regions.json b/spec/fixtures/show_regions.json
@@ -1,11 +1,6 @@
 {
   "status": "OK",
   "regions": [
-    {
-      "id": 1,
-      "name": "Region 1",
-      "slug": "reg1"
-    },
     {
       "id": 2,
       "name": "Region 2",
@@ -15,6 +10,11 @@
       "id": 3,
       "name": "Region 3",
       "slug": "reg3"
+    },
+    {
+      "id": 1,
+      "name": "Region 1",
+      "slug": "reg1"
     }
   ]
 }