Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Test run customization documentation #226

Merged

Conversation

mw-hrastega
Copy link
Member

No description provided.

CONFIGDOC.md Outdated
To customize your test run, select options in the **Customize Test Run** section:

* To apply strict checks when running the tests, select **Strict**. When you select this option, the plugin generates a qualification failure whenever a test issues a warning. Selecting **Strict** is the same as specifying the `Strict` name-value argument of [`runtests`](https://www.mathworks.com/help/matlab/ref/runtests.html) as `true`.
* To run tests in parallel (requires Parallel Computing Toolbox™), select **Use Parallel**. This feature might not be compatible with certain options, in which case, the plugin runs the tests in serial regardless of your selection. Selecting **Use parallel** is the same as specifying the `UseParallel` name-value argument of `runtests` as `true`.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we Add the disclaimer that this will work only if user has valid PCT licence installed ?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Strengthened the tone in the new version. Let me know what you think.

CONFIGDOC.md Outdated
* To apply strict checks when running the tests, select **Strict**. When you select this option, the plugin generates a qualification failure whenever a test issues a warning. Selecting **Strict** is the same as specifying the `Strict` name-value argument of [`runtests`](https://www.mathworks.com/help/matlab/ref/runtests.html) as `true`.
* To run tests in parallel (requires Parallel Computing Toolbox™), select **Use Parallel**. This feature might not be compatible with certain options, in which case, the plugin runs the tests in serial regardless of your selection. Selecting **Use parallel** is the same as specifying the `UseParallel` name-value argument of `runtests` as `true`.
* To control the amount of event detail displayed for your test run, select a value from the **Output detail** list. Selecting a value for this option is the same as specifying the `OutputDetail` name-value argument of `runtests` as that value.
* To display diagnostics logged by the [`log (TestCase)`](https://www.mathworks.com/help/matlab/ref/matlab.unittest.testcase.log.html) or [`log (Fixture)`](https://www.mathworks.com/help/matlab/ref/matlab.unittest.fixtures.fixture.log.html) methods at a specified verbosity level and below, select a value from the **Logging level** list. Selecting a value for this option is the same as specifying the `LoggingLevel` name-value argument of `runtests` as that value.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For both Logging level and Output details should we mention what default selection does ?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added information on default behavior.

CONFIGDOC.md Outdated

Input | Description
------------------------- | ---------------
`sourceFolder` | (Optional) Location of the folder containing source code, relative to the project root folder. The specified folder and its subfolders are added to the top of the MATLAB search path. If you specify `sourceFolder` and then generate a coverage report, the plugin uses only the source code in the specified folder and its subfolders to generate the report.<br/>**Example:** `['source']`<br/>**Example:** `['source/folderA', 'source/folderB']`
`selectByFolder` | (Optional) Location of the folder used to select test suite elements, relative to the project root folder. To create a test suite, the plugin uses only the tests in the specified folder and its subfolders.<br/>**Example:** `['test']`<br/>**Example:** `['test/folderA', 'test/folderB']`
`selectByTag` | (Optional) Test tag used to select test suite elements. To create a test suite, the plugin uses only the test elements with the specified tag.<br/>**Example:** `'FeatureA'`
`strict` | (Optional) Whether to apply strict checks when running the tests, specified as `false` or `true`. By default, the value is `false`. If you specify a value of `true`, the plugin generates a qualification failure whenever a test issues a warning.
`useParallel` | (Optional) Whether to run tests in parallel, specified as `false` or `true`. By default, the value is `false` and tests run in serial. If the test runner configuration is suited for parallelization, you can specify a value of `true` to run tests in parallel. This input requires a Parallel Computing Toolbox&trade; license.
`outputDetail` | (Optional) Amount of event detail displayed for the test run, specified as `none`, `terse`, `concise`, `detailed`, or `verbose`. Selecting a value for this input is the same as specifying the `OutputDetail` name-value argument of [`runtests`](https://www.mathworks.com/help/matlab/ref/runtests.html) as that value.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

there is an issue to consider here that when someone specify the OutPutDetail or Logging level in pipeline then only values supported are as described in document. Default as value will throw an warnning and fall backs to the default state of the level which is 4 or detailed. closes issue #224 . per me we can clearly mention this in quick note section. so that we can avoid this as Xbug

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added information on the default behavior. Not sure whether we need a note because the users are provided with a finite list of options and should choose from that list.

Copy link
Member

@nbhoski nbhoski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me Ship it !

@mw-hrastega mw-hrastega merged commit 2ebfe04 into 2.8.1-SNAPSHOT-Qualification Oct 10, 2022
@mw-hrastega mw-hrastega deleted the TestRunCustomization-Documentation branch February 14, 2023 19:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants