Skip to content

Conversation

@joperezr
Copy link
Member

Related to #67793.

An outerloop test that was added with the IgnoreCase work started failing some of our outerloop legs, so this PR is disabling that for now against the already opened issue. I will figure out if the test can be re-written in order to work for those platforms, or alternatively disable it only in those platforms and re-enable it on the rest of them as a follow-up.

cc: @stephentoub

@ghost
Copy link

ghost commented May 23, 2022

Tagging subscribers to this area: @dotnet/area-system-text-regularexpressions
See info in area-owners.md if you want to be subscribed.

Issue Details

Related to #67793.

An outerloop test that was added with the IgnoreCase work started failing some of our outerloop legs, so this PR is disabling that for now against the already opened issue. I will figure out if the test can be re-written in order to work for those platforms, or alternatively disable it only in those platforms and re-enable it on the rest of them as a follow-up.

cc: @stephentoub

Author: joperezr
Assignees: -
Labels:

area-System.Text.RegularExpressions

Milestone: -

@joperezr
Copy link
Member Author

/azp run runtime-libraries-coreclr outerloop

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@stephentoub
Copy link
Member

I will figure out if the test can be re-written in order to work for those platforms, or alternatively disable it only in those platforms and re-enable it on the rest of them as a follow-up.

Which platforms is it? Do we know why it's consistently failing there?

@joperezr
Copy link
Member Author

The one I've seen it most consistent reproing is OSX. I've also seen this happening on some Linux, but haven't been able to draw conclusions as to why it fails in some but not others so that is part of the work that I plan on doing next.

@joperezr
Copy link
Member Author

Checked the outerloop legs and all failures now seem to be non-Regex related, so I'll go ahead and merge this.

@joperezr joperezr merged commit 9859f70 into dotnet:main May 24, 2022
@joperezr joperezr deleted the DisableTest branch May 24, 2022 01:22
@ghost ghost locked as resolved and limited conversation to collaborators Jun 23, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants