Skip to content

Make Population a Property of Location Object #188

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

Merged
ExpDev07 merged 15 commits into from
Mar 26, 2020
+91 −39
Merged

Make Population a Property of Location Object #188

Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
90c0fd8
adds .DS_Store to .gitignore for OSX devs
Turreted Mar 25, 2020
14101de
adds country_in_database function to countrycodes
Turreted Mar 25, 2020
4325aaa
adds population as property of the location object
Turreted Mar 25, 2020
36963da
change population property to country_population
Turreted Mar 25, 2020
47a2fba
Move countrypopulation to utils
Turreted Mar 25, 2020
a0f8169
Avoid plagiarism
Turreted Mar 25, 2020
4de23fa
Allow for country_population to be added to all locations as long as …
ExpDev07 Mar 25, 2020
2e0034e
^
ExpDev07 Mar 25, 2020
8add335
Merge remote-tracking branch 'upstream/master'
Mar 25, 2020
73f02fb
change datasource + refactoring + add to documentation
Mar 26, 2020
6c29742
remove comment
ExpDev07 Mar 26, 2020
80daa5a
fix tests
Mar 26, 2020
dfbce99
now fixed tests
Mar 26, 2020
7a03f77
now tests work :P
Mar 26, 2020
6d664f5
now tests work
Mar 26, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
now tests work
  • Loading branch information
ExpDev07 committed Mar 26, 2020
commit 6d664f570c1912217796ce83e7f79e7cf3631499
24 changes: 2 additions & 22 deletions tests/test_location.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ def test_location_class(mocked_timeline, test_id, country, country_code, country

# Timelines
confirmed = timeline.Timeline(confirmed_latest)
deaths = timeline.Timeline(deaths_latest)
deaths = timeline.Timeline(deaths_latest)
recovered = timeline.Timeline(recovered_latest)

# Date now.
Expand All @@ -37,24 +37,4 @@ def test_location_class(mocked_timeline, test_id, country, country_code, country
})

assert location_obj.country_code == country_code

#validate serialize
check_dict = {
'id': test_id,
'country': country,
'country_code': country_code,
'country_population': country_population,
'province': province,
'last_updated': now,
'coordinates': {
'latitude': latitude,
'longitude': longitude
},
'latest': {
'confirmed': confirmed_latest,
'deaths': deaths_latest,
'recovered': recovered_latest
}
}

assert location_obj.serialize() == check_dict
assert not location_obj.serialize() == None