Update using-microsoft-visualstudio-testtools-unittesting-members-in-… #8920
Conversation
|
@engyebrahim : Thanks for your contribution! The author(s) have been notified to review your proposed change. |
|
Learn Build status updates of commit ae92fd8: ✅ Validation status: passed
For more details, please refer to the build report. Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report. For any questions, please:
|
|
Can you review the proposed changes? When the changes are ready for publication, add a #label:"aq-pr-triaged" |
prmerger-automator
bot
added
the
aq-pr-triaged
| @@ -239,7 +239,7 @@ public class MyOtherTestClass | |||
|
|
|||
| It's possible to control the inheritance behavior: only for current class using `InheritanceBehavior.None` or for all derived classes using `InheritanceBehavior.BeforeEachDerivedClass`. | |||
|
|
|||
| It's also possible to configure whether the class cleanup should be run at the end of the class or at the end of the assembly (current default). | |||
| It's also possible to configure whether the class cleanup should be run at the end of the class or at the end of the assembly (No longer supported starting from MSTest v4 as EndOfClass is the default and only ClassCleanup behavior). | |||
There was a problem hiding this comment.
Same as other PR. Can we make "ClassCleanup" a link? I don't think it's mentioned elsewhere in this doc, so might be confusing.
docs/test/using-microsoft-visualstudio-testtools-unittesting-members-in-unit-tests.md
Outdated
Show resolved
Hide resolved
…embers-in-unit-tests.md Co-authored-by: Mike Jones <mikejo@microsoft.com>
|
@engyebrahim : Thanks for your contribution! The author(s) have been notified to review your proposed change. |
|
Learn Build status updates of commit 2b0332f: ✅ Validation status: passed
For more details, please refer to the build report. Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report. For any questions, please:
|
|
@engyebrahim : Thanks for your contribution! The author(s) have been notified to review your proposed change. |
|
Learn Build status updates of commit 32fbfa4:
|
| File | Status | Preview URL | Details |
|---|---|---|---|
| docs/test/using-microsoft-visualstudio-testtools-unittesting-members-in-unit-tests.md | Details |
docs/test/using-microsoft-visualstudio-testtools-unittesting-members-in-unit-tests.md
- Line 242, Column 210: [Warning: file-not-found - See documentation]
Invalid file link: 'Microsoft.VisualStudio.TestTools.UnitTesting.ClassCleanupAttribute'.
For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Learn Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:
- Try searching the learn.microsoft.com contributor guides
- Post your question in the Learn support channel
|
@engyebrahim : Thanks for your contribution! The author(s) have been notified to review your proposed change. |
|
Learn Build status updates of commit 5924db4: ✅ Validation status: passed
For more details, please refer to the build report. Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report. For any questions, please:
|
|
@Mikejo5000 MSTest v4 is not yet available so maybe you want to keep this PR on-hold for now. We will ping you once v4 is released. Note that if that's fine for you, I don't mind merging it now. |
|
@Evangelink Is there a planned release date? |
|
@Mikejo5000 Not at the moment. We are discussing planning so I hope to be able to give you more info soon. |
|
@Evangelink Any updates on the release date? These are small changes, and seem OK to merge if you're sure these changes will be part of v4, when it comes. |
|
Hi @Mikejo5000 we don't know yet the release date, but yeah we are sure it'll be part of v4 so you can merge it. Thanks!! |
|
#sign-off |
|
@engyebrahim : Thanks for your contribution! The author(s) have been notified to review your proposed change. |
|
@engyebrahim @Mikejo5000 - without staging, we can't test the xref: link - which looks okay but you should verify that it works after publish. thanks. |
related to: microsoft/testfx#1574