search

AMReX-Codes / regression_testing

The regression test routines for AMReX applications
BSD 3-Clause "New" or "Revised" License
5 stars 24 forks source link

CI: Print error reports when tests fail #114

Closed ax3l closed 2 years ago

ax3l commented 2 years ago

Print compilation messages and script output messages when they fail. This is needed for CI to spot the underlying error.

Port of https://github.com/ECP-WarpX/regression_testing/commit/8abcbe3ac29506034ac7062e30aa1facb68ceeed by @RemiLehe

ping @EZoni

zingale commented 2 years ago

can we make this an option? I run on the commandline a lot and this will really clutter things up.

ax3l commented 2 years ago

Yes, absolutely no problem :) Do you like to propose/push an option for it?

zingale commented 2 years ago

what about just adding a general verbose option to the suite?

ax3l commented 2 years ago

Sounds good to me, that would be on for our CI / non-interactive mode then :)

ax3l commented 2 years ago

With regards to #106: We could also add printing of stdout and stderr file below suite.run_test(test, base_cmd) for verbose runs.

ax3l commented 2 years ago

Option added in #116 :)

zingale commented 2 years ago

I think we need to merge main into this so the CI test passes

ax3l commented 2 years ago

Thx, yes rebased now :)

ax3l commented 2 years ago

@zingale thx, updated :)