Parametric tests#333
Merged
Merged
Conversation
There was a problem hiding this comment.
Pull request overview
This PR updates Vehicle Framework’s pathing/region infrastructure to support a newer pathfinding pipeline (via IPathingManager, new grid/source abstractions, and ownership handling changes) and reworks the test suite toward parameterized/fixture-style tests. It also bumps the mod version and adjusts build/test harness configuration.
Changes:
- Refactors vehicle region/grid systems to depend on
IPathingManagerand newRegionDatacomposition rather than reaching directly intoVehiclePathingSystem.VehiclePathData. - Introduces new pathfinding grids/sources (e.g., road prefer/avoid grids, building grid, path settings, grid calculator) and updates designators/debug tools accordingly.
- Converts/remodels many unit tests to
[TestFixture]style and adds parameter sources; updates project references forSmashTools.Burstand local-burst toggling.
Reviewed changes
Copilot reviewed 233 out of 235 changed files in this pull request and generated 7 comments.
Show a summary per file
| File | Description |
|---|---|
| Version.txt | Version bump. |
| Updates/UpdateLog.xml | Updates current version. |
| TestPlans/Full.xml | Updates CLI args for test/smoke runs. |
| Source/Vehicles/World/Caravan/EnterMapUtilityVehicles.cs | Switches to PathData alias/type. |
| Source/Vehicles/Vehicles.csproj | Adds conditional local/nuget Burst dependency; updates Burst package version. |
| Source/Vehicles/Utility/Performance/ThreadEnabler.cs | Doc comment wording update. |
| Source/Vehicles/Utility/Performance/ThreadDisabler.cs | Doc comment wording update. |
| Source/Vehicles/Utility/Helpers/FlashMapGrid.cs | Updates claimant flashing logic; routes modifier lookup through new pathfinder. |
| Source/Vehicles/Utility/Helpers/AsyncActions/AsyncRebuildRegionsAction.cs | Migrates from VehiclePathData alias to PathData. |
| Source/Vehicles/Utility/Helpers/AsyncActions/AsyncConnectionAction.cs | Marks obsolete for planned removal. |
| Source/Vehicles/Utility/Extensions/Ext_Vehicles.cs | Adds traversal helpers and new IPathingManager overload; switches to PathData. |
| Source/Vehicles/PreprocessorList.md | Removes HIERARCHAL_PATHFINDING documentation. |
| Source/Vehicles/Pathing/WorldGridOwners.cs | Initializes with provided vehicle def list and uses it during config generation. |
| Source/Vehicles/Pathing/World/WorldVehicleReachability.cs | Updates TODO tag/reference. |
| Source/Vehicles/Pathing/RegionGrid/VehicleRoom.cs | Refactors to bind to IPathingManager; updates room registration path. |
| Source/Vehicles/Pathing/RegionGrid/VehicleRegionTraverser.cs | Internalizes APIs and switches to IPathingManager flow. |
| Source/Vehicles/Pathing/RegionGrid/VehicleRegionManager.cs | Refactors base to use IPathingManager and introduces ownership-change hook. |
| Source/Vehicles/Pathing/RegionGrid/VehicleRegionGridManager.cs | Refactors init/deps; injects maker/dirtyer; adjusts grid types list. |
| Source/Vehicles/Pathing/RegionGrid/VehicleRegionGrid.cs | Refactors to use map from pathing; injects updater. |
| Source/Vehicles/Pathing/RegionGrid/VehicleRegionDirtyer.cs | Refactors to use IPathingManager; hooks path-grid walkability change events. |
| Source/Vehicles/Pathing/RegionGrid/VehicleRegionConnector.cs | Marks obsolete and partially detaches from region manager. |
| Source/Vehicles/Pathing/RegionGrid/VehicleRegionAndRoomUpdater.cs | Refactors to injected dirtyer/path grid and new init flow. |
| Source/Vehicles/Pathing/RegionGrid/VehicleRegionAndRoomQuery.cs | Adds IPathingManager overloads; marks as public API. |
| Source/Vehicles/Pathing/RegionGrid/VehicleReachabilityUtility.cs | Adds refactor note. |
| Source/Vehicles/Pathing/RegionGrid/TouchPathEndModeUtilityVehicles.cs | Adds overload to pass RegionGridType. |
| Source/Vehicles/Pathing/RegionGrid/Sources/RegionSourceNormal.cs | Refactors expected-region logic to use IPathingManager. |
| Source/Vehicles/Pathing/RegionGrid/Sources/RegionSourceBreach.cs | Refactors breach region logic to use IPathingManager and tighter breach rules. |
| Source/Vehicles/Pathing/RegionGrid/RegionGridType.cs | Adds Invalid enum value. |
| Source/Vehicles/Pathing/RegionGrid/RegionData.cs | New composition root for region-related managers per vehicle. |
| Source/Vehicles/Pathing/RegionGrid/IRegionSource.cs | Switches interface to accept IPathingManager. |
| Source/Vehicles/Pathing/RegionGrid/IGridDebouncerSource.cs | Reworks interface; adds ActiveDebouncer setter and documentation. |
| Source/Vehicles/Pathing/RegionGrid/GridDebouncer.cs | Assigns/clears active debouncer into sources on create/dispose. |
| Source/Vehicles/Pathing/RegionGrid/GenGridVehicles.cs | Adds IPathingManager walkable helper; formatting/doc adjustments. |
| Source/Vehicles/Pathing/MapGridOwners.cs | Refactors to accept IPathingManager + explicit def list. |
| Source/Vehicles/Pathing/Map/VehicleCostOffsets.cs | New base class for cost offset grids integrating with pathfinder manager dirties. |
| Source/Vehicles/Pathing/Map/VehicleBitOffsets.cs | New base class for boolean offset grids with per-settings costs. |
| Source/Vehicles/Pathing/Map/PathSources/RoadPreferGrid.cs | New road “prefer” bit-offset grid. |
| Source/Vehicles/Pathing/Map/PathSources/RoadHeuristic.cs | New heuristic grid placeholder for road-related heuristics. |
| Source/Vehicles/Pathing/Map/PathSources/RoadAvoidGrid.cs | New road “avoid” bit-offset grid. |
| Source/Vehicles/Pathing/Map/PathSources/BuildingGrid.cs | New path-cost grid for building hitpoints (breach scoring). |
| Source/Vehicles/Pathing/Map/PathSettings.cs | New path settings struct for pathfinder v2 behavior selection. |
| Source/Vehicles/Pathing/Map/PathGridCalculator.cs | New path-cost calculator implementation. |
| Source/Vehicles/Pathing/Map/IPathingManager.cs | New interface abstracting map pathing ownership/grids. |
| Source/Vehicles/Pathing/Map/IPathGridCalculator.cs | New interface for path-cost calculation. |
| Source/Vehicles/Pathing/Map/IPathCostGrid.cs | New interface for cost grids used by pathfinder. |
| Source/Vehicles/Pathing/Map/DeferredGridGeneration.cs | Updates terminology and switches to PathData. |
| Source/Vehicles/Pathing/GridOwners.cs | Removes recache logic from static class; leaves world owners as static. |
| Source/Vehicles/Pathing/GridOwnerList.cs | Changes owner list to operate on an explicit vehicleDefs list; constructor initializes. |
| Source/Vehicles/Misc/ModSettings/SettingsSection/SectionMain.cs | Renames burst probation flag to testingBurst and adds testedBurstOn. |
| Source/Vehicles/Misc/ModSettings/SettingsSection/SectionDebug.cs | Removes some debug toggles and conditional options. |
| Source/Vehicles/Misc/ModSettings/BurstProbation.cs | Updates settings flag name. |
| Source/Vehicles/Misc/Enums/Vehicle/VehicleType.cs | Flattens enum to top-level. |
| Source/Vehicles/Misc/Enums/Debug/DebugRegionType.cs | Removes/renumbers some debug region flags. |
| Source/Vehicles/Misc/DefOf/VehicleStatUpgradeCategoryDefOf.cs | Refactors DefOf class layout and adds new category defs. |
| Source/Vehicles/Misc/DefOf/VehicleEventDefOf.cs | Adds new FactionChanged event def-of. |
| Source/Vehicles/Misc/CellFinderExtended.cs | Switches to PathData. |
| Source/Vehicles/Harmony/VehicleHarmony.cs | Updates burst enable/disable workflow and recache location. |
| Source/Vehicles/Harmony/Patches/Patch_VehiclePathing.cs | Adjusts patch targets for incremental dirtying capture. |
| Source/Vehicles/Harmony/Patches/Patch_NpcPathing.cs | Uses feature flag rather than debug setting for raiders. |
| Source/Vehicles/Harmony/Patches/Patch_NpcAi.cs | Uses feature flag rather than debug setting for raiders. |
| Source/Vehicles/Gizmo/Designators/Designator_AreaRoad.cs | Updates road designation to new pathfinder v2 grids/heuristics/scalars. |
| Source/Vehicles/ExternalAnnotations/Assembly-CSharp.xml | Updates external annotation attribute for DebugActionAttribute. |
| Source/Vehicles/CustomFeatures/Reservation/VehiclePositionManager.cs | Uses EntityRect for diagonal hitboxes and adds Dispose cleanup. |
| Source/Vehicles/CustomFeatures/AerialVehicles/ArrivalActions/AerialVehicleArrivalModes/AerialVehicleArrivalModeWorker_CenterDrop.cs | Adds annotations for reflection usage. |
| Source/Vehicles/Components/Vehicles/VehicleProperties.cs | Adds building collision multipliers and reorders track field. |
| Source/Vehicles/Components/Vehicles/VehiclePawn/VehiclePawn_Rendering.cs | Removes debug path preview menu in rendering partial. |
| Source/Vehicles/Components/Vehicles/VehicleDef.cs | Updates cargo icon caching using backing-field feature. |
| Source/Vehicles/Components/Vehicles/Misc/VehicleOrientationController.cs | Renames fit check helper and adjusts logic. |
| Source/Vehicles/Components/Vehicles/FloatOptionProviders/FloatMenuOptionProvider_OrderVehicle.cs | Uses TryOrderMoveTo and adds feedback fleck on success. |
| Source/Vehicles/Components/Rendering/Tracks/VehicleTrack.cs | Refactors to top-level abstract class. |
| Source/Vehicles/Components/Rendering/Tracks/VehicleTrack_Wake.cs | Adds annotations for reflection usage. |
| Source/Vehicles/AI/TransferableSearch.cs | Formatting change in explicit interface member. |
| Source/Vehicles/AI/ThinkNode/ThinkNode_ConditionalVehicleState.cs | Refactors to top-level class and fixes DeepCopy. |
| Source/Vehicles/AI/ThinkNode/ThinkNode_ConditionalHasTurret.cs | Adds public API annotation. |
| Source/Vehicles/AI/ThinkNode/ThinkNode_ConditionalEngaging.cs | New conditional node for turret engagement state. |
| Source/Vehicles/AI/JobGivers/NPC/JobGiver_RangedSupport.cs | Updates combat position selection/expiry and scanning flags. |
| Source/Vehicles/AI/JobGivers/NPC/JobGiver_ExitMap.cs | Switches to PathData. |
| Source/Vehicles/AI/JobGivers/NPC/JobGiver_CombatFormation.cs | Minor formatting modernization. |
| Source/Vehicles/AI/JobGivers/JobGiver_GotoTravelDestinationVehicle.cs | Refactors and centralizes traverse mode selection. |
| Source/Vehicles/AI/JobGivers/JobGiver_AwaitOrders.cs | Uses brakes and modernizes job creation/ternary formatting. |
| Source/Vehicles/AI/JobDrivers/JobDriverGetItemForVehicleBase.cs | Formatting change in helper classes. |
| Source/Vehicles/AI/JobDrivers/JobDriver_LoadVehicle.cs | Removes unused using and formatting change. |
| Source/Test_Vehicles/Tests/Utils/VehicleTestCase.cs | Namespace change and disables grouping scope usage. |
| Source/Test_Vehicles/Tests/Utils/VehicleSources.cs | New parameter source for vehicle grid sizes. |
| Source/Test_Vehicles/Tests/Utils/VehicleGroup.cs | Namespace change and removes verbose dev logging call. |
| Source/Test_Vehicles/Tests/Utils/TestUtils.cs | Namespace change and adds terrain null assertion. |
| Source/Test_Vehicles/Tests/Utils/TestActions.cs | Updates failures to Test.Fail and tightens cleanup logic. |
| Source/Test_Vehicles/Tests/Utils/SetTerrainOnDispose.cs | Namespace change. |
| Source/Test_Vehicles/Tests/Utils/PawnAnchor.cs | Namespace change. |
| Source/Test_Vehicles/Tests/Utils/MaterialPoolWatcher.cs | Namespace change. |
| Source/Test_Vehicles/Tests/Utils/CompGenerator.cs | Namespace change. |
| Source/Test_Vehicles/Tests/UnitTest_SpawnPlacement.cs | Removes legacy test. |
| Source/Test_Vehicles/Tests/UnitTest_PositionManager.cs | Removes legacy test. |
| Source/Test_Vehicles/Tests/UnitTest_PathGrid.cs | Removes legacy test. |
| Source/Test_Vehicles/Tests/UnitTest_GridOwners.cs | Removes legacy test. |
| Source/Test_Vehicles/Tests/UnitTest_GasGrid.cs | Removes legacy test. |
| Source/Test_Vehicles/Tests/UnitTest_CoverGrid.cs | Removes legacy test. |
| Source/Test_Vehicles/Tests/Test_WorldTargeter.cs | Converts to fixture style and refactors setup into helper. |
| Source/Test_Vehicles/Tests/Test_WorldReachability.cs | Converts to fixture style and relaxes/grouping usage. |
| Source/Test_Vehicles/Tests/Test_WorldPawns.cs | Converts to fixture style. |
| Source/Test_Vehicles/Tests/Test_VehicleTurretDef.cs | Converts to fixture style and adjusts inheritance naming. |
| Source/Test_Vehicles/Tests/Test_VehicleTurret.cs | Converts to fixture style. |
| Source/Test_Vehicles/Tests/Test_VehicleTurret_Rendering.cs | Converts to fixture style. |
| Source/Test_Vehicles/Tests/Test_VehicleSkyfaller_Tick.cs | Converts to fixture style and updates method refs. |
| Source/Test_Vehicles/Tests/Test_VehicleRoleHandler.cs | Converts to fixture style. |
| Source/Test_Vehicles/Tests/Test_VehiclePermissions.cs | Converts to fixture style. |
| Source/Test_Vehicles/Tests/Test_VehiclePawn.cs | Converts to fixture style. |
| Source/Test_Vehicles/Tests/Test_VehiclePawn_Rendering.cs | Converts to fixture style and changes setup/teardown pattern. |
| Source/Test_Vehicles/Tests/Test_VehiclePawn_Reactors.cs | Converts to fixture style. |
| Source/Test_Vehicles/Tests/Test_VehiclePawn_Pathing.cs | Converts to fixture style. |
| Source/Test_Vehicles/Tests/Test_VehiclePawn_Health.cs | Converts to fixture style. |
| Source/Test_Vehicles/Tests/Test_VehiclePawn_Comfort.cs | Converts to fixture style. |
| Source/Test_Vehicles/Tests/Test_VehiclePawn_Alerts.cs | Converts to fixture style and updates setup attribute. |
| Source/Test_Vehicles/Tests/Test_VehicleDefTest.cs | Converts base def test to fixture style. |
| Source/Test_Vehicles/Tests/Test_VehicleCaravan.cs | Converts to fixture style. |
| Source/Test_Vehicles/Tests/Test_VehicleCaravan_Tick.cs | Converts to fixture style and updates method refs. |
| Source/Test_Vehicles/Tests/Test_VehicleCaravan_Needs.cs | Converts to fixture style and updates type names. |
| Source/Test_Vehicles/Tests/Test_Trade_Settlement.cs | Converts to fixture style. |
| Source/Test_Vehicles/Tests/Test_ThingGrid.cs | Converts to fixture style and updates base class name. |
| Source/Test_Vehicles/Tests/Test_StashedVehicle.cs | Converts to fixture style and updates method refs. |
| Source/Test_Vehicles/Tests/Test_RegionGridOwner.cs | New test for ownership transfer updating all region managers. |
| Source/Test_Vehicles/Tests/Test_PositionManager.cs | New parameterized test for position manager claims. |
| Source/Test_Vehicles/Tests/Test_PathGridVehicle.cs | New pathgrid-vs-vehicle behavior test skeleton. |
| Source/Test_Vehicles/Tests/Test_PathGridPawn.cs | New pawn pathgrid behavior test skeleton. |
| Source/Test_Vehicles/Tests/Test_MaterialPoolDefs.cs | Converts to fixture style; parameterizes pattern def test. |
| Source/Test_Vehicles/Tests/Test_MaterialPool.cs | Converts to fixture style. |
| Source/Test_Vehicles/Tests/Test_MapTest.cs | Converts map-test base to fixture lifecycle and adds common vehicle size list. |
| Source/Test_Vehicles/Tests/Test_MapRemoval.cs | Converts map removal base to fixture lifecycle. |
| Source/Test_Vehicles/Tests/Test_MapRemoval_SpaceMapParent.cs | Renames to new base class naming. |
| Source/Test_Vehicles/Tests/Test_MapRemoval_Site.cs | Renames to new base class naming. |
| Source/Test_Vehicles/Tests/Test_MapRemoval_Settlement.cs | Renames to new base class naming. |
| Source/Test_Vehicles/Tests/Test_MapRemoval_DestroyedSettlement.cs | Renames to new base class naming. |
| Source/Test_Vehicles/Tests/Test_MapRemoval_CrashSite.cs | Renames to new base class naming. |
| Source/Test_Vehicles/Tests/Test_MapRemoval_CaravansBattlefield.cs | Renames to new base class naming. |
| Source/Test_Vehicles/Tests/Test_MapRemoval_Camp.cs | Renames to new base class naming. |
| Source/Test_Vehicles/Tests/Test_MapPawns.cs | Converts to fixture style. |
| Source/Test_Vehicles/Tests/Test_MapGeneration_River.cs | Converts to fixture style and updates method refs. |
| Source/Test_Vehicles/Tests/Test_MapGeneration_Coast.cs | Converts to fixture style and removes static map field. |
| Source/Test_Vehicles/Tests/Test_HotReload.cs | Converts to fixture style and updates setup/teardown attributes. |
| Source/Test_Vehicles/Tests/Test_GasGrid.cs | New parameterized gas grid test. |
| Source/Test_Vehicles/Tests/Test_FeatureFlag.cs | Converts to fixture style. |
| Source/Test_Vehicles/Tests/Test_DeferredGridUrgency.cs | New tests for deferred/urgent grid generation selection. |
| Source/Test_Vehicles/Tests/Test_CoverGrid.cs | New parameterized cover grid test. |
| Source/Test_Vehicles/Tests/Test_CompFueledTravel.cs | Converts to fixture style. |
| Source/Test_Vehicles/Tests/Test_ColonistBar_GetAdjustedTarget.cs | Converts to fixture style. |
| Source/Test_Vehicles/Tests/Test_BlitTarget.cs | Converts to fixture style. |
| Source/Test_Vehicles/Tests/Test_AerialVehicle.cs | Converts to fixture style and updates lifecycle attributes. |
| Source/Test_Vehicles/Tests/Test_AerialVehicle_Tick.cs | Converts to fixture style and updates method refs. |
| Source/Test_Vehicles/Test_Vehicles.csproj | Adds conditional local/nuget Burst dependency; updates Burst package version. |
| Source/Test_Vehicles/StartupActions/UITesting.cs | Updates startup action error text. |
| Source/Test_Vehicles/Benchmarking/Benchmark_RegionGridGeneration.cs | Switches to PathData. |
| Source/Test_Vehicles/Benchmarking/Benchmark_PathGridGeneration.cs | Switches to PathData. |
| Source/Test_Vehicles/Benchmarking/Benchmark_PathFinder.cs | Removes old benchmark. |
| Source/Test_SmashTools/Tests/Utils/TestUtils.cs | Namespace renaming to SmashTools.Testing. |
| Source/Test_SmashTools/Tests/Utils/TestCategoryNames.cs | Namespace renaming to SmashTools.Testing. |
| Source/Test_SmashTools/Tests/UnitTest_Ext_Math.cs | Removes legacy math test file. |
| Source/Test_SmashTools/Tests/Test_WorldUnloaded.cs | Converts to fixture style + namespace rename. |
| Source/Test_SmashTools/Tests/Test_SelfOrderingList.cs | Converts to fixture style + namespace rename. |
| Source/Test_SmashTools/Tests/Test_ScopedRollback.cs | Converts to fixture style + namespace rename. |
| Source/Test_SmashTools/Tests/Test_RenderTexture.cs | Converts to fixture style and removes unused teardown list. |
| Source/Test_SmashTools/Tests/Test_Numerics.cs | Converts to fixture style + namespace rename. |
| Source/Test_SmashTools/Tests/Test_LineRenderer.cs | Converts to fixture style + namespace rename. |
| Source/Test_SmashTools/Tests/Test_KuhnMunkres.cs | Converts to fixture style + namespace rename. |
| Source/Test_SmashTools/Tests/Test_ITargeter.cs | Converts to fixture style + namespace rename. |
| Source/Test_SmashTools/Tests/Test_IDictionary.cs | Converts to fixture style + namespace rename. |
| Source/Test_SmashTools/Tests/Test_EventManager.cs | Converts to fixture style and fixes setup attribute usage. |
| Source/Test_SmashTools/Tests/Test_EventListener.cs | Converts to fixture style + namespace rename. |
| Source/Test_SmashTools/Tests/Test_ComponentCache.cs | Converts to fixture style + namespace rename. |
| Source/Test_SmashTools/Tests/Test_CellRectUtils.cs | Converts to fixture style + namespace rename. |
| Source/Test_SmashTools/Tests/Test_CellRectOverlap.cs | Converts to fixture style + namespace rename. |
| Source/Test_SmashTools/Tests/Test_AnimDynamicMethod.cs | Converts to fixture style + namespace rename. |
| Source/Test_SmashTools/Test_SmashTools.csproj | Adds conditional local/nuget Burst dependency; updates Burst package version. |
| Source/Test_SmashTools/Sandbox/UnsafeReadOnly.cs | Namespace rename + fixture attribute update. |
| Source/Test_SmashTools/Sandbox/LudeonCode.cs | Namespace rename + fixture attribute update + updated ParseHelper strings. |
| Source/Test_CoreLib/Tests/TestCategoryNames.cs | Namespace renaming to CoreLib.Testing. |
| Source/Test_CoreLib/Tests/Test_UnityThread.cs | Converts to fixture style; improves failure reporting. |
| Source/Test_CoreLib/Tests/Test_UnityObjectPool.cs | Converts to fixture style and adds disposal test. |
| Source/Test_CoreLib/Tests/Test_ReadOnlyList.cs | Converts to fixture style + namespace rename. |
| Source/Test_CoreLib/Tests/Test_ReadOnlyArray.cs | New read-only array wrapper tests. |
| Source/Test_CoreLib/Tests/Test_ObjectPool.cs | Converts to fixture style + namespace rename. |
| Source/Test_CoreLib/Tests/Test_IPoolable.cs | Namespace rename (SmashTools.Testing). |
| Source/Test_CoreLib/Tests/Test_ExtEnumerable.cs | New enumerable extension tests. |
| Source/Test_CoreLib/Tests/Test_DedicatedThread.cs | Converts to fixture style and tightens assertions/flow. |
| Source/Test_CoreLib/Tests/Test_Debouncer.cs | Converts to fixture style + namespace rename. |
| Source/Test_CoreLib/Tests/Test_AsyncPool.cs | Converts to fixture style + namespace rename. |
| Source/Test_CoreLib/Test_CoreLib.csproj | Adds Unity.Mathematics reference. |
| Source/Test_Burst/Tests/Test_PathFinder.cs | Converts to fixture attribute naming. |
| Source/Test_Burst/Test_Burst.csproj | Adds conditional local/nuget Burst dependency; updates Burst package version. |
| Directory.Build.props | Formatting + adds UseLocalBurstLib property. |
| Configs/UnitTestConfig.xml | Adds allowed warnings and removes map-size block. |
| Configs/SmokeTestConfig.xml | Removes verbose logging block. |
| About/About.xml | Version bump. |
| 1.6/Defs/VehiclesDefs/StatDefs/StatUpgradeCategoryDefs.xml | Adds building collision upgrade categories. |
| 1.6/Defs/VehiclesDefs/Events/VehicleEventDef_General.xml | Adds general FactionChanged event def. |
| .editorconfig | Updates ReSharper inspections and severities. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
DevTools now supports parameterized tests and better categorization of groups of tests. Pathfinding was further broken up to better support existing tests by moving away from VehiclePathingSystem, which has a lot of baggage since it is the sole manager of all pathfinding-related behavior. Region, path grid, and pathfinders are now decoupled from VehiclePathingSystem and instead rely on events and a few interfaces to drive behavior. Alongside this change, the test fixtures for pathfinding now mock the pathfinding manager instead of relying on VehiclePathingSystem.
SmashPhil
force-pushed
the
parametric-tests
branch
from
ae591b1 to
96690c8
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.