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

Gateway tests #333

Merged
merged 90 commits into from
Mar 11, 2024
Merged

Gateway tests #333

merged 90 commits into from
Mar 11, 2024

Conversation

lotuuu
Copy link
Contributor

@lotuuu lotuuu commented Feb 26, 2024 •
edited
Loading

Implement some basic integration tests. Tests should be expanded later but this serves as a sort of template for them in the future.

To test

Run mix test. No further setup should be needed. Let me know if that does not work on your machine!

lotuuu added 30 commits February 23, 2024 16:10
@lotuuu
General fixes to Champions.Units
253f42c
@lotuuu
Add doc to unselect_unit.ex
24414fb
@lotuuu
Refactor units
a6b2da1
@lotuuu
Create GameBackend.Transaction
f112fc4
@lotuuu
Remove Jason.Encoder derive
39b8e6d
@lotuuu
Add CurrencyCost
59714c5
@lotuuu
Add Currency operations
0cee293
@lotuuu
Add game_id util
f3d7a41
@lotuuu
Set slots range to 1..6
406c7f7
@lotuuu
Return ok tuple in register user
6e0d89e
@lotuuu
Update unit.ex
e9d2080
@lotuuu
Add add_level to GameBackend.Units
a403f23
@lotuuu
Implement Champions unit level up
a3c85d8 
- Also create CurrencyCost
@lotuuu
Run format after protos
782b15b
@lotuuu
Add gateway commands for new functionalities
6b84d44
@lotuuu
Format Champions.Units
49c4a1e
@lotuuu
Add doc
5805b10
@lotuuu
Add ChampionsSocketHandler handlers
7b3a5aa
@lotuuu
Update SocketTester
4988803
@lotuuu
Fix rarity type
bb2b729
@lotuuu
Rename unit_level to level and level assoc to campaign_level
062905c
@lotuuu
Revert "Rename unit_level to level and level assoc to campaign_level"
2519f22 
This reverts commit 062905c.
@lotuuu
Rename unit_level to level and level assoc to campaign_level
0263347
@lotuuu
Rename update_selected
2a12d4c
@lotuuu
Fix selection
8ccbec9
@lotuuu
Merge branch 'main' into unit-level-up
9d76bae
@lotuuu
Merge branch 'unit-level-up' into rename-unit-attributes
d967e7e
@lotuuu
Format
79ce702
@lotuuu
Merge branch 'unit-level-up' into unit-tier-up
b576dc4
@lotuuu
Add rank to unit
7201a8d
Base automatically changed from unit-fusion to main March 7, 2024 21:48
@lotuuu lotuuu marked this pull request as ready for review March 7, 2024 21:58
agustinesco
agustinesco approved these changes Mar 11, 2024
View reviewed changes
Copy link
Contributor

@agustinesco agustinesco left a comment

Choose a reason for hiding this comment

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

LGTM! 🪨 great testing

ncontinanza
ncontinanza approved these changes Mar 11, 2024
View reviewed changes
Copy link
Contributor

@ncontinanza ncontinanza left a comment

Choose a reason for hiding this comment

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

LGTM!

@ncontinanza ncontinanza merged commit 1d6b38a into main Mar 11, 2024
1 check passed
@ncontinanza ncontinanza deleted the gateway-tests branch March 11, 2024 18:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@agustinesco agustinesco agustinesco approved these changes

@ncontinanza ncontinanza ncontinanza approved these changes

Assignees

@lotuuu lotuuu

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@lotuuu @ncontinanza @agustinesco