Opened 6 years ago

Last modified 5 years ago

#5169 new defect

Warning and segfault for disabled pathfinder tests

Reported by: fatherbushido Owned by:
Priority: If Time Permits Milestone: Backlog
Component: Simulation Keywords:
Cc: Patch:

Description

In simulation2/components/tests there are some DISABLED tests.

1) test_perf2, test_perf3, test_performance lead to warnings like:

ERROR: Error in timer on entity 3, IID86, function UpdateSequences: TypeError: this.sequences is undefined
  StatisticsTracker.prototype.UpdateSequences@simulation/components/StatisticsTracker.js:591:2
  Timer.prototype.OnUpdate@simulation/components/Timer.js:120:4

2) test_performance_short leads to a segfault.

Here is a valgrind output

==7038== Process terminating with default action of signal 11 (SIGSEGV)
==7038==  Access not within mapped region at address 0x0
==7038==    at 0x4040B7: AddTerrainEdges (CCmpPathfinder_Vertex.cpp:267)
==7038==    by 0x4040B7: CCmpPathfinder::ComputeShortPath(IObstructionTestFilter const&, CFixed<int, 2147483647, 32, 15, 16, 65536>, CFixed<int, 2147483647, 32, 15, 16, 65536>, CFixed<int, 2147483647, 32, 15, 16, 65536>, CFixed<int, 2147483647, 32, 15, 16, 65536>, PathGoal const&, unsigned short, WaypointPath&) (CCmpPathfinder_Vertex.cpp:666)
==7038==    by 0x2AB965: test_performance_short (test_Pathfinder.h:194)
==7038==    by 0x2AB965: TestDescription_suite_TestCmpPathfinder_test_performance_short::runTest() (test_Pathfinder.cpp:45)
==7038==    by 0x319108: CxxTest::RealTestDescription::run() (RealDescriptions.cpp:121)
==7038==    by 0x320623: runTest (TestRunner.h:105)
==7038==    by 0x320623: runSuite (TestRunner.h:89)
==7038==    by 0x320623: runWorld (TestRunner.h:69)
==7038==    by 0x320623: runAllTests (TestRunner.h:42)
==7038==    by 0x320623: run (ErrorFormatter.h:63)
==7038==    by 0x320623: int CxxTest::Main<CxxTest::ErrorPrinter>(CxxTest::ErrorPrinter&, int, char**) (TestMain.h:147)
==7038==    by 0x1864B0: main (test_root.cpp:20)

Change History (2)

comment:1 by fatherbushido, 6 years ago

(I put that in simulation but it can be in core engine as well)

comment:2 by Imarok, 5 years ago

Component: UI & SimulationSimulation

Move tickets to Simulation as UI & Simulation got some sub components.

Note: See TracTickets for help on using tickets.