Skip to content
This repository was archived by the owner on Oct 11, 2022. It is now read-only.

Conversation

@JDCain
Copy link

@JDCain JDCain commented Jul 31, 2018

RFC 009 for Issue 450.

@msftclas
Copy link

msftclas commented Jul 31, 2018

CLA assistant check
All CLA requirements met.

@michalkovy
Copy link

I have one other use case for the same need.

The use case is behavior-driven development using TickSpec (https://github.com/fsprojects/TickSpec). TickSpec is a .NET framework for BDD tests that allows generating tests from feature files on runtime instead of compilation time. The benefit of runtime generation is that business people can write new tests just with feature file changes when they reuse existing steps.

When you use BDD then you have many feature files which each contain several scenarios. We are able to generate on runtime when using NUnit or xUnit tests that show in Test Explorer as a test per scenario. When we try the same with MSTest we only get one test in Test Explorer. It is unusable even with just 20 scenarios and you can have many more, thousands.

1. DataTestMethod Attribute should allow the option to treat it's tests as separate test objects.
2. Should not alter current behavior for existing users.

### Proposed solution
Copy link
Contributor

Choose a reason for hiding this comment

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

There are multiple ways to data drive a test.

  • using DataRow
  • using DynamicData
  • directly extending ITestDataSource

Is your proposal for all of these approaches?

@nohwnd nohwnd changed the base branch from master to main May 13, 2021 11:59
@Evangelink
Copy link
Member

Closing this as we are moving repository to the main one. Also, I opened a ticket for us to revisit documentation of existing behaviors.

@Evangelink Evangelink closed this Oct 10, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants