Skip to content

Commit 4c3fa7d

Browse files
authored
Merge branch 'main' into 1024-load-tests-are-failing-with-gcp-token-message
2 parents 5b8910b + 904cfeb commit 4c3fa7d

File tree

23 files changed

+623
-127
lines changed

23 files changed

+623
-127
lines changed

api/src/feeds/impl/models/gtfs_dataset_impl.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -51,4 +51,5 @@ def from_orm(cls, gtfs_dataset: Gtfsdataset | None) -> GtfsDataset | None:
5151
validation_report=cls.from_orm_latest_validation_report(gtfs_dataset.validation_reports),
5252
service_date_range_start=gtfs_dataset.service_date_range_start,
5353
service_date_range_end=gtfs_dataset.service_date_range_end,
54+
agency_timezone=gtfs_dataset.agency_timezone,
5455
)

api/src/feeds/impl/models/latest_dataset_impl.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -53,6 +53,7 @@ def from_orm(cls, dataset: Gtfsdataset | None) -> LatestDataset | None:
5353
downloaded_at=dataset.downloaded_at,
5454
service_date_range_start=dataset.service_date_range_start,
5555
service_date_range_end=dataset.service_date_range_end,
56+
agency_timezone=dataset.agency_timezone,
5657
hash=dataset.hash,
5758
validation_report=validation_report,
5859
)

api/src/feeds/impl/models/search_feed_item_result_impl.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -46,6 +46,7 @@ def from_orm_gtfs(cls, feed_search_row):
4646
hash=feed_search_row.latest_dataset_hash,
4747
service_date_range_start=feed_search_row.latest_dataset_service_date_range_start,
4848
service_date_range_end=feed_search_row.latest_dataset_service_date_range_end,
49+
agency_timezone=feed_search_row.latest_dataset_agency_timezone,
4950
)
5051
if feed_search_row.latest_dataset_id
5152
else None,

api/tests/test_data/extra_test_data.json

