diff --git a/tests/test_wave_01.py b/tests/test_wave_01.py index 669efee6..f41325b9 100644 --- a/tests/test_wave_01.py +++ b/tests/test_wave_01.py @@ -4,7 +4,7 @@ from viewing_party.party import * from tests.test_constants import * -@pytest.mark.skip() +# @pytest.mark.skip() def test_create_successful_movie(): # Arrange movie_title = MOVIE_TITLE_1 @@ -19,7 +19,7 @@ def test_create_successful_movie(): assert new_movie["genre"] == GENRE_1 assert new_movie["rating"] == pytest.approx(RATING_1) -@pytest.mark.skip() +# @pytest.mark.skip() def test_create_no_title_movie(): # Arrange movie_title = None @@ -32,20 +32,21 @@ def test_create_no_title_movie(): # Assert assert new_movie is None -@pytest.mark.skip() +# @pytest.mark.skip() def test_create_no_genre_movie(): # Arrange movie_title = "Title A" genre = None rating = 3.5 + # Act new_movie = create_movie(movie_title, genre, rating) # Assert assert new_movie is None -@pytest.mark.skip() +# @pytest.mark.skip() def test_create_no_rating_movie(): # Arrange movie_title = "Title A" @@ -58,7 +59,7 @@ def test_create_no_rating_movie(): # Assert assert new_movie is None -@pytest.mark.skip() +# @pytest.mark.skip() def test_adds_movie_to_user_watched(): # Arrange movie = { @@ -79,7 +80,7 @@ def test_adds_movie_to_user_watched(): assert updated_data["watched"][0]["genre"] == GENRE_1 assert updated_data["watched"][0]["rating"] == RATING_1 -@pytest.mark.skip() +# @pytest.mark.skip() def test_adds_movie_to_non_empty_user_watched(): # Arrange movie = { @@ -99,7 +100,7 @@ def test_adds_movie_to_non_empty_user_watched(): assert movie in updated_data["watched"] assert FANTASY_2 in updated_data["watched"] -@pytest.mark.skip() +# @pytest.mark.skip() def test_adds_movie_to_user_watchlist(): # Arrange movie = { @@ -120,7 +121,7 @@ def test_adds_movie_to_user_watchlist(): assert updated_data["watchlist"][0]["genre"] == GENRE_1 assert updated_data["watchlist"][0]["rating"] == RATING_1 -@pytest.mark.skip() +# @pytest.mark.skip() def test_adds_movie_to_non_empty_user_watchlist(): # Arrange movie = { @@ -140,7 +141,7 @@ def test_adds_movie_to_non_empty_user_watchlist(): assert movie in updated_data["watchlist"] assert FANTASY_2 in updated_data["watchlist"] -@pytest.mark.skip() +# @pytest.mark.skip() def test_moves_movie_from_watchlist_to_empty_watched(): # Arrange janes_data = { @@ -151,6 +152,14 @@ def test_moves_movie_from_watchlist_to_empty_watched(): }], "watched": [] } + expected_result = { + "watchlist": [], + "watched": [{ + "title": MOVIE_TITLE_1, + "genre": GENRE_1, + "rating": RATING_1 + }] + } # Act updated_data = watch_movie(janes_data, MOVIE_TITLE_1) @@ -158,13 +167,14 @@ def test_moves_movie_from_watchlist_to_empty_watched(): # Assert assert len(updated_data["watchlist"]) == 0 assert len(updated_data["watched"]) == 1 - - raise Exception("Test needs to be completed.") + assert updated_data == expected_result + + # raise Exception("Test needs to be completed.") # ******************************************************************************************* # ****** Add assertions here to test that the correct movie was added to "watched" ********** # ******************************************************************************************* -@pytest.mark.skip() +# @pytest.mark.skip() def test_moves_movie_from_watchlist_to_watched(): # Arrange movie_to_watch = HORROR_1 @@ -175,6 +185,13 @@ def test_moves_movie_from_watchlist_to_watched(): ], "watched": [FANTASY_2] } + expected_result = { + "watchlist": [ + FANTASY_1 + + ], + "watched": [FANTASY_2, movie_to_watch] + } # Act updated_data = watch_movie(janes_data, movie_to_watch["title"]) @@ -182,13 +199,14 @@ def test_moves_movie_from_watchlist_to_watched(): # Assert assert len(updated_data["watchlist"]) == 1 assert len(updated_data["watched"]) == 2 - - raise Exception("Test needs to be completed.") + assert updated_data == expected_result + + # raise Exception("Test needs to be completed.") # ******************************************************************************************* # ****** Add assertions here to test that the correct movie was added to "watched" ********** # ******************************************************************************************* -@pytest.mark.skip() +# @pytest.mark.skip() def test_does_nothing_if_movie_not_in_watchlist(): # Arrange movie_to_watch = HORROR_1 diff --git a/tests/test_wave_02.py b/tests/test_wave_02.py index 19f045c7..64b62729 100644 --- a/tests/test_wave_02.py +++ b/tests/test_wave_02.py @@ -2,7 +2,7 @@ from viewing_party.party import * from tests.test_constants import * -@pytest.mark.skip() +# @pytest.mark.skip() def test_calculates_watched_average_rating(): # Arrange janes_data = clean_wave_2_data() @@ -14,7 +14,7 @@ def test_calculates_watched_average_rating(): assert average == pytest.approx(3.58333) assert janes_data == clean_wave_2_data() -@pytest.mark.skip() +# @pytest.mark.skip() def test_empty_watched_average_rating_is_zero(): # Arrange janes_data = { @@ -27,7 +27,7 @@ def test_empty_watched_average_rating_is_zero(): # Assert assert average == pytest.approx(0.0) -@pytest.mark.skip() +# @pytest.mark.skip() def test_most_watched_genre(): # Arrange janes_data = clean_wave_2_data() @@ -39,7 +39,7 @@ def test_most_watched_genre(): assert popular_genre == "Fantasy" assert janes_data == clean_wave_2_data() -@pytest.mark.skip() +# @pytest.mark.skip() def test_most_watched_genre_order_mixed(): # Arrange janes_data = clean_wave_2b_data() @@ -51,7 +51,7 @@ def test_most_watched_genre_order_mixed(): assert popular_genre == "Fantasy" assert janes_data == clean_wave_2b_data() -@pytest.mark.skip() +# @pytest.mark.skip() def test_genre_is_None_if_empty_watched(): # Arrange janes_data = { diff --git a/tests/test_wave_03.py b/tests/test_wave_03.py index 04642936..73cceb1e 100644 --- a/tests/test_wave_03.py +++ b/tests/test_wave_03.py @@ -2,7 +2,7 @@ from viewing_party.party import * from tests.test_constants import * -@pytest.mark.skip() +# @pytest.mark.skip() def test_my_unique_movies(): # Arrange amandas_data = clean_wave_3_data() @@ -16,7 +16,7 @@ def test_my_unique_movies(): assert INTRIGUE_2 in amandas_unique_movies assert amandas_data == clean_wave_3_data() -@pytest.mark.skip() +# @pytest.mark.skip() def test_my_not_unique_movies(): # Arrange amandas_data = clean_wave_3_data() @@ -28,7 +28,7 @@ def test_my_not_unique_movies(): # Assert assert len(amandas_unique_movies) == 0 -@pytest.mark.skip() +# @pytest.mark.skip() def test_friends_unique_movies(): # Arrange amandas_data = clean_wave_3_data() @@ -43,24 +43,35 @@ def test_friends_unique_movies(): assert FANTASY_4 in friends_unique_movies assert amandas_data == clean_wave_3_data() -@pytest.mark.skip() +# @pytest.mark.skip() def test_friends_unique_movies_not_duplicated(): # Arrange amandas_data = clean_wave_3_data() amandas_data["friends"][0]["watched"].append(INTRIGUE_3) - + # Act friends_unique_movies = get_friends_unique_watched(amandas_data) + + INTRIGUE_3_count = 0 + for movie in friends_unique_movies: + if movie == INTRIGUE_3: + INTRIGUE_3_count += 1 + # Assert assert len(friends_unique_movies) == 3 + assert HORROR_1 in friends_unique_movies + assert INTRIGUE_3 in friends_unique_movies + assert FANTASY_4 in friends_unique_movies + assert INTRIGUE_3_count == 1 + - raise Exception("Test needs to be completed.") + # raise Exception("Test needs to be completed.") # ************************************************************************************************* # ****** Add assertions here to test that the correct movies are in friends_unique_movies ********** # ************************************************************************************************** -@pytest.mark.skip() +# @pytest.mark.skip() def test_friends_not_unique_movies(): # Arrange amandas_data = { diff --git a/tests/test_wave_04.py b/tests/test_wave_04.py index 49966907..0b0b3c7e 100644 --- a/tests/test_wave_04.py +++ b/tests/test_wave_04.py @@ -2,7 +2,7 @@ from viewing_party.party import * from tests.test_constants import * -@pytest.mark.skip() +# @pytest.mark.skip() def test_get_available_friend_rec(): # Arrange amandas_data = clean_wave_4_data() @@ -16,7 +16,7 @@ def test_get_available_friend_rec(): assert FANTASY_4b in recommendations assert amandas_data == clean_wave_4_data() -@pytest.mark.skip() +# @pytest.mark.skip() def test_no_available_friend_recs(): # Arrange amandas_data = { @@ -38,7 +38,7 @@ def test_no_available_friend_recs(): # Assert assert len(recommendations) == 0 -@pytest.mark.skip() +# @pytest.mark.skip() def test_no_available_friend_recs_watched_all(): # Arrange amandas_data = { diff --git a/tests/test_wave_05.py b/tests/test_wave_05.py index b2ba9ad3..e9c0ab57 100644 --- a/tests/test_wave_05.py +++ b/tests/test_wave_05.py @@ -2,7 +2,7 @@ from viewing_party.party import * from tests.test_constants import * -@pytest.mark.skip() +# @pytest.mark.skip() def test_new_genre_rec(): # Arrange sonyas_data = clean_wave_5_data() @@ -17,7 +17,7 @@ def test_new_genre_rec(): assert FANTASY_4b in recommendations assert sonyas_data == clean_wave_5_data() -@pytest.mark.skip() +# @pytest.mark.skip() def test_new_genre_rec_from_empty_watched(): # Arrange sonyas_data = { @@ -38,7 +38,7 @@ def test_new_genre_rec_from_empty_watched(): # Assert assert len(recommendations) == 0 -@pytest.mark.skip() +# @pytest.mark.skip() def test_new_genre_rec_from_empty_friends(): # Arrange sonyas_data = { @@ -52,13 +52,21 @@ def test_new_genre_rec_from_empty_friends(): } ] } + expected_result = [] + # Act + recommendations = get_new_rec_by_genre(sonyas_data) + + # Assert + assert not recommendations + assert len(recommendations) == 0 + - raise Exception("Test needs to be completed.") + # raise Exception("Test needs to be completed.") # ********************************************************************* # ****** Complete the Act and Assert Portions of these tests ********** # ********************************************************************* -@pytest.mark.skip() +# @pytest.mark.skip() def test_unique_rec_from_favorites(): # Arrange sonyas_data = clean_wave_5_data() @@ -72,7 +80,7 @@ def test_unique_rec_from_favorites(): assert INTRIGUE_2b in recommendations assert sonyas_data == clean_wave_5_data() -@pytest.mark.skip() +# @pytest.mark.skip() def test_unique_from_empty_favorites(): # Arrange sonyas_data = { @@ -94,7 +102,7 @@ def test_unique_from_empty_favorites(): # Assert assert len(recommendations) == 0 -@pytest.mark.skip() +# @pytest.mark.skip() def test_new_rec_from_empty_friends(): # Arrange sonyas_data = { diff --git a/viewing_party/party.py b/viewing_party/party.py index 6d34a6b5..2efa5c96 100644 --- a/viewing_party/party.py +++ b/viewing_party/party.py @@ -1,23 +1,154 @@ +import copy # ------------- WAVE 1 -------------------- def create_movie(title, genre, rating): - pass + if not title or not genre or not rating: + return None + new_movie = { + "title": title, + "genre": genre, + "rating": rating + } + return new_movie + +def add_to_watched(user_data, movie): + user_data["watched"].append(movie) + + + return user_data + +def add_to_watchlist(user_data, movie): + user_data["watchlist"].append(movie) + + return user_data + + +def watch_movie(user_data, title): + new_watchlist = [] + new_watched = [] + + for movie in user_data["watchlist"]: + new_watchlist.append(movie) + + for movie in user_data["watched"]: + new_watched.append(movie) + + for movie in new_watchlist: + if movie["title"] == title: + new_watched.append(movie) + new_watchlist.remove(movie) + break + + updated_user_data = { + "watchlist": new_watchlist, + "watched": new_watched + } + + return updated_user_data # ----------------------------------------- # ------------- WAVE 2 -------------------- # ----------------------------------------- +def get_watched_avg_rating(user_data): + if not len(user_data["watched"]): + return 0.0 + sum = 0 + for movie in user_data["watched"]: + sum += movie["rating"] + + average = sum / len(user_data["watched"]) + return average + +def get_most_watched_genre(user_data): + if not user_data: + return None + + genres = {} + for movie in user_data["watched"]: + movie_genre = movie["genre"] + count = genres.get(movie_genre, 0) + genres[movie_genre] = count + 1 + entry = None + max_number_of_occurences = 0 + + for genre, count in genres.items(): + if count > max_number_of_occurences: + entry = genre + max_number_of_occurences = count + + return entry # ----------------------------------------- # ------------- WAVE 3 -------------------- # ----------------------------------------- +def get_unique_watched(user_data): + friend_movie = [] + + for friend_list in user_data["friends"]: + for movie in friend_list["watched"]: + if movie not in friend_movie: + friend_movie.append(movie) + + unique_user_movie = [] + + for movie in user_data["watched"]: + if movie not in friend_movie: + unique_user_movie.append(movie) + return unique_user_movie + + +def get_friends_unique_watched(user_data): + unique_user_movie = [] + + for movie in user_data["watched"]: + if movie not in unique_user_movie: + unique_user_movie.append(movie) + + friend_unique_movie = [] + + for friend_list in user_data["friends"]: + for movie in friend_list["watched"]: + if movie not in unique_user_movie and movie not in friend_unique_movie: + friend_unique_movie.append(movie) + + return friend_unique_movie - # ----------------------------------------- # ------------- WAVE 4 -------------------- # ----------------------------------------- + +def get_available_recs(user_data): + friend_unique_movie = get_friends_unique_watched(user_data) + rec_movies = [] + for movie in friend_unique_movie: + if movie["host"] in user_data["subscriptions"]: + rec_movies.append(movie) + return rec_movies # ----------------------------------------- # ------------- WAVE 5 -------------------- # ----------------------------------------- + +def get_new_rec_by_genre(user_data): + rec_movies = [] + most_watched_genre = get_most_watched_genre(user_data) + friend_unique_movie = get_friends_unique_watched(user_data) + + for movie in friend_unique_movie: + if most_watched_genre == movie["genre"]: + rec_movies.append(movie) + + return rec_movies + + +def get_rec_from_favorites(user_data): + unique_watched_movie = get_unique_watched(user_data) + fav_titles = {movie["title"] for movie in user_data["favorites"]} + + + rec_movies = [movie for movie in unique_watched_movie if movie["title"] in fav_titles] + + + return rec_movies