search

vipm-io / caraya

Assertion and unit test framework for LabVIEW
Other
59 stars 33 forks source link

Added Artificial Hierarchy. #81

Closed taylorh140 closed 2 years ago

taylorh140 commented 4 years ago

Added Artificial Hierarchy to the Library this allows for the reporting hierarchy without extra VIs. I find it useful.

There was a quirk to this change in the Assert float with tolerance, all of the normal input terminals were used. So I picked a non-normal one.

CLAassistant commented 4 years ago

CLA assistant check
All committers have signed the CLA.

francois-normandin commented 2 years ago

Will not implement in main branch. This feature is a developer preference and would make for a nice extension to the UI... when the Test Engine is decoupled from the current Test Manager. At the moment, the engine is running in the main UI, so any changes to the views is susceptible to impact lots of developers.

Also, re: floating precision terminal, every pull request should have a single change.

@taylorh140 I hope that Artificial Hierarchy can be easily added to the Test Result viewer in Caraya 2.0 (still to be determined when this is meant to happen).

taylorh140 commented 2 years ago

Thanks for looking at this and closing it. It's been a minute and I had nearly forgotten about it. I'll have to take a look at what you have planned for 2.0. Again thanks!