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

refactor and test voucher #480

Merged
merged 8 commits into from
May 28, 2023
Merged

refactor and test voucher #480

merged 8 commits into from
May 28, 2023

Conversation

fremartini
Copy link
Member

No description provided.

@fremartini fremartini marked this pull request as draft May 27, 2023 06:49
@fremartini fremartini requested a review from a team May 27, 2023 06:49
@ghost
Copy link

ghost commented May 27, 2023

👇 Click on the image for a new way to code review

Review these changes using an interactive CodeSee Map

Legend

CodeSee Map legend

@codecov
Copy link

codecov bot commented May 27, 2023

Codecov Report

Merging #480 (3831652) into develop (bae775e) will increase coverage by 1.39%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           develop     #480      +/-   ##
===========================================
+ Coverage    70.80%   72.19%   +1.39%     
===========================================
  Files          104      108       +4     
  Lines         1137     1158      +21     
===========================================
+ Hits           805      836      +31     
+ Misses         332      322      -10     
Flag Coverage Δ
unittests 72.19% <100.00%> (+1.39%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...ures/voucher/domain/entities/redeemed_voucher.dart 100.00% <ø> (ø)
...ures/voucher/presentation/cubit/voucher_state.dart 100.00% <ø> (ø)
...r/data/datasources/voucher_remote_data_source.dart 80.00% <100.00%> (ø)
...es/voucher/data/models/redeemed_voucher_model.dart 100.00% <100.00%> (ø)
...s/voucher/domain/usecases/redeem_voucher_code.dart 100.00% <100.00%> (ø)
...ures/voucher/presentation/cubit/voucher_cubit.dart 100.00% <100.00%> (ø)

@fremartini fremartini marked this pull request as ready for review May 27, 2023 07:00
@fremartini fremartini enabled auto-merge (squash) May 27, 2023 07:04
@fremartini fremartini requested a review from marfavi May 28, 2023 06:25
marfavi
marfavi previously approved these changes May 28, 2023
marfavi
marfavi previously approved these changes May 28, 2023
Copy link
Member

@marfavi marfavi left a comment

Choose a reason for hiding this comment

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

If you use the 'Update branch' button on GitHub, you won't invalidate any reviews :)

@fremartini
Copy link
Member Author

@marfavi merged develop again. Can't Update Branch from here because merge conflicts

Copy link
Member

@marfavi marfavi left a comment

Choose a reason for hiding this comment

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

All good

@fremartini fremartini merged commit 7505c32 into develop May 28, 2023
@fremartini fremartini deleted the frem/refactor_voucher branch May 28, 2023 07:09
@marfavi marfavi mentioned this pull request May 28, 2023
marfavi added a commit that referenced this pull request May 30, 2023
# New features
* show status on receipts (#468)
* show timestamp on receipt list entry (#478)

# Fixes
* Fix showing raw html in error messages (#463)
* fix(tickets): Fix multiple `CoffeeCard`s sharing the same product id (#471)
* fix: Avoid type casting 204 return types from API (#479)
* fix(receipts): Remove purchase receipt view & refactor payment status display (#478)
* Added awaits for useTicket and FetchReceipts (#464)

# Code refactoring and added tests
* test(utils): Improve String.capitalize() and add tests (#472)
* Refactor and test Leaderboard (#443)
* Refactor payment (#465)
* Refactor environment logic (#469)
* refactor(settings): tidy up `settings_page.dart` (#473)
* Restructure settings (#481)
* Refactor and test login (#483)
* refactor and test voucher (#480)
* Refactor and test register (#486)
* Refactor and test products (#482)

# Chores
* chore: update Bloc packages (#475)
* chore: bump flutter required version to 3.10.2 (#476)
* Migrate to Dart 3 and Flutter 3.10.0 (#452)

# Developer related
* migrate to fpdart (#457)
* Install the CodeSee workflow (#467)
* Update README.md to show coverage on develop branch (#484)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants