Adding OperationalPointIdLocation as location for POST /timetable/{id}

[bloussou]

close #6182

I propose this solution since it doesn't make the backend use a SNCF extension and we doesn't have to do extra computation to extract the parts of an operational point corresponding to the uic/ch

It is improving the import when we try to respect commercial stops +7.8%:
New stats:

Previous stats:

When we try to respect all-waypoints stats are improving too +8.7%:
New stats:

Previous stats:

Proposal when we have TrainSchedule V2: obj_idis equivalent to uic + ch

[codecov]

Codecov Report

Attention: 55 lines in your changes are missing coverage. Please review.

Comparison is base (472bff2) 28.04% compared to head (fd5f3cf) 28.08%.

Files	Patch %	Lines
editoast/src/views/timetable/import.rs	69.56%	42 Missing ⚠️
...oast/src/models/infra_objects/operational_point.rs	0.00%	13 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##                dev    #6466      +/-   ##
============================================
+ Coverage     28.04%   28.08%   +0.03%     
  Complexity     2169     2169              
============================================
  Files          1034     1034              
  Lines        127757   127883     +126     
  Branches       2603     2603              
============================================
+ Hits          35831    35913      +82     
- Misses        90411    90455      +44     
  Partials       1515     1515              

Flag	Coverage Δ
core	78.54% <ø> (ø)
editoast	75.47% <63.57%> (-0.07%)	⬇️
front	8.62% <100.00%> (+<0.01%)	⬆️
gateway	2.50% <ø> (ø)
railjson_generator	87.25% <ø> (ø)
tests	81.95% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[shenriotpro]

Nice, I like the approach.
Do you think it would be possible to write more complete tests, possibly using small_infra?

[clarani]

LGTM ✅

[flomonster]

LGTM

[Castavo]

Did not test it, is there a payload which we could use to test it ?

[flomonster]

LGTM