Lines changed: 12 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -8,8 +8,9 @@
88
"hash": "hash",
99
"downloaded_at": "2024-01-31T00:00:00+00:00",
1010
"feed_stable_id": "mdb-1",
11-
"service_date_range_start": "2024-01-01",
12-
"service_date_range_end":"2025-01-01"
11+
"service_date_range_start": "2024-09-29T00:00:00+00:00",
12+
"service_date_range_end":"2025-09-29T00:00:00+00:00",
13+
"agency_timezone": "Canada/Atlantic"
1314
},
1415
{
1516
"id": "dataset-2",
@@ -19,8 +20,9 @@
1920
"hash": "hash",
2021
"downloaded_at": "2024-02-01T00:00:00+00:00",
2122
"feed_stable_id": "mdb-1",
22-
"service_date_range_start": "2024-01-01",
23-
"service_date_range_end":"2025-01-01"
23+
"service_date_range_start": "2024-09-29T00:00:00+00:00",
24+
"service_date_range_end":"2025-09-29T00:00:00+00:00",
25+
"agency_timezone": "Canada/Atlantic"
2426
},
2527
{
2628
"id": "dataset-3",
@@ -30,8 +32,9 @@
3032
"hash": "hash",
3133
"downloaded_at": "2024-02-02T00:00:00+00:00",
3234
"feed_stable_id": "mdb-10",
33-
"service_date_range_start": "2024-01-01",
34-
"service_date_range_end":"2025-01-01"
35+
"service_date_range_start": "2024-09-29T00:00:00+00:00",
36+
"service_date_range_end":"2025-09-29T00:00:00+00:00",
37+
"agency_timezone": "Canada/Atlantic"
3538
},
3639
{
3740
"id": "dataset-4",
@@ -41,8 +44,9 @@
4144
"hash": "hash",
4245
"downloaded_at": "2024-02-03T00:00:00+00:00",
4346
"feed_stable_id": "mdb-10",
44-
"service_date_range_start": "2024-01-01",
45-
"service_date_range_end":"2025-01-01"
47+
"service_date_range_start": "2024-09-29T00:00:00+00:00",
48+
"service_date_range_end":"2025-09-29T00:00:00+00:00",
49+
"agency_timezone": "Canada/Atlantic"
4650
}
4751
],
4852
"validation_reports": [

api/tests/test_data/test_datasets.json

Lines changed: 15 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -9,8 +9,9 @@
99
"hash": "hash-1",
1010
"latest": true,
1111
"bounding_box": "POLYGON((-122.75 36.8, -122.75 37.8, -121.75 37.8, -121.75 36.8, -122.75 36.8))",
12-
"service_date_range_start": "2024-01-01",
13-
"service_date_range_end":"2025-01-01"
12+
"service_date_range_start": "2024-09-29T00:00:00+00:00",
13+
"service_date_range_end":"2025-09-29T00:00:00+00:00",
14+
"agency_timezone": "Canada/Atlantic"
1415
},
1516
{
1617
"id": "dataset-2",
@@ -20,8 +21,9 @@
2021
"hash": "hash-2",
2122
"latest": false,
2223
"bounding_box": "POLYGON((-122.75 36.8, -122.75 37.8, -121.75 37.8, -121.75 36.8, -122.75 36.8))",
23-
"service_date_range_start": "2024-01-01",
24-
"service_date_range_end":"2025-01-01"
24+
"service_date_range_start": "2024-09-29T00:00:00+00:00",
25+
"service_date_range_end":"2025-09-29T00:00:00+00:00",
26+
"agency_timezone": "Canada/Atlantic"
2527
},
2628
{
2729
"id": "dataset-3",
@@ -31,8 +33,9 @@
3133
"hash": "hash-3",
3234
"latest": true,
3335
"bounding_box": "POLYGON((-122.75 36.8, -122.75 37.8, -121.75 37.8, -121.75 36.8, -122.75 36.8))",
34-
"service_date_range_start": "2024-01-01",
35-
"service_date_range_end":"2025-01-01"
36+
"service_date_range_start": "2024-09-29T00:00:00+00:00",
37+
"service_date_range_end":"2025-09-29T00:00:00+00:00",
38+
"agency_timezone": "Canada/Atlantic"
3639
},
3740
{
3841
"id": "dataset-4",
@@ -42,8 +45,9 @@
4245
"hash": "hash-4",
4346
"latest": false,
4447
"bounding_box": "POLYGON((-122.75 36.8, -122.75 37.8, -121.75 37.8, -121.75 36.8, -122.75 36.8))",
45-
"service_date_range_start": "2024-01-01",
46-
"service_date_range_end":"2025-01-01"
48+
"service_date_range_start": "2024-09-29T00:00:00+00:00",
49+
"service_date_range_end":"2025-09-29T00:00:00+00:00",
50+
"agency_timezone": "Canada/Atlantic"
4751
},
4852
{
4953
"id": "dataset-5",
@@ -53,8 +57,9 @@
5357
"hash": "hash-5",
5458
"latest": true,
5559
"bounding_box": "POLYGON((-122.75 36.8, -122.75 37.8, -121.75 37.8, -121.75 36.8, -122.75 36.8))",
56-
"service_date_range_start": "2024-01-01",
57-
"service_date_range_end":"2025-01-01"
60+
"service_date_range_start": "2024-09-29T00:00:00+00:00",
61+
"service_date_range_end":"2025-09-29T00:00:00+00:00",
62+
"agency_timezone": "Canada/Atlantic"
5863
}
5964
],
6065
"validation_reports": [

api/tests/unittest/models/test_basic_feed_impl.py

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
import copy
22
import unittest
3-
from datetime import datetime, date
3+
from datetime import datetime
4+
from zoneinfo import ZoneInfo
45

56
from shared.database_gen.sqlacodegen_models import (
67
Feed,
@@ -64,8 +65,9 @@
6465
downloaded_at="downloaded_at",
6566
hash="hash",
6667
bounding_box="bounding_box",
67-
service_date_range_start=date(2024, 1, 1),
68-
service_date_range_end=date(2025, 1, 1),
68+
service_date_range_start=datetime(2024, 1, 1, 0, 0, 0, tzinfo=ZoneInfo("Canada/Atlantic")),
69+
service_date_range_end=datetime(2025, 1, 1, 0, 0, 0, tzinfo=ZoneInfo("Canada/Atlantic")),
70+
agency_timezone="Canada/Atlantic",
6971
validation_reports=[
7072
Validationreport(
7173
id="id",

api/tests/unittest/models/test_gtfs_dataset_impl.py

Lines changed: 8 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
import unittest
2-
from datetime import datetime, date
2+
from datetime import datetime
3+
from zoneinfo import ZoneInfo
34

45
from geoalchemy2 import WKTElement
56

@@ -42,8 +43,9 @@ def test_from_orm(self):
4243
Validationreport(validator_version="0.2.0"),
4344
Validationreport(validator_version="1.1.1"),
4445
],
45-
service_date_range_start=date(2024, 1, 1),
46-
service_date_range_end=date(2025, 1, 1),
46+
service_date_range_start=datetime(2024, 1, 1, 0, 0, 0, tzinfo=ZoneInfo("Canada/Atlantic")),
47+
service_date_range_end=datetime(2025, 1, 1, 0, 0, 0, tzinfo=ZoneInfo("Canada/Atlantic")),
48+
agency_timezone="Canada/Atlantic",
4749
)
4850
result = GtfsDatasetImpl.from_orm(orm)
4951
assert result.id == "stable_id"
@@ -58,7 +60,8 @@ def test_from_orm(self):
5860
assert result.bounding_box.minimum_longitude == 3.0
5961
assert result.bounding_box.maximum_longitude == 4.0
6062
assert result.validation_report.validator_version == "1.1.1"
61-
assert result.service_date_range_start == date(2024, 1, 1)
62-
assert result.service_date_range_end == date(2025, 1, 1)
63+
assert result.service_date_range_start == datetime(2024, 1, 1, 0, 0, 0, tzinfo=ZoneInfo("Canada/Atlantic"))
64+
assert result.service_date_range_end == datetime(2025, 1, 1, 0, 0, 0, tzinfo=ZoneInfo("Canada/Atlantic"))
65+
assert result.agency_timezone == "Canada/Atlantic"
6366

6467
assert GtfsDatasetImpl.from_orm(None) is None

api/tests/unittest/models/test_gtfs_feed_impl.py

Lines changed: 8 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
import copy
22
import unittest
3-
from datetime import datetime, date
3+
from datetime import datetime
4+
from zoneinfo import ZoneInfo
45

56
from geoalchemy2 import WKTElement
67

@@ -87,8 +88,9 @@ def create_test_notice(notice_code: str, total_notices: int, severity: str):
8788
note="note",
8889
downloaded_at=datetime(year=2022, month=12, day=31, hour=13, minute=45, second=56),
8990
hash="hash",
90-
service_date_range_start=date(2024, 1, 1),
91-
service_date_range_end=date(2025, 1, 1),
91+
service_date_range_start=datetime(2024, 1, 1, 0, 0, 0, tzinfo=ZoneInfo("Canada/Atlantic")),
92+
service_date_range_end=datetime(2025, 1, 1, 0, 0, 0, tzinfo=ZoneInfo("Canada/Atlantic")),
93+
agency_timezone="Canada/Atlantic",
9294
bounding_box=WKTElement(POLYGON, srid=4326),
9395
latest=True,
9496
validation_reports=[
@@ -171,8 +173,9 @@ def create_test_notice(notice_code: str, total_notices: int, severity: str):
171173
unique_warning_count=4,
172174
unique_info_count=2,
173175
),
174-
service_date_range_start="2024-01-01",
175-
service_date_range_end="2025-01-01",
176+
service_date_range_start=datetime(2024, 1, 1, 0, 0, 0, tzinfo=ZoneInfo("Canada/Atlantic")),
177+
service_date_range_end=datetime(2025, 1, 1, 0, 0, 0, tzinfo=ZoneInfo("Canada/Atlantic")),
178+
agency_timezone="Canada/Atlantic",
176179
),
177180
)
178181

