Feature/disable test history

[AlexNetcare]

Adding a new Property on a page to disable Test Histories for that page. This is turned off by default.
This might be taking care of this Request #1488
The new Property does not work recursive though.

[six42]

Hi Alex,

Are you aware of the existing nohistory parameter. This works recursive.
https://fitnesse.org/FitNesse/UserGuide/AdministeringFitNesse/RestfulServices.html

[AlexNetcare]

Hi @six42,
I actually was not aware of that parameter. But after looking what it does, it is not really usable for our Testers.
The idea of my change was, that you could disable the Test History of a single page. So that when you run a suite, those disabled Tests do not generate a Test History. That is because some of our Tests only generate informative stuff which does not need to have a History.
Also in our case, our Testers execute Tests manually most of the time. So when they press on "Suite" or "Test" the nohistory parameter would not be added.
I also saw that my logic didn't do anything for Suites. So I added some logic, that when the property to disable Test Histories is set on a Suite, it will follow the same rule as the nohistory parameter.

[six42]

Would you like the feature to work recursively?

I am wondering if this feature should be configured via a page property as you did or rather by a variable.
Most parameters which steer the test system are defined as variables like

!define slim
!define slim.port {1}

Variables have the advantage that they can be defined in many ways:

• via plugin.properties for the whole wiki,
• as parameters to a single REST request
• or on a page with define.

They always work recursively.

Page properties so far define the page security and the menu layout, the type of page and the source of the data.

What do others think when should be something be configured via a property and when via a variable?
Would be good to have some guiding rules.

[AlexNetcare]

Sorry for the late answer but I had some discussions with colleagues about this. To cut it short, from our side we would like to keep it as a page property for now.
We don't really want this variable to be used on sub pages since some of them should still generate test histories.
But I would also appreciate some more input from others. If changing the page properties here is the wrong place, then I would change to page variables and see if that works for our Testers.

[AlexNetcare]

This has been a while now. Is it possible to get any new Input on this?

[fhoeben]

@six42 any further thoughts?