test(autoware_behavior_path_start_planner_module): add test helper and implement unit tests for FreespacePullOut #9832
CodeScene Delta Analysis / CodeScene Cloud Delta Analysis (main)
failed
Jan 10, 2025 in 1m 2s
CodeScene PR Check
❌ Code Health Quality Gates: FAILED
Change in average Code Health of affected files: -0.02 (6.02 -> 6.00)
- Declining Code Health: 3 findings(s) 🚩
- Improving Code Health: 1 findings(s) ✅
- Affected Hotspots: 1 files(s) 🔥
Details
🚩 Declining Code Health (highest to lowest):
- Brain Method shift_pull_out.cpp: ShiftPullOut::calcPullOutPaths
- Complex Method shift_pull_out.cpp: ShiftPullOut::plan
- Excess Number of Function Arguments start_planner_test_helper.cpp: StartPlannerTestHelper::set_costmap
✅ Improving Code Health:
- Lines of Code in a Single File goal_planner_module.cpp 🔥
Annotations
codescene-delta-analysis / CodeScene Cloud Delta Analysis (main)
❌ New issue: Brain Method
ShiftPullOut::calcPullOutPaths is a brain method. A Brain Method -- aka a God Function -- is a large and complex function that centralizes the behavior of a module. Brain Methods are detected using a combination of the following code smells: Deeply Nested Logic + High Cyclomatic Complexity + Many Lines of Code + Many Function Arguments.
codescene-delta-analysis / CodeScene Cloud Delta Analysis (main)
❌ Getting worse: Complex Method
ShiftPullOut::plan already has high cyclomatic complexity, and now it increases in Lines of Code from 87 to 90. This function has many conditional statements (e.g. if, for, while), leading to lower code health. Avoid adding more conditionals and code to it without refactoring.
codescene-delta-analysis / CodeScene Cloud Delta Analysis (main)
ℹ New issue: Excess Number of Function Arguments
ShiftPullOut::calcPullOutPaths has 5 arguments, threshold = 4. This function has too many arguments, indicating a lack of encapsulation. Avoid adding more arguments.
codescene-delta-analysis / CodeScene Cloud Delta Analysis (main)
✅ Getting better: Lines of Code in a Single File
The lines of code decreases from 1872 to 1866, improve code health by reducing it to 1000. The number of Lines of Code in a single file. More Lines of Code lowers the code health.
codescene-delta-analysis / CodeScene Cloud Delta Analysis (main)
❌ New issue: Excess Number of Function Arguments
StartPlannerTestHelper::set_costmap has 5 arguments, threshold = 4. This function has too many arguments, indicating a lack of encapsulation. Avoid adding more arguments.
Loading