refactor: Reduce repetition of parsing of show/hide options

[claremacrae]

Types of changes

Internal changes:

• Refactor (prefix: refactor - non-breaking change which only improves the design or structure of existing code, and making no changes to its external behaviour)
• Tests (prefix: test - additions and improvements to unit tests and the smoke tests)

Description

Refactor to reduce repetition in QueryLayoutOptions and TaskLayoutComponent.

Motivation and Context

To make the code easier to read and to modify.

How has this been tested?

• Automated tests

Specifically, I added full tests of the existing code first, and then refactored the original code.

I am conscious that there is now some similarity between code in the tests and the implementation - but the tests came first, to ensure I didn't break the code during refactoring.

Screenshots (if appropriate)

Checklist

• My code follows the code style of this project and passes yarn run lint.
• My change has adequate Unit Test coverage.

Terms

• My contribution follows this project's contributing guide
• I agree to follow this project's Code of Conduct

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud