Add option to filter test results in GetTestRuns to specific test outcomes

[pascalberger]

Add an option to only read test results in GetTestRuns for specific test outcomes

[yansklyarenko]

@pascalberger How would you suggest passing the test outcomes?

I'm thinking about the following signature:

public IEnumerable<AzureDevOpsTestRun> GetTestRuns(int? maxResultsPerTestRun, string testOutcomeFilter)

where testOutcomeFilter is a comma-separated list of outcomes, e.g. "Failed, Aborted", which is then translated to the appropriate IEnumerable<TestOutcome> down the stack.

Does this make sense?

[pascalberger]

@yansklyarenko what is your idea for passing a comma-separated list instead of an enumerable?

[yansklyarenko]

@pascalberger Well, I just thought it looks more intuitive from the user perspective.
Like, this:
var results = GetTestRuns(5, "Failed, Aborted");
instead of this:
var results = GetTestRuns(5, new string[] { "Failed", "Aborted" });

Does it look like an unnecessary optimization to you?

[pascalberger]

@yansklyarenko My issue with it is more that it is less "typesafe" and has the potential to be used wrong, like this for example:

var results = GetTestRuns(5, "Failed; Aborted");

[yansklyarenko]

@pascalberger Yep, that makes sense, thanks!