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

Update code to support null safety #55

Open
wants to merge 13 commits into
base: master
Choose a base branch
from
Open

Update code to support null safety #55

wants to merge 13 commits into from

Conversation

ngtphat-towa
Copy link

@ngtphat-towa ngtphat-towa commented Oct 10, 2023 •
edited
Loading

This is update code in flutter 3.0.0+ support null safety to fix this #47
data_connection_checker cannot run with sound null safety

I also add some new test stub and generated Mockito test to fix the issue #54

@ngtphat-towa
Update the the code to current flutter version
cf196fb 
Document used:
https://dart.dev/null-safety
https://pub.dev/packages/internet_connection_checker
@ngtphat-towa
Upgrading pre 1.12 Android projects
6d27b9d
@ngtphat-towa
Removed meta's required annotation
f58409a
@ngtphat-towa
Replace Key with nullable key
57e30a9
@ngtphat-towa
Replace data_connection_checker with internet_connection_checker
f8cb23f 
https://pub.dev/packages/internet_connection_checker
@ngtphat-towa
Fix non-nullable instatnce feild must be initialized.
b2b4883
@ngtphat-towa
Replace String url to Uri parse from string
4dc2338 
The version: http: ^0.12.0+3
->  http: ^1.1.0
@ngtphat-towa
Remove deprecated Widget
0c1a14d
@ngtphat-towa
Fixed Widget is a potentially non-nullable type
c491b09
@ngtphat-towa
Add Mock dependencies using Mockito instead of inheriting Mock
1445c89 
https://docs.flutter.dev/cookbook/testing/unit/mocking
@ngtphat-towa
Refactor NumberTriviaBloc constructor
1beacac
@ngtphat-towa
Replace Mock Class with GenerateMocks
0e9aa8e 
Added more stub in Assert in order to success the Test
@ngtphat-towa
Updated tests to use async* for state change tracking in Bloc
f46c907
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@ngtphat-towa