Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sapphire - Mel M. & Jen T. #50

Open
wants to merge 49 commits into
base: main
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
49 commits
Select commit Hold shift + click to select a range
78eceb5
READ ME was changed
MelMott Mar 27, 2023
1874fc5
edited README.md
JenniferWTam Mar 27, 2023
45fba94
Changed test watchlist to empty watched
MelMott Mar 27, 2023
a68e40e
changes to test moves from watchlist to watched
MelMott Mar 27, 2023
ce827ea
Wave 2 - 2 failed/3 passed
JenniferWTam Mar 28, 2023
7e874ab
Wave 2 passed all tests
JenniferWTam Mar 28, 2023
0f3fb4b
Started create movie wave 1
MelMott Mar 28, 2023
30096ab
Merge branch 'main' of https://github.com/MelMott/viewing-party
MelMott Mar 28, 2023
62dcb92
Passed create movie tests and reskipped tests
MelMott Mar 28, 2023
192b564
added notes to Wave 2
JenniferWTam Mar 28, 2023
b8a069c
Merge branch 'main' of https://github.com/MelMott/viewing-party
JenniferWTam Mar 28, 2023
4e8d6b8
create movie passed all tests
MelMott Mar 28, 2023
1c5e152
Merge branch 'main' of https://github.com/MelMott/viewing-party
JenniferWTam Mar 28, 2023
6a6728e
completed add to watched function
MelMott Mar 28, 2023
4a278a1
Finished third function and passed tests
MelMott Mar 28, 2023
90d2279
Finished last function watchmovie & assertions fix
MelMott Mar 28, 2023
55309e2
started Wave 3
JenniferWTam Mar 28, 2023
9c9f181
Merge branch 'main' of https://github.com/MelMott/viewing-party
JenniferWTam Mar 28, 2023
9827d54
edited test comments for 1
JenniferWTam Mar 28, 2023
9c78cfb
Started wave 4 errors
MelMott Mar 28, 2023
2cb342e
additions made to Wave 3
JenniferWTam Mar 28, 2023
7ccd116
changes made to Wave 3
JenniferWTam Mar 28, 2023
98f8a75
updated wave 4, not done
MelMott Mar 29, 2023
8fdd413
Wave 3 4 passed/1 failed
JenniferWTam Mar 29, 2023
9e80530
Merge branch 'main' of https://github.com/MelMott/viewing-party
JenniferWTam Mar 29, 2023
705628d
passing 2 tests wave 4
MelMott Mar 29, 2023
4c035bb
Wave 3 passed all tests. yayyyy
JenniferWTam Mar 29, 2023
7507e51
Merge branch 'main' of https://github.com/MelMott/viewing-party
JenniferWTam Mar 29, 2023
8a63e96
Wave 5 passed 2 tests / left earlier implementations commented
JenniferWTam Mar 30, 2023
ce10bf7
edited out earlier implementations
JenniferWTam Mar 30, 2023
cb2e98e
added assertion to Wave 5 test
JenniferWTam Mar 30, 2023
a243ae8
wrote second function for Wave 5 // 2 failed tests
JenniferWTam Mar 30, 2023
17352af
edited notes in Wave 5
JenniferWTam Mar 30, 2023
2e2574c
alternate implementation for Wave 2 without using multimode
JenniferWTam Mar 30, 2023
f9d9a14
edited super long implementation for second function of Wave 2
JenniferWTam Mar 30, 2023
a165862
Made more progress on wave 4
MelMott Mar 30, 2023
5e20bd5
Merge branch 'main' of https://github.com/MelMott/viewing-party
MelMott Mar 30, 2023
13bae3b
Finished wave 4 test passed
MelMott Mar 30, 2023
bbe931c
Wave 5 passed all tests
JenniferWTam Mar 30, 2023
f2044b6
Merge branch 'main' of https://github.com/MelMott/viewing-party
JenniferWTam Mar 30, 2023
7ae2262
edited notes
JenniferWTam Mar 30, 2023
04f78a8
Refactored & made it DRY-er
MelMott Mar 30, 2023
57ae7e2
edited notes
JenniferWTam Mar 30, 2023
77ba054
doing .get
MelMott Mar 30, 2023
44d99ac
Never mind
MelMott Mar 30, 2023
8109e1b
Revert "Never mind"
MelMott Mar 30, 2023
682d9ac
Revert "doing .get"
MelMott Mar 30, 2023
c15cde3
Did doc strings!
MelMott Mar 30, 2023
929910a
edited docstrings
JenniferWTam Mar 30, 2023
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 11 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
@@ -1,4 +1,15 @@
# Viewing Party
## MEL & JEN

#Pair Plan:
Access needs - In Person and then if needed zoom/liveshare. (Possibly evening works for both of us for zoom)

