Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Convert workspaces and new document formatting integration tests to new framework #63610

Merged
merged 6 commits into from
Aug 30, 2022

Conversation

dibarbet
Copy link
Member

@dibarbet dibarbet commented Aug 26, 2022

These were having restore issues in the scouting queue on the old framework. Running them locally showed that the new framework is much less flaky.

Also appears to resolve #34264

@dibarbet dibarbet requested a review from a team as a code owner August 26, 2022 00:59
@@ -65,9 +66,7 @@ public static string GetStringFromBulkContent(IEnumerable<object> content)
private static string GetStringFromTextBlock(TextBlock textBlock)
{
if (!string.IsNullOrEmpty(textBlock.Text))
{
Copy link
Member Author

Choose a reason for hiding this comment

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

this appears to have been autoformatted after changing the namespace

@dibarbet dibarbet changed the title Convert workspaces integration tests to new framework Convert workspaces and new document formatting integration tests to new framework Aug 26, 2022
@dibarbet dibarbet marked this pull request as draft August 26, 2022 08:10
@dibarbet
Copy link
Member Author

Ended up disabling the new doc formatting tests - they are still hitting restore issues consistently on CI (though I've never hit it locally even on 100 iterations on my machine). In order to move integration tests forward to the scouting queue I've disabled them

@dibarbet dibarbet marked this pull request as ready for review August 26, 2022 22:28
@dibarbet dibarbet requested a review from a team as a code owner August 27, 2022 00:46
@dibarbet dibarbet force-pushed the dev/dibarbet/convert_integration_tests branch 3 times, most recently from f06b026 to af38f19 Compare August 29, 2022 20:12
@dibarbet dibarbet force-pushed the dev/dibarbet/convert_integration_tests branch from af38f19 to ea12594 Compare August 29, 2022 22:30
Copy link
Member

@JoeRobich JoeRobich left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@dibarbet dibarbet merged commit 879bf68 into main Aug 30, 2022
@dibarbet dibarbet deleted the dev/dibarbet/convert_integration_tests branch August 30, 2022 23:40
@ghost ghost added this to the Next milestone Aug 30, 2022
@Cosifne Cosifne modified the milestones: Next, 17.4 P3 Sep 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

WorkspacesNetCore.MetadataReference should not need to wait for all asynchronous operations
3 participants