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

Testing #39

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

Testing #39

wants to merge 119 commits into from

Conversation

Alystrasz
Copy link
Member

@Alystrasz Alystrasz commented Jun 15, 2021 •
edited
Loading

  • tests cover application code;
  • those run on CI through a test script;
  • Android and iOS packages are not built if tests fail;
  • coverage report is uploaded at the end of the CI run;
  • introduces required object properties.

@Alystrasz
[chore] adding mockito dependency
ddbefbb
@Alystrasz
[refactor] popular times parser is no more static + accepts different…
a007e31 
… http clients
@Alystrasz
[refactor] making a parser method static no more
256e43e
@Alystrasz
[test] parser can parse popular times data from an API file
ab05890
@Alystrasz
[test] parser can check if a given place has no popular times data
8b16cb9
@Alystrasz
[refactor] reorganizing populartimes test package
5d27207
@Alystrasz
[fix] making Favorite.id required field
776ae1b 
A Favorite instance without id would be stored in app caches under
"null" key, which would not raise errors but provoke unwanted
behaviors.
@Alystrasz
[fix] removing useless imports
7cd3452
@Alystrasz
[test] popular times parser cache can store different places' stats
aca615d
@Alystrasz
[refactor] moving parser cache in correct package
860c4f4
@Alystrasz
[refactor] indicators computer now accepts a database as constructor …
1b8d1e8 
…parameter
@Alystrasz
[refactor] transforming database_test into indicators_computer_test
1922278
@Alystrasz
[refactor] exporting reports loading into a dedicated method
dea34e3
@Alystrasz
[fix] making all DailyReport fields required
66580a8
@Alystrasz
[test] indicators computer can store today's report
ce4470a
@Alystrasz
[test] indicators computer can update today's report
6dbfa4d
@Alystrasz
[test] computer can update today's exposition rate
afbbf7d
@Alystrasz
[test] computer does not update today's exposition rate if report doe…
227ded5 
…sn't exist
@Alystrasz
[test] checking exposition rate value before updating it
1f2eed2
@Alystrasz
[test] adding a coverage badge to README
edd9c86
@Alystrasz
[docs] updating README with test instructions
f5a10a1
@Alystrasz
[chore] adding a script to run tests + generate coverage badge
a5cd6c7
@Alystrasz
[test] computer can update today's broadcast rate
e70faa4
@Alystrasz
[test] also checking if side indicators are affected by methods
72f71df
@Alystrasz
[docs] adding TODO regarding provider testing
359521a
@Alystrasz
[test] adding basic init test for popularity points cache
44fde09
@Alystrasz
[fix] making PlacesAPIResult.placeId a required field
994bb84
@Alystrasz
[fix] making PlacesAPIResult.location a required field
323ed60
@Alystrasz
[test] popularity points cache correctly store points
ec5c68c
@Alystrasz
[test] popularity points cache returns same points if zoom and popula…
4de29e6 
…rity levels didn't change
Alystrasz and others added 27 commits September 25, 2020 12:49
@Alystrasz
[test] fixing a missing parameter
a522c46
@Alystrasz
[docs] updating coverage information
f97dcd4
@Alystrasz
[test] checking if AppModel has API key
d5a14d2
@Alystrasz
[test] checking AppModel's parser
4c1b528
@Alystrasz
[test] cleaning DayResult inputs
aa9859d
@Alystrasz
[test] PopularTimesParser can store + retrieve data in cache
017316c
@Alystrasz
[fix] adding missing inputs to day results
a3279d4
@Alystrasz
[docs] generating coverage information
7cf4ff3
@Alystrasz
Merge branch 'master' of github.com:pandemia-app/pandemia into test/t…
b7a0130 
…ests
@Alystrasz
[chore] fixing test script carriage returns
9e337d6
@Alystrasz
[chore] not generating html coverage if lcov is not installed
7a5147b
@Alystrasz
[chore] moving coverage badge to coverage folder
477fb1a
@Alystrasz
[chore] renaming tools folder
a44c2bf
@Alystrasz
[chore] CI runs tests script
899ceaf
@Alystrasz
[chore] typo in chmod command
4ac8574
@Alystrasz
[chore] generating .env file in CI test job
39ce26e
@Alystrasz
[chore] setting jobs order
64b85b9
@Alystrasz
[chore] exiting tests script if some do not pass
cc45de5
@Alystrasz
[chore] injecting gmaps key in CI tests
2312725
@Alystrasz
[fix] converting daily report timestamp to UTC when printing
489c6ae
@Alystrasz
[chore] removing useless argument in CI test instruction
3c76b8c
@Alystrasz
[chore] updating CI test job name
1a74173
@Alystrasz
[chore] installing lcov in test CI
84e2887
@Alystrasz
[chore] uploading coverage report from CI test job
7a0bbc3
@Alystrasz
[chore] archiving html coverage
dc7ecd9
@Alystrasz
[chore] uploading correct coverage folder
5af09fa
@Alystrasz
[chore] adding FLUTTER_VERSION variable to CI jobs
a70995c
@Alystrasz Alystrasz requested a review from rouvoy June 15, 2021 07:48
@Alystrasz Alystrasz marked this pull request as ready for review June 15, 2021 10:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rouvoy rouvoy Awaiting requested review from rouvoy

At least 1 approving review is required to merge this pull request.

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
@Alystrasz