Enhancement - Improve Deployment-Phase Towing Logic

[psikomonkie]

The original implementation of "put warning signs where your tractor/trailer wouldn't be valid" did not properly consider an already deployed tractors/trailers that were more than one link away. This is specifically tested now with TowLinkWarningTest::testDeployTrailerMultiDeployed(). This also includes a change to Princess to use the new method.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 29.14%. Comparing base (5407b8b) to head (af1ab22).
Report is 53 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##             master    #6606    +/-   ##
==========================================
  Coverage     29.13%   29.14%            
- Complexity    15366    15405    +39     
==========================================
  Files          2863     2863            
  Lines        279868   279996   +128     
  Branches      49291    49302    +11     
==========================================
+ Hits          81553    81611    +58     
- Misses       192855   192908    +53     
- Partials       5460     5477    +17     

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

[Scoppio]

this PR uses alot of lists and sets, which is great, but it also uses alot of nulls and null checks, please take a look if you can change some or most of the nulls you are using for empty collections instead, just to reduce the number of situations where we have to check for null, also there is deeply nested code in many places, see if its possible to avoid some of the nesting and aggregate multiple conditionals in a single test.

[psikomonkie]

this PR uses alot of lists and sets, which is great, but it also uses alot of nulls and null checks, please take a look if you can change some or most of the nulls you are using for empty collections instead, just to reduce the number of situations where we have to check for null, also there is deeply nested code in many places, see if its possible to avoid some of the nesting and aggregate multiple conditionals in a single test.

The usage of nulls & null checking was intentional, null is returned for those sets/lists as an alternative to getting and returning every valid coord. I'll see if I can get this refactored to not need to do that.

[psikomonkie]

this PR uses alot of lists and sets, which is great, but it also uses alot of nulls and null checks, please take a look if you can change some or most of the nulls you are using for empty collections instead, just to reduce the number of situations where we have to check for null, also there is deeply nested code in many places, see if its possible to avoid some of the nesting and aggregate multiple conditionals in a single test.

Removed all of the null lists, reduced conditional nesting, and added a manual stack overflow exception for if a tractor->trailer->tractor situation occurs (which should & is prevented outside of this class). Thanks Luana!