Two attributes added with tests

kadhikari · kadhikari · commit c109375ad856 · 2024-12-18T14:11:52.000+01:00
diff --git a/source/jormungandr/jormungandr/interfaces/v1/serializer/status.py b/source/jormungandr/jormungandr/interfaces/v1/serializer/status.py
@@ -149,6 +149,8 @@ class TravelerProfilesSerializer(serpy.Serializer):
     max_bss_direct_path_duration = Field(schema_type=int)
     max_car_direct_path_duration = Field(schema_type=int)
     max_ridesharing_direct_path_duration = Field(schema_type=int)
+    max_taxi_direct_path_duration = Field(schema_type=int)
+    max_car_no_park_direct_path_duration = Field(schema_type=int)
 
 
 class AutocompleteSerializer(serpy.DictSerializer):
diff --git a/source/jormungandr/jormungandr/travelers_profile.py b/source/jormungandr/jormungandr/travelers_profile.py
@@ -59,11 +59,13 @@ def __init__(
         max_car_duration_to_pt=15 * 60,  # type: int
         walking_step_penalty=30.0,  # type: float
         walking_use_hills=0.5,  # type: float
-        max_walking_direct_path_duration=24 * 60 * 60,  # type: int
-        max_bike_direct_path_duration=24 * 60 * 60,  # type: int
-        max_bss_direct_path_duration=24 * 60 * 60,  # type: int
-        max_car_direct_path_duration=24 * 60 * 60,  # type: int
-        max_ridesharing_direct_path_duration=24 * 60 * 60,  # type: int
+        max_walking_direct_path_duration=60 * 60,  # type: int
+        max_bike_direct_path_duration=60 * 60,  # type: int
+        max_bss_direct_path_duration=60 * 60,  # type: int
+        max_car_direct_path_duration=40 * 60,  # type: int
+        max_ridesharing_direct_path_duration=60 * 60,  # type: int
+        max_taxi_direct_path_duration=60 * 60,  # type: int
+        max_car_no_park_direct_path_duration=30 * 60,  # type: int
         traveler_type='',  # type: Text
         is_from_db=False,  # type: bool
     ):
@@ -101,6 +103,8 @@ def __init__(
         self.max_bss_direct_path_duration = max_bss_direct_path_duration
         self.max_car_direct_path_duration = max_car_direct_path_duration
         self.max_ridesharing_direct_path_duration = max_ridesharing_direct_path_duration
+        self.max_taxi_direct_path_duration = max_taxi_direct_path_duration
+        self.max_car_no_park_direct_path_duration = max_car_no_park_direct_path_duration
 
         self.wheelchair = wheelchair
 
@@ -127,6 +131,8 @@ def override_params(self, args):
             ('max_bss_direct_path_duration', self.max_bss_direct_path_duration),
             ('max_car_direct_path_duration', self.max_car_direct_path_duration),
             ('max_ridesharing_direct_path_duration', self.max_ridesharing_direct_path_duration),
+            ('max_taxi_direct_path_duration', self.max_taxi_direct_path_duration),
+            ('max_car_no_park_direct_path_duration', self.max_car_no_park_direct_path_duration),
         )
 
         def override(pair):
@@ -181,6 +187,8 @@ def make_traveler_profile(cls, coverage, traveler_type):
             max_bss_direct_path_duration=model.max_bss_direct_path_duration,
             max_car_direct_path_duration=model.max_car_direct_path_duration,
             max_ridesharing_direct_path_duration=model.max_ridesharing_direct_path_duration,
+            max_taxi_direct_path_duration=model.max_taxi_direct_path_duration,
+            max_car_no_park_direct_path_duration=model.max_car_no_park_direct_path_duration,
             is_from_db=True,
         )
 
diff --git a/source/navitiacommon/navitiacommon/default_traveler_profile_params.py b/source/navitiacommon/navitiacommon/default_traveler_profile_params.py
@@ -43,11 +43,13 @@
         'last_section_mode': ['walking', 'bss', 'bike'],
         'walking_step_penalty': 30.0,
         'walking_use_hills': 0.5,
