Fix order of API view functions

Before, the routes/views with parameters in the URL where defined first.
E.g. `@api.route("/api/movie/genre/{genre}")` was defined before
`@api.route("/api/movie/genre/all")`.

When trying to hit the URL for `/api/movie/genre/all` the response would
be empty. This is because the this would not trigger the desired view,
but the parameterized view would be passes `genre = all`. Since there
are no movies with the genre "all" the response would be empty.

To solve this issue, the ordering of the routes/views had to be
adjusted. Now the most specific views are defined first, and then the
more flexible views are defined.

Author: tbrlpld

days/097-100-docker/demo/services/movie_svc/views/api_views.py

@@ -40,10 +40,9 @@ def search_director(_, resp, director_name: str):
     resp.media = {'keyword': director_name, 'hits': movies_dicts, 'truncated_results': limited}
 
 
-@api.route("/api/movie/genre/{genre}")
-def movies_by_genre(_, resp: Response, genre: str):
-    hits = db.movies_by_genre(genre)
-    print("Searching for movies by genre {}, {} results".format(genre, len(hits)))
+@api.route("/api/movie/top")
+def top_movies(_, resp: Response):
+    hits = db.movies_by_popularity()
 
     limited = len(hits) > response_count_max
     if limited:
@@ -54,7 +53,9 @@ def movies_by_genre(_, resp: Response, genre: str):
         for m in hits
     ]
 
-    resp.media = {'genre': genre, 'hits': hits_dicts, 'truncated_results': limited}
+    keyword = "top_{}".format(response_count_max)
+
+    resp.media = {'keyword': keyword, 'hits': hits_dicts, 'truncated_results': limited}
 
 
 @api.route("/api/movie/{imdb_number}")
@@ -65,9 +66,15 @@ def search_imdb(_, resp, imdb_number: str):
     resp.media = db.movie_to_dict(movie)
 
 
-@api.route("/api/movie/top")
-def top_movies(_, resp: Response):
-    hits = db.movies_by_popularity()
+@api.route("/api/movie/genre/all")
+def all_genres(_, resp: Response):
+    resp.media = db.all_genres()
+
+
+@api.route("/api/movie/genre/{genre}")
+def movies_by_genre(_, resp: Response, genre: str):
+    hits = db.movies_by_genre(genre)
+    print("Searching for movies by genre {}, {} results".format(genre, len(hits)))
 
     limited = len(hits) > response_count_max
     if limited:
@@ -78,11 +85,4 @@ def top_movies(_, resp: Response):
         for m in hits
     ]
 
-    keyword = "top_{}".format(response_count_max)
-
-    resp.media = {'keyword': keyword, 'hits': hits_dicts, 'truncated_results': limited}
-
-
-@api.route("/api/movie/genre/all")
-def all_genres(_, resp: Response):
-    resp.media = db.all_genres()
+    resp.media = {'genre': genre, 'hits': hits_dicts, 'truncated_results': limited}