api/tests/unittest/models/test_latest_dataset_impl.py

Lines changed: 8 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
import unittest
2-
from datetime import datetime, date
2+
from datetime import datetime
3+
from zoneinfo import ZoneInfo
34

45
from geoalchemy2 import WKTElement
56

@@ -23,8 +24,9 @@ def test_from_orm(self):
2324
downloaded_at=now,
2425
hash="hash",
2526
bounding_box=WKTElement(POLYGON, srid=4326),
26-
service_date_range_start=date(2024, 1, 1),
27-
service_date_range_end=date(2025, 1, 1),
27+
service_date_range_start=datetime(2024, 1, 1, 0, 0, 0, tzinfo=ZoneInfo("Canada/Atlantic")),
28+
service_date_range_end=datetime(2025, 1, 1, 0, 0, 0, tzinfo=ZoneInfo("Canada/Atlantic")),
29+
agency_timezone="Canada/Atlantic",
2830
validation_reports=[
2931
Validationreport(validator_version="1.0.0"),
3032
Validationreport(
@@ -52,8 +54,9 @@ def test_from_orm(self):
5254
minimum_longitude=3.0,
5355
maximum_longitude=4.0,
5456
),
55-
service_date_range_start=date(2024, 1, 1),
56-
service_date_range_end=date(2025, 1, 1),
57+
service_date_range_start=datetime(2024, 1, 1, 0, 0, 0, tzinfo=ZoneInfo("Canada/Atlantic")),
58+
service_date_range_end=datetime(2025, 1, 1, 0, 0, 0, tzinfo=ZoneInfo("Canada/Atlantic")),
59+
agency_timezone="Canada/Atlantic",
5760
validation_report={
5861
"validator_version": "1.2.0",
5962
"total_error": 3,

api/tests/unittest/models/test_search_feed_item_result_impl.py

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -40,8 +40,9 @@ def __init__(self, **kwargs):
4040
latest_dataset_downloaded_at=downloaded_at,
4141
latest_dataset_bounding_box=None,
4242
latest_dataset_hash="latest_dataset_hash",
43-
latest_dataset_service_date_range_start="2030-10-10",
44-
latest_dataset_service_date_range_end="2031-10-10",
43+
latest_dataset_service_date_range_start="2030-09-29T00:00:00+00:00",
44+
latest_dataset_service_date_range_end="2031-09-29T00:00:00+00:00",
45+
latest_dataset_agency_timezone="Canada/Atlantic",
4546
external_ids=[],
4647
redirect_ids=[],
4748
feed_reference_ids=[],
@@ -84,6 +85,7 @@ def test_from_orm_gtfs(self):
8485
hash=item.latest_dataset_hash,
8586
service_date_range_start=item.latest_dataset_service_date_range_start,
8687
service_date_range_end=item.latest_dataset_service_date_range_end,
88+
agency_timezone=item.latest_dataset_agency_timezone,
8789
),
8890
)
8991
assert result == expected

0 commit comments

Comments
 (0)