esa / LADDS

Large-scale Deterministic Debris Simulation - Codebase for the ARIADNA Study between TU Munich and ESA's Advanced Concepts Team.
GNU General Public License v3.0
5 stars 3 forks source link

Feature simulation test #78

Open albertNos opened 2 years ago

albertNos commented 2 years ago

featureNotebook -> featureSimulationTest

albertNos commented 2 years ago

commit e7ca12c: build successful commit d50505e: add constellation folders to tests/data -> build fails locally the build works (as well as the test) why?

FG-TUM commented 2 years ago

commit e7ca12c: build successful commit d50505e: add constellation folders to tests/data -> build fails locally the build works (as well as the test) why?

You can see the logs when you click on "Details" next to the failing Actions:

/home/runner/work/LADDS/LADDS/tests/testladds/SimulationTest.cpp:512:41: error: non-const lvalue reference to type 'ConfigReader' cannot bind to a value of unrelated type 'YAML::Node'
  auto autopas = simulation.initAutoPas(config);

Seems like you copy pasted the old tests that work with the plain YAML node and not the new ones that use the wrapper. Would be awesome if we could at some point squash all those tests to some generated test template to reduce code duplication and to avoid these kinds of problems.