-        'max_walking_direct_path_duration': 24 * 60 * 60,
-        'max_bike_direct_path_duration': 24 * 60 * 60,
-        'max_bss_direct_path_duration': 24 * 60 * 60,
-        'max_car_direct_path_duration': 24 * 60 * 60,
-        'max_ridesharing_direct_path_duration': 24 * 60 * 60,
+        'max_walking_direct_path_duration': 60 * 60,
+        'max_bike_direct_path_duration': 60 * 60,
+        'max_bss_direct_path_duration': 60 * 60,
+        'max_car_direct_path_duration': 40 * 60,
+        'max_ridesharing_direct_path_duration': 60 * 60,
+        'max_taxi_direct_path_duration': 60 * 60,
+        'max_car_no_park_direct_path_duration': 30 * 60,
     },
     'slow_walker': {
         'traveler_type': 'slow_walker',
@@ -63,11 +65,13 @@
         'last_section_mode': ['walking', 'bss', 'bike'],
         'walking_step_penalty': 30.0,
         'walking_use_hills': 0.5,
-        'max_walking_direct_path_duration': 24 * 60 * 60,
-        'max_bike_direct_path_duration': 24 * 60 * 60,
-        'max_bss_direct_path_duration': 24 * 60 * 60,
-        'max_car_direct_path_duration': 24 * 60 * 60,
-        'max_ridesharing_direct_path_duration': 24 * 60 * 60,
+        'max_walking_direct_path_duration': 60 * 60,
+        'max_bike_direct_path_duration': 60 * 60,
+        'max_bss_direct_path_duration': 60 * 60,
+        'max_car_direct_path_duration': 40 * 60,
+        'max_ridesharing_direct_path_duration': 60 * 60,
+        'max_taxi_direct_path_duration': 60 * 60,
+        'max_car_no_park_direct_path_duration': 30 * 60,
     },
     'fast_walker': {
         'traveler_type': 'fast_walker',
@@ -83,11 +87,13 @@
         'last_section_mode': ['walking', 'bss', 'bike'],
         'walking_step_penalty': 30.0,
         'walking_use_hills': 0.5,
-        'max_walking_direct_path_duration': 24 * 60 * 60,
-        'max_bike_direct_path_duration': 24 * 60 * 60,
-        'max_bss_direct_path_duration': 24 * 60 * 60,
-        'max_car_direct_path_duration': 24 * 60 * 60,
-        'max_ridesharing_direct_path_duration': 24 * 60 * 60,
+        'max_walking_direct_path_duration': 60 * 60,
+        'max_bike_direct_path_duration': 60 * 60,
+        'max_bss_direct_path_duration': 60 * 60,
+        'max_car_direct_path_duration': 40 * 60,
+        'max_ridesharing_direct_path_duration': 60 * 60,
+        'max_taxi_direct_path_duration': 60 * 60,
+        'max_car_no_park_direct_path_duration': 30 * 60,
     },
     'luggage': {
         'traveler_type': 'luggage',
@@ -104,11 +110,13 @@
         'wheelchair': True,
         'walking_step_penalty': 3600.0,
         'walking_use_hills': 0,
-        'max_walking_direct_path_duration': 24 * 60 * 60,
-        'max_bike_direct_path_duration': 24 * 60 * 60,
-        'max_bss_direct_path_duration': 24 * 60 * 60,
-        'max_car_direct_path_duration': 24 * 60 * 60,
-        'max_ridesharing_direct_path_duration': 24 * 60 * 60,
+        'max_walking_direct_path_duration': 60 * 60,
+        'max_bike_direct_path_duration': 60 * 60,
+        'max_bss_direct_path_duration': 60 * 60,
+        'max_car_direct_path_duration': 40 * 60,
+        'max_ridesharing_direct_path_duration': 60 * 60,
+        'max_taxi_direct_path_duration': 60 * 60,
+        'max_car_no_park_direct_path_duration': 30 * 60,
     },
     'wheelchair': {
         'traveler_type': 'wheelchair',
@@ -125,11 +133,13 @@
         'wheelchair': True,
         'walking_step_penalty': 3600.0,
         'walking_use_hills': 0,
-        'max_walking_direct_path_duration': 24 * 60 * 60,
-        'max_bike_direct_path_duration': 24 * 60 * 60,
-        'max_bss_direct_path_duration': 24 * 60 * 60,
-        'max_car_direct_path_duration': 24 * 60 * 60,
-        'max_ridesharing_direct_path_duration': 24 * 60 * 60,
+        'max_walking_direct_path_duration': 60 * 60,
+        'max_bike_direct_path_duration': 60 * 60,
+        'max_bss_direct_path_duration': 60 * 60,
+        'max_car_direct_path_duration': 40 * 60,
+        'max_ridesharing_direct_path_duration': 60 * 60,
+        'max_taxi_direct_path_duration': 60 * 60,
+        'max_car_no_park_direct_path_duration': 30 * 60,
     },
     # Temporary Profile
     'cyclist': {
@@ -146,11 +156,13 @@
         'last_section_mode': ['walking', 'bss'],
         'walking_step_penalty': 30.0,
         'walking_use_hills': 0.5,
-        'max_walking_direct_path_duration': 24 * 60 * 60,
-        'max_bike_direct_path_duration': 24 * 60 * 60,
-        'max_bss_direct_path_duration': 24 * 60 * 60,
-        'max_car_direct_path_duration': 24 * 60 * 60,
-        'max_ridesharing_direct_path_duration': 24 * 60 * 60,
+        'max_walking_direct_path_duration': 60 * 60,
+        'max_bike_direct_path_duration': 60 * 60,
+        'max_bss_direct_path_duration': 60 * 60,
+        'max_car_direct_path_duration': 40 * 60,
+        'max_ridesharing_direct_path_duration': 60 * 60,
+        'max_taxi_direct_path_duration': 60 * 60,
+        'max_car_no_park_direct_path_duration': 30 * 60,
     },
     # Temporary Profile
     'motorist': {
@@ -167,11 +179,13 @@
         'last_section_mode': ['walking'],
         'walking_step_penalty': 30.0,
         'walking_use_hills': 0.5,
-        'max_walking_direct_path_duration': 24 * 60 * 60,
-        'max_bike_direct_path_duration': 24 * 60 * 60,
-        'max_bss_direct_path_duration': 24 * 60 * 60,
-        'max_car_direct_path_duration': 24 * 60 * 60,
-        'max_ridesharing_direct_path_duration': 24 * 60 * 60,
+        'max_walking_direct_path_duration': 60 * 60,
+        'max_bike_direct_path_duration': 60 * 60,
+        'max_bss_direct_path_duration': 60 * 60,
+        'max_car_direct_path_duration': 40 * 60,
+        'max_ridesharing_direct_path_duration': 60 * 60,
+        'max_taxi_direct_path_duration': 60 * 60,
+        'max_car_no_park_direct_path_duration': 30 * 60,
     },
 }
 
