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

Update some of the documentation for APIs removed in NUnit 4 #986

Closed
stevenaw opened this issue Nov 11, 2024 · 5 comments · Fixed by #987
Closed

Update some of the documentation for APIs removed in NUnit 4 #986

stevenaw opened this issue Nov 11, 2024 · 5 comments · Fixed by #987

Comments

@stevenaw
Copy link
Member

I noticed today that the ListMapper docs have the following warning at the top:

ListMapper has been deprecated as of NUnit Framework 3.13.2. The functionality will be removed in NUnit Framework 4.0.

We may want to consider changing "will be removed" to present tense "has been removed".

@SeanKilleen
Copy link
Member

Nice catch - I'll do a quick search for this and get a PR

@stevenaw
Copy link
Member Author

Thanks @SeanKilleen !

@SeanKilleen
Copy link
Member

@stevenaw there are two other spots mentioned -- in the console command line.

|--params=PARAMETER, --p=PARAMETER | (**NOTE:** _this is deprecated and will be removed in a future release. Please use --testparam instead_.) A test PARAMETER specified in the form NAME=VALUE for consumption by tests. Multiple parameters may be specified, separated by semicolons or by repeating the --params option multiple times. Case-sensitive. |

And

|--labels=VALUE| Specify whether to write test case names to the output. Values: Off, OnOutputOnly, Before, After, BeforeAndAfter, On, or All. If not specified, defaults to Off. **Off**: No labeling is used. Both normal and immediate output appear in the order produced - i.e. immediate first. **OnOutputOnly**: A label appears before each sequence of output lines from the same test. Since tests may be run in parallel, output from different tests may be intermixed. **Before**: A label appears at the start of every test, whether it produces output or not. Additional labels are produced as needed if interspersed output takes place, just as for--labels=OnOutputOnly. **labels=After**: A label appears at the end of every test, whether it produced output or not. This label includes the pass/fail status of the test in addition to its name. Additional labels are produced as needed if there is any output, just as for --labels=OnOutputOnly. **BeforeAndAfter** A label appears at the start and end of every test, whether it produced output or not. The label at the end of the test includes the pass/fail status of the test in addition to its name. Additional labels are produced as needed if interspersed output takes place, just as for --labels=OnOutputOnly. **On**. Synonym for --labels=OnOutputOnly, now deprecated (will be removed in a future release). **All**: Synonym for --labels=Before, now deprecated (will be removed in a future release).|

Not sure if we removed --params or the On and All values for --labels. Can you confirm?

@stevenaw
Copy link
Member Author

Thanks for checking for these @SeanKilleen
I don't have as much working knowledge on the console side but I think those should stay as-is. It looks like those have been identified for restructuring or removal within version 4 of the console runner, which hasn't yet been released (nunit/nunit-console#752 (comment))

Strictly with regards to the labels case, I've found the PR to deprecate those options but can't find any PR which removes them outright.

@SeanKilleen
Copy link
Member

Thank so much for looking into this, @stevenaw! I was strapped for time and couldn't get it done. I'll merge the PR as-is to fix the one instance you found, which will close this.

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 a pull request may close this issue.

2 participants