bite 21

Author: [email protected]

Diff for: .gitignore

@@ -21,3 +21,5 @@
 /165/README.md
 /167/README.md
 /169/README.md
+/172/README.md
+/21/README.md

Diff for: 21/cars.py

@@ -0,0 +1,54 @@
+cars = {
+    'Ford': ['Falcon', 'Focus', 'Festiva', 'Fairlane'],
+    'Holden': ['Commodore', 'Captiva', 'Barina', 'Trailblazer'],
+    'Nissan': ['Maxima', 'Pulsar', '350Z', 'Navara'],
+    'Honda': ['Civic', 'Accord', 'Odyssey', 'Jazz'],
+    'Jeep': ['Grand Cherokee', 'Cherokee', 'Trailhawk', 'Trackhawk']
+}
+
+
+def get_all_jeeps(cars=cars):
+    """return a comma  + space (', ') separated string of jeep models
+       (original order)"""
+    all_jeeps = []
+    # print(cars['Jeep'][0])
+    for car in cars['Jeep']:
+        all_jeeps.append(car)
+    all_jeeps = ", ".join(all_jeeps)
+    return all_jeeps
+
+
+# print(get_all_jeeps(cars))
+
+
+def get_first_model_each_manufacturer(cars=cars):
+    """return a list of matching models (original ordering)"""
+    first_model = list()
+    for car in cars.keys():
+        first_model.append(cars[car][0])
+    return first_model
+
+# print(get_first_model_each_manufacturer(cars))
+
+
+def get_all_matching_models(cars=cars, grep='trail'):
+    """return a list of all models containing the case insensitive
+       'grep' string which defaults to 'trail' for this exercise,
+       sort the resulting sequence alphabetically"""
+    match = list()
+    for manufacturer in cars.keys():
+        for car in cars[manufacturer]:
+            if grep.lower() in car.lower():
+                match.append(car)
+    return sorted(match)
+# print(get_all_matching_models(grep='CO'))
+
+
+def sort_car_models(cars=cars):
+    """return a copy of the cars dict with the car models (values)
+       sorted alphabetically"""
+    for manufacturer in cars.keys():
+        cars[manufacturer] = sorted(cars[manufacturer])
+    return cars
+
+# print(sort_car_models())

Diff for: 21/test_cars.py

@@ -0,0 +1,40 @@
+from cars import (get_all_jeeps, get_first_model_each_manufacturer,
+                  get_all_matching_models, sort_car_models)
+
+
+def test_get_all_jeeps():
+    expected = 'Grand Cherokee, Cherokee, Trailhawk, Trackhawk'
+    actual = get_all_jeeps()
+    assert type(actual) == str
+    assert actual == expected
+
+
+def test_get_first_model_each_manufacturer():
+    actual = get_first_model_each_manufacturer()
+    expected = ['Falcon', 'Commodore', 'Maxima', 'Civic', 'Grand Cherokee']
+    assert actual == expected
+
+
+def test_get_all_matching_models_default_grep():
+    expected = ['Trailblazer', 'Trailhawk']
+    assert get_all_matching_models() == expected
+
+
+def test_get_all_matching_models_different_grep():
+    expected = ['Accord', 'Commodore', 'Falcon']
+    assert get_all_matching_models(grep='CO') == expected
+
+
+def test_sort_dict_alphabetically():
+    actual = sort_car_models()
+    # Order of keys should not matter, two dicts are equal if they have the
+    # same keys and the same values.
+    # The car models (values) need to be sorted here though
+    expected = {
+        'Ford': ['Fairlane', 'Falcon', 'Festiva', 'Focus'],
+        'Holden': ['Barina', 'Captiva', 'Commodore', 'Trailblazer'],
+        'Honda': ['Accord', 'Civic', 'Jazz', 'Odyssey'],
+        'Jeep': ['Cherokee', 'Grand Cherokee', 'Trackhawk', 'Trailhawk'],
+        'Nissan': ['350Z', 'Maxima', 'Navara', 'Pulsar'],
+    }
+    assert actual == expected