fix: update the tests from trainrunsections.view.spec and data.view.s…

[Math-R]

Description

This PR updates unit tests to accommodate the changes made to TrainrunSectionViewObject in the collapsed nodes feature implementation.

Changes

Modified Test Files

• trainrunsections.view.spec.ts: Updated tests for methods that now expect TrainrunSectionViewObject instead of TrainrunSection:

  • getPosition() - 2 tests updated
  • enforceStartTextAnchor() - 8 tests updated
  • getAdditionTextCloseToNodePositioningValue() - 4 tests updated
  • getTrainrunSectionNextAndDestinationNodeToShow() - 1 test updated

• data.view.spec.ts:

  • Updated TrainrunSectionViewObject constructor call to use array of sections ([ts] instead of ts)
  • Updated expected key value to reflect new key generation logic

Key Changes

The main model change was:

• TrainrunSectionViewObject.trainrunSection: TrainrunSection → TrainrunSectionViewObject.trainrunSections: TrainrunSection[]

This change supports grouping multiple sections together when intermediate nodes are collapsed, allowing the view layer to render direct paths between expanded nodes.

Test Pattern

All updated tests now follow this pattern:

const ts = trainrunSectionService.getTrainrunSectionFromId(id);
const viewObj = new TrainrunSectionViewObject(
  editorView, 
  [ts],  // Array instead of single section
  false, false, false, false, false, false, false, false
);
// Use viewObj instead of ts in method calls

Test Results
✅ All 514 tests passing
✅ No TypeScript compilation errors

Issues
Related to #554 - Part of the collapsed nodes feature implementation

Checklist
This PR contains a description of the changes I'm making
I've read the Contribution Guidelines
I've added tests for changes or features I've introduced (updated existing tests to match new signatures)
I documented any high-level concepts I'm introducing in documentation (test updates don't require documentation)
CI is currently green and this is ready for review

[louisgreiner]

Does these changes are aligned with #596?

[emersion]

Why can't we create a regular TrainrunSectionViewObject here?

[emersion]

As far as I understand this test is supposed to check that an up-to-date input DTO isn't mutated when loaded. We have another check below for numberOfStops migration.

Can we update the input to reset numberOfStops and create collapsed nodes, instead of updating the test check?