diff --git a/source/navitiacommon/navitiacommon/models/__init__.py b/source/navitiacommon/navitiacommon/models/__init__.py
@@ -1209,6 +1209,20 @@ class TravelerProfile(db.Model):  # type: ignore
         server_default=str(default_values.max_ridesharing_direct_path_duration),
     )
 
+    max_taxi_direct_path_duration = db.Column(
+        db.Integer,
+        default=default_values.max_taxi_direct_path_duration,
+        nullable=False,
+        server_default=str(default_values.max_taxi_direct_path_duration),
+    )
+
+    max_car_no_park_direct_path_duration = db.Column(
+        db.Integer,
+        default=default_values.max_car_no_park_direct_path_duration,
+        nullable=False,
+        server_default=str(default_values.max_car_no_park_direct_path_duration),
+    )
+
     @classmethod
     def get_by_coverage_and_type(cls, coverage, traveler_type):
         model = (
diff --git a/source/tyr/migrations/versions/79bd4783fd9e_add_walking_and_bike_params.py b/source/tyr/migrations/versions/79bd4783fd9e_add_walking_and_bike_params.py
@@ -101,6 +101,14 @@ def upgrade():
         'traveler_profile',
         sa.Column('max_ridesharing_direct_path_duration', sa.Integer(), server_default='86400', nullable=False),
     )
