Skip to content

Commit

Permalink
black fix
Browse files Browse the repository at this point in the history
  • Loading branch information
BibianaC committed Oct 28, 2020
1 parent 075a8b8 commit 6a037a5
Show file tree
Hide file tree
Showing 12 changed files with 56 additions and 24 deletions.
Expand Up @@ -8,7 +8,9 @@ class Command(BaseCommand):

def add_arguments(self, parser):
parser.add_argument(
"--no-prompt", action="store_true", help="Don't prompt before delete",
"--no-prompt",
action="store_true",
help="Don't prompt before delete",
)

def handle(self, *args, **options):
Expand Down
4 changes: 2 additions & 2 deletions datastore/additional_data/sources/find_that_charity.py
Expand Up @@ -7,8 +7,8 @@


class FindThatCharitySource(object):
""" This is responsible for inputting and outputting data from the
FindThatCharity (FTC) organisation info data sources """
"""This is responsible for inputting and outputting data from the
FindThatCharity (FTC) organisation info data sources"""

ADDITIONAL_DATA_KEY = "recipientOrgInfos"

Expand Down
15 changes: 12 additions & 3 deletions datastore/additional_data/sources/geo_lookup.py
Expand Up @@ -17,14 +17,20 @@ class GeoLookupSource(object):
"url_latlong": "https://raw.githubusercontent.com/drkane/geo-lookups/master/lsoa_latlong.csv",
"field_areacode": "lsoa11cd",
"field_areaname": "lsoa11nm",
"field_transforms": {"lad20cd": "ladcd", "lad20nm": "ladnm",},
"field_transforms": {
"lad20cd": "ladcd",
"lad20nm": "ladnm",
},
},
"msoa": {
"url_lookup": "https://raw.githubusercontent.com/drkane/geo-lookups/master/msoa_la.csv",
"url_latlong": "https://raw.githubusercontent.com/drkane/geo-lookups/master/msoa_latlong.csv",
"field_areacode": "msoa11cd",
"field_areaname": "msoa11hclnm",
"field_transforms": {"lad20cd": "ladcd", "lad20nm": "ladnm",},
"field_transforms": {
"lad20cd": "ladcd",
"lad20nm": "ladnm",
},
},
"ward": {
"url_lookup": "https://raw.githubusercontent.com/drkane/geo-lookups/master/ward_all_codes.csv",
Expand All @@ -36,7 +42,10 @@ class GeoLookupSource(object):
"url_lookup": "https://raw.githubusercontent.com/drkane/geo-lookups/master/la_all_codes.csv",
"field_areacode": "ladcd",
"field_areaname": "ladnm",
"field_transforms": {"lad20cd": "ladcd", "lad20nm": "ladnm",},
"field_transforms": {
"lad20cd": "ladcd",
"lad20nm": "ladnm",
},
},
}

Expand Down
12 changes: 6 additions & 6 deletions datastore/db/management/commands/create_datagetter_data.py
Expand Up @@ -31,13 +31,13 @@ def add_arguments(self, parser):
)

def handle(self, *args, **options):
""" Create grantnav package:
"""Create grantnav package:
- data_all.json (all sources)
- json_all/
|- grants.json (lists of grants)
|- grants.json
...
- data_all.json (all sources)
- json_all/
|- grants.json (lists of grants)
|- grants.json
...
"""
spinner = Spinner()
spinner.start()
Expand Down
8 changes: 6 additions & 2 deletions datastore/db/management/commands/delete_datagetter_data.py
Expand Up @@ -16,11 +16,15 @@ def add_arguments(self, parser):
)

parser.add_argument(
"--no-prompt", action="store_true", help="Don't prompt for deletes",
"--no-prompt",
action="store_true",
help="Don't prompt for deletes",
)

parser.add_argument(
"--oldest", action="store_true", help="Delete the oldest datagetter data",
"--oldest",
action="store_true",
help="Delete the oldest datagetter data",
)

def handle(self, *args, **options):
Expand Down
3 changes: 2 additions & 1 deletion datastore/db/management/commands/load_datagetter_data.py
Expand Up @@ -97,7 +97,8 @@ def load_data(self):
grants_added = grants_added + len(grant_data["grants"])
except (FileNotFoundError, KeyError, TypeError, json.JSONDecodeError) as e:
print(
"Skipping loading due to: '%s'" % e, file=self.stdout,
"Skipping loading due to: '%s'" % e,
file=self.stdout,
)
pass

Expand Down
4 changes: 3 additions & 1 deletion datastore/db/migrations/0001_initial.py
Expand Up @@ -50,7 +50,9 @@ class Migration(migrations.Migration):
),
("prefix", models.CharField(default=0, max_length=300)),
],
options={"unique_together": {("getter_run", "prefix")},},
options={
"unique_together": {("getter_run", "prefix")},
},
),
migrations.CreateModel(
name="SourceFile",
Expand Down
4 changes: 3 additions & 1 deletion datastore/db/migrations/0005_auto_20190816_1015.py
Expand Up @@ -11,6 +11,8 @@ class Migration(migrations.Migration):

operations = [
migrations.AlterField(
model_name="status", name="when", field=models.DateTimeField(auto_now=True),
model_name="status",
name="when",
field=models.DateTimeField(auto_now=True),
),
]
5 changes: 4 additions & 1 deletion datastore/db/migrations/0008_remove_latest_total_grants.py
Expand Up @@ -10,5 +10,8 @@ class Migration(migrations.Migration):
]

operations = [
migrations.RemoveField(model_name="latest", name="total_grants",),
migrations.RemoveField(
model_name="latest",
name="total_grants",
),
]
6 changes: 4 additions & 2 deletions datastore/db/migrations/0010_auto_20200916_1109.py
Expand Up @@ -11,9 +11,11 @@ class Migration(migrations.Migration):

operations = [
migrations.AlterModelOptions(
name="publisher", options={"ordering": ["prefix"]},
name="publisher",
options={"ordering": ["prefix"]},
),
migrations.AlterModelOptions(
name="sourcefile", options={"ordering": ["data__publisher__prefix"]},
name="sourcefile",
options={"ordering": ["data__publisher__prefix"]},
),
]
12 changes: 9 additions & 3 deletions datastore/settings/settings.py
Expand Up @@ -104,9 +104,15 @@
{
"NAME": "django.contrib.auth.password_validation.UserAttributeSimilarityValidator",
},
{"NAME": "django.contrib.auth.password_validation.MinimumLengthValidator",},
{"NAME": "django.contrib.auth.password_validation.CommonPasswordValidator",},
{"NAME": "django.contrib.auth.password_validation.NumericPasswordValidator",},
{
"NAME": "django.contrib.auth.password_validation.MinimumLengthValidator",
},
{
"NAME": "django.contrib.auth.password_validation.CommonPasswordValidator",
},
{
"NAME": "django.contrib.auth.password_validation.NumericPasswordValidator",
},
]


Expand Down
3 changes: 2 additions & 1 deletion datastore/tests/test_additional_data_geolookups.py
Expand Up @@ -139,7 +139,8 @@ def test_nspl_update_additional_data_beneficiaryLocation_prioritised(self):

self.assertIn("locationLookup", additional_data)
self.assertEqual(
len(additional_data["locationLookup"]), 1,
len(additional_data["locationLookup"]),
1,
)

def test_nspl_update_additional_data_recipientOrganization(self):
Expand Down

0 comments on commit 6a037a5

Please sign in to comment.