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

Create 009-break-data-driven-test-into-multiple-tests-objects.md #52

Closed
wants to merge 1 commit into from
Closed

Create 009-break-data-driven-test-into-multiple-tests-objects.md #52

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
25 changes: 25 additions & 0 deletions RFCs/009-break-data-driven-test-into-multiple-tests-objects.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,25 @@
#Break data driven test into multiple test objects

##Summary
Allow data-driven tests to start and stop recording between each execution.

##Motivation
When recording videos of tests using Microsoft.VisualStudio.TestTools.DataCollection.VideoRecorder data-driven tests are treated as a single block causing every execution of the test to be within the same video. See Issue [450](https://github.com/Microsoft/testfx/issues/450).

##Detailed Design

### Requirements
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

@pvlakshm pvlakshm Oct 19, 2018

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?

Allow a switch when using DynamicDataAttribute to allow for the optional breaking up of test videos

```csharp
[DynamicData(nameof(Data), DynamicDataSourceType.Property, separate: true )]
...
public DynamicDataAttribute(string dynamicDataSourceName, DynamicDataSourceType dynamicDataSourceType = DynamicDataSourceType.Property, bool separate = false )
{
...
}
```