+    op.add_column(
+        'traveler_profile',
+        sa.Column('max_taxi_direct_path_duration', sa.Integer(), server_default='86400', nullable=False),
+    )
+    op.add_column(
+        'traveler_profile',
+        sa.Column('max_car_no_park_direct_path_duration', sa.Integer(), server_default='86400', nullable=False),
+    )
 
 
 def downgrade():
@@ -138,3 +146,5 @@ def downgrade():
     op.drop_column('traveler_profile', 'max_bss_direct_path_duration')
     op.drop_column('traveler_profile', 'max_car_direct_path_duration')
     op.drop_column('traveler_profile', 'max_ridesharing_direct_path_duration')
+    op.drop_column('traveler_profile', 'max_taxi_direct_path_duration')
+    op.drop_column('traveler_profile', 'max_car_no_park_direct_path_duration')
diff --git a/source/tyr/tests/integration/instance_test.py b/source/tyr/tests/integration/instance_test.py
@@ -393,9 +393,11 @@ def test_create_traveler_profile(create_instance, traveler_profile_params):
     check_traveler_profile(resp[0], traveler_profile_params)
 
     # Let's check some attributes absent in traveler_profile_params
-    # They should have default value of database
-    assert resp[0]['max_car_direct_path_duration'] == 24 * 60 * 60
-    assert resp[0]['max_ridesharing_direct_path_duration'] == 24 * 60 * 60
+    # They should have default value of default_traveler_profile_params
+    assert resp[0]['max_car_direct_path_duration'] == 40 * 60
+    assert resp[0]['max_ridesharing_direct_path_duration'] == 60 * 60
+    assert resp[0]['max_taxi_direct_path_duration'] == 60 * 60
+    assert resp[0]['max_car_no_park_direct_path_duration'] == 30 * 60
 
     # Those present in traveler_profile_params should have values as declared
     assert resp[0]['walking_use_hills'] == 0.5
@@ -422,8 +424,8 @@ def test_update_traveler_profile(create_instance, traveler_profile_params):
 
     # Let's check some attributes absent in traveler_profile_params
     # They should have default value of database
-    assert resp[0]['max_car_direct_path_duration'] == 24 * 60 * 60
-    assert resp[0]['max_ridesharing_direct_path_duration'] == 24 * 60 * 60
+    assert resp[0]['max_car_direct_path_duration'] == 40 * 60
+    assert resp[0]['max_ridesharing_direct_path_duration'] == 60 * 60
 
     # Those present in traveler_profile_params should have values as declared
     assert resp[0]['walking_use_hills'] == 0.5
diff --git a/source/tyr/tyr/fields.py b/source/tyr/tyr/fields.py
@@ -145,6 +145,8 @@ def format(self, value):
     'max_bss_direct_path_duration': fields.Raw,
     'max_car_direct_path_duration': fields.Raw,
     'max_ridesharing_direct_path_duration': fields.Raw,
+    'max_taxi_direct_path_duration': fields.Raw,
+    'max_car_no_park_direct_path_duration': fields.Raw,
     'error': fields.String,
 }
 

Original file line number	Diff line number	Diff line change
`@@ -145,6 +145,8 @@ def format(self, value):`
`145`	`145`	`'max_bss_direct_path_duration': fields.Raw,`
`146`	`146`	`'max_car_direct_path_duration': fields.Raw,`
`147`	`147`	`'max_ridesharing_direct_path_duration': fields.Raw,`
	`148`	`+ 'max_taxi_direct_path_duration': fields.Raw,`
	`149`	`+ 'max_car_no_park_direct_path_duration': fields.Raw,`
`148`	`150`	`'error': fields.String,`
`149`	`151`	`}`
`150`	`152`