Add cmd line arg to suppress header output

[tophmatthews]

Combined #29663 into this. Changed the header output stream to _console, and added command line arg to suppress header output. Added command line arg to MooseTestApp to optionally print a header to allow testing.

Closes #29662

[moosebuild]

Job Coverage, step Generate coverage on 510f565 wanted to post the following:

Framework coverage

	f5a14f	#29669 510f56
	Total	Total	+/-	New
Rate	85.23%	85.24%	+0.00%	100.00%
Hits	108490	108494	+4	5
Misses	18794	18790	-4	0

Diff coverage report

Full coverage report

Modules coverage

Coverage did not change

Full coverage reports

Reports

• framework
• chemical_reactions
• combined
• contact
• electromagnetics
• external_petsc_solver
• fluid_properties
• fsi
• functional_expansion_tools
• geochemistry
• heat_transfer
• level_set
• misc
• navier_stokes
• optimization
• peridynamics
• phase_field
• porous_flow
• ray_tracing
• rdg
• reactor
• richards
• scalar_transport
• solid_mechanics
• solid_properties
• stochastic_tools
• thermal_hydraulics
• xfem

This comment will be updated on new commits.

[tophmatthews]

Can someone pass judgement on including a test on this @GiudGiud @loganharbour @pbehne ? The simplest is a header() in MooseTestApp like 9e54f63

[loganharbour]

Can someone pass judgement on including a test on this @GiudGiud @loganharbour @pbehne ? The simplest is a header() in MooseTestApp like 9e54f63

I think we should just make another dummy app for it so that we don't get a header in every moose_test-opt run

[tophmatthews]

Could also put a header in MiscApp

[loganharbour]

Could also put a header in MiscApp

I don't like testing framework content outside of the framework. misc testing doesn't contribute to framework coverage for example.

If we don't want another app, we can just do something silly like --test-set-header="foo" in MooseTestApp and then have header() return that result. So the header only shows up when we set it to. Goofy cause we'll basically have

--suppress-header --test-set-header="foo"

but we've done worse.

[moosebuild]

Job Documentation, step Docs: sync website on 510f565 wanted to post the following:

View the site here

This comment will be updated on new commits.

[tophmatthews]

--test-set-header="foo"

Nice! I like this option! I kinda need the fix for setGlobalCommandLineParam @loganharbour is working on to cascade to all subapp levels first to do it right, but I'll whip something up before that fix is done.

[tophmatthews]

Pretty easy to put

  params.addCommandLineParam<std::string>(
      "append_header", "--append_header <header>", "String to print at top of console output");

in MooseApp.C too for someone that may want that optoin...any reason to or just leave it in MooseTestApp.C?

[GiudGiud]

It's not very useful imo

[tophmatthews]

This works for cascading to subapps @loganharbour ...maybe that previous fix you suggested did work 😬

[tophmatthews]

Alright, ready for review!

[YaqiWang]

Does this #29671 fix the missing prefix in the indentation reported in the issue?

[tophmatthews]

Does this #29671 fix the missing prefix in the indentation reported in the issue?

Yeah mostly, but this still adds the capability to turn off the header.

This is good to go for review @loganharbour . Thanks!

[tophmatthews]

Thanks @loganharbour !