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.

Sign up for GitHub

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

Jump to bottom

Backend - websockets + unit testing for websockets #28

Merged
merged 32 commits into from
Dec 18, 2023
Merged

Backend - websockets + unit testing for websockets #28

merged 32 commits into from
Dec 18, 2023

Conversation

siemieniuk
Copy link
Collaborator

No description provided.

olalyska and others added 30 commits November 26, 2023 15:30
@olalyska
hotfix
a97b17a
@olalyska
Merge branch 'backend' of https://github.com/akai-org/love_letter int…
4f6bcf5 
…o backend
@olalyska
feat: added creating deck and princess card
f148a11
@olalyska
Merge https://github.com/akai-org/love_letter into backend
3322251
@siemieniuk
Add react-router-dom and create basic routing for
73a3f9f 
BrowserPage and LobbyPage
@siemieniuk
LifoQueue, code refactoring, changes points to score
fa67148
@siemieniuk
New constructor for player
8fbee96
@siemieniuk
New method: start_game
e26fad3
@siemieniuk
Convertion Game and Player to dict
0f00fb8
@siemieniuk
Changed to PriorityQueue due to of Chancellor
0f2fd95
@siemieniuk
New interface for adding new players to the game
58c788c
@olalyska
Merge branch 'backend' of https://github.com/akai-org/love_letter int…
c2eed76 
…o backend
@siemieniuk
Introduced type GameStatus
58467d7
@siemieniuk
WS Reconnect; block connect after game start
5fc5467
@siemieniuk
Fix websockets
82fd469
@siemieniuk
Now absolute path starts with src
3079beb
@siemieniuk
Initial isort
7e3eac1
@siemieniuk
base for testing
add9266
@siemieniuk
TestClient
ca3768d
@olalyska
added find_player method
14628f3
@olalyska
Merge branch 'backend' of https://github.com/akai-org/love_letter int…
855b340 
…o backend
@siemieniuk
Separated GameManager instance to src.dependencies
fbc7261
@siemieniuk
Merge branch 'main' into backend
3550e97
@siemieniuk
Dependency "get_manager"; new websocket endpoints
430b629
@siemieniuk
Timed tests
e97449e
@siemieniuk
Test connections
a19e89f
@siemieniuk
Tests for two people with the same identity and three connections whe…
f87b6a3 
…re 2 allowed
@siemieniuk
UseCase: /ws/<client_id>/<password>; /ws/<client_id>
57a3ffa
@siemieniuk
manager.disconnect in place of remove
86d4e55
@siemieniuk
to_dict in place of __dict__; fix create_new_deck; simplify remove_pl…
534d5aa 
…ayer
@mit3r mit3r self-assigned this Dec 13, 2023
mit3r
mit3r approved these changes Dec 13, 2023
View reviewed changes
Copy link
Contributor

@mit3r mit3r left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good

@siemieniuk siemieniuk merged commit 92a5b8f into main Dec 18, 2023
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mit3r mit3r mit3r approved these changes

Assignees

@mit3r mit3r

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@siemieniuk @mit3r @olalyska @MaciejWojt