Skip to content

Comments

changed NUnit LifeCycle from "InstancePerTestCase" to "SingleInstance"#1038

Open
RalfKoban wants to merge 2 commits intomasterfrom
LifeCycle_SingleInstance
Open

changed NUnit LifeCycle from "InstancePerTestCase" to "SingleInstance"#1038
RalfKoban wants to merge 2 commits intomasterfrom
LifeCycle_SingleInstance

Conversation

@RalfKoban
Copy link
Owner

No description provided.

@RalfKoban RalfKoban changed the title changed NUNit LifeCycle from "InstancePerTestCase" to "SingleInstance" changed NUnit LifeCycle from "InstancePerTestCase" to "SingleInstance" Sep 20, 2024
Repository owner deleted a comment from QodoAI-Agent Sep 20, 2024
@RalfKoban
Copy link
Owner Author

@CodiumAI-Agent /review

@QodoAI-Agent
Copy link

PR Reviewer Guide 🔍

⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ Key issues to review

Configuration Change
The change from LifeCycle.InstancePerTestCase to LifeCycle.SingleInstance could significantly affect test isolation and performance. This change means that the same instance of the test class will be used for all tests in the class, which can lead to shared state issues if not handled carefully.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: On hold

Development

Successfully merging this pull request may close these issues.

2 participants