Your learning style - Getting errors and failed tests, also making print tests. Comments to help catch what the code is doing behind the scenes.

How you prefer to receive feedback - Communicating ideas to each other and finding common ground or compromise for ideas that we are split on.

One team communication skill you want to improve with this experience - New to team collaboration so we want to see what we might need to improve and what we excel at.


## Skills Assessed

Expand Down
10 changes: 10 additions & 0 deletions tests/test_constants.py
Original file line number Diff line number Diff line change
Expand Up @@ -91,6 +91,16 @@

#-----WAVE 3--------
USER_DATA_3 = copy.deepcopy(USER_DATA_2)
# USER_DATA_2 = {
# "watched": [
# FANTASY_1,
# FANTASY_2,
# FANTASY_3,
# ACTION_1,
# INTRIGUE_1,
# INTRIGUE_2
# ],
# }
USER_DATA_3["friends"] = [
{
"watched": [
Expand Down
35 changes: 22 additions & 13 deletions tests/test_wave_01.py
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand All @@ -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
Expand All @@ -32,7 +32,7 @@ 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"
Expand All @@ -45,7 +45,7 @@ def test_create_no_genre_movie():
# Assert
assert new_movie is None

@pytest.mark.skip()
#@pytest.mark.skip()
def test_create_no_rating_movie():
# Arrange
movie_title = "Title A"
Expand All @@ -58,7 +58,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 = {
Expand All @@ -79,7 +79,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 = {
Expand All @@ -99,7 +99,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 = {
Expand All @@ -120,7 +120,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 = {
Expand All @@ -140,7 +140,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 = {
Expand All @@ -159,12 +159,14 @@ def test_moves_movie_from_watchlist_to_empty_watched():
assert len(updated_data["watchlist"]) == 0
assert len(updated_data["watched"]) == 1

raise Exception("Test needs to be completed.")
assert updated_data["watched"][0]["title"] == MOVIE_TITLE_1
assert updated_data["watched"][0]["genre"] == GENRE_1
assert updated_data["watched"][0]["rating"] == RATING_1
# *******************************************************************************************
# ****** 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
Expand All @@ -182,13 +184,20 @@ def test_moves_movie_from_watchlist_to_watched():
# Assert
assert len(updated_data["watchlist"]) == 1
assert len(updated_data["watched"]) == 2

# Had to fix the assertions
assert updated_data["watchlist"][0] == FANTASY_1
assert movie_to_watch in updated_data["watched"]

raise Exception("Test needs to be completed.")
# Old assertions
#assert watchlist in updated_data["watched"]
# assert FANTASY_1 in updated_data["watchlist"]

# *******************************************************************************************
# ****** 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
Expand Down
10 changes: 5 additions & 5 deletions tests/test_wave_02.py
Original file line number Diff line number Diff line change
Expand Up @@ -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()
Expand All @@ -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 = {
Expand All @@ -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()
Expand All @@ -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()
Expand All @@ -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 = {
Expand Down
14 changes: 8 additions & 6 deletions tests/test_wave_03.py
Original file line number Diff line number Diff line change
Expand Up @@ -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()
Expand All @@ -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()
Expand All @@ -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()
Expand All @@ -43,7 +43,7 @@ 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()
Expand All @@ -54,13 +54,15 @@ def test_friends_unique_movies_not_duplicated():

# Assert
assert len(friends_unique_movies) == 3
assert INTRIGUE_3 in friends_unique_movies
assert HORROR_1 in friends_unique_movies
assert FANTASY_4 in friends_unique_movies

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 = {
Expand Down
6 changes: 3 additions & 3 deletions tests/test_wave_04.py
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
from viewing_party.party import *
from tests.test_constants import *

@pytest.mark.skip()

def test_get_available_friend_rec():
# Arrange
amandas_data = clean_wave_4_data()
Expand All @@ -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()

def test_no_available_friend_recs():
# Arrange
amandas_data = {
Expand All @@ -38,7 +38,7 @@ def test_no_available_friend_recs():
# Assert
assert len(recommendations) == 0

@pytest.mark.skip()

def test_no_available_friend_recs_watched_all():
# Arrange
amandas_data = {
Expand Down
17 changes: 10 additions & 7 deletions tests/test_wave_05.py
Original file line number Diff line number Diff line change
Expand Up @@ -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()
Expand All @@ -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 = {
Expand All @@ -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 = {
Expand All @@ -53,12 +53,15 @@ def test_new_genre_rec_from_empty_friends():
]
}

raise Exception("Test needs to be completed.")
recommendations = get_new_rec_by_genre(sonyas_data)

assert len(recommendations) == 0

# *********************************************************************
# ****** 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()
Expand All @@ -72,7 +75,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 = {
Expand All @@ -94,7 +97,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 = {
Expand Down
Loading