From 740b193415e926425da211d2dc5e82bf990fb879 Mon Sep 17 00:00:00 2001 From: ashmrtn Date: Mon, 24 Oct 2022 12:12:49 -0700 Subject: [PATCH] Remove flakey mail test (#1309) ## Description Test flakes because CI cleanup job goes through and deletes some folders the folder cache ends up looking for. Solve this by only selecting specific mail folders instead of all of them. Will need to rely on other tests to check if fetching all mail folders works. As only some folders are checked for though, overall impact of removing this test should be minor as the more targeted test is still present Mistakenly re-added by #1134 ## Type of change - [ ] :sunflower: Feature - [x] :bug: Bugfix - [ ] :world_map: Documentation - [ ] :robot: Test - [ ] :computer: CI/Deployment - [ ] :hamster: Trivial/Minor ## Issue(s) * closes #1222 ## Test Plan - [ ] :muscle: Manual - [x] :zap: Unit test - [ ] :green_heart: E2E --- src/internal/connector/graph_connector_test.go | 8 -------- 1 file changed, 8 deletions(-) diff --git a/src/internal/connector/graph_connector_test.go b/src/internal/connector/graph_connector_test.go index 7c193ce68..112f4cfa1 100644 --- a/src/internal/connector/graph_connector_test.go +++ b/src/internal/connector/graph_connector_test.go @@ -371,14 +371,6 @@ func (suite *GraphConnectorIntegrationSuite) TestMailFetch() { scope selectors.ExchangeScope folderNames map[string]struct{} }{ - { - name: "Mail Iterative Check", - scope: selectors.NewExchangeBackup().MailFolders([]string{userID}, selectors.Any())[0], - folderNames: map[string]struct{}{ - exchange.DefaultMailFolder: {}, - "Sent Items": {}, - }, - }, { name: "Folder Iterative Check Mail", scope: selectors.NewExchangeBackup().MailFolders(