ActsEvent: Increment surfaces with states and matching test

I had found that adding: BOOST_CHECK_EQUAL(ts.hasReferenceSurface(), false); to the unit tests caused an out of bounds exception.

I believe the problem was we need to increment m_surfaces with the states (i.e. this: !66279 (diffs) ), but I would like @tbold and @wolter to confirm they agree?

(I had also to modify the fillTrackState test function to handle the case where the surface WAS already filled).

cc @pagessin

changed title from Increment surfaces with states and matching test to ActsEvent: Increment surfaces with states and matching test

This merge request affects 1 package:

• Tracking/Acts/ActsEvent

This merge request affects 2 files:

• Tracking/Acts/ActsEvent/Root/MultiTrajectory.cxx
• Tracking/Acts/ActsEvent/test/MultiTrajectoryBasic_test.cxx

Adding @pagessin ,@jojungge ,@tbold ,@toyamaza ,@cvarni as watchers

added ACTS Tracking main review-pending-level-1 labels

changed the description

CI Result SUCCESS (hash e96dbe75)

	Athena
externals
cmake
make
tests

Full details available on this CI monitor view. Check the JIRA CI status board for known problems
Athena: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 78940]

Looks good to me!

resolved all threads

@tbold should I wait to approve this one, or can we proceed? Clara (L1)

resolved all threads

Changes look fine and the CI is good. All threads resolved. Approving. Clara (L1)

added review-approved label and removed review-pending-level-1 label

merged

mentioned in commit 85795451

added sweep:ignore label

mentioned in merge request !65871 (closed)