From bc316df6d93dcebcc2f170881f193fb56c2fc08e Mon Sep 17 00:00:00 2001
From: ashmrtn <ashmrtnz@alcion.ai>
Date: Wed, 19 Oct 2022 11:31:06 -0700
Subject: [PATCH] Remove flakey test (#1223)

## Description

There's enough other tests in the codebase that check the functionality of getting multiple folders that it should be fine without this one.

## Type of change

<!--- Please check the type of change your PR introduces: --->
- [ ] :sunflower: Feature
- [x] :bug: Bugfix
- [ ] :world_map: Documentation
- [x] :robot: Test
- [ ] :computer: CI/Deployment
- [ ] :hamster: Trivial/Minor

## Issue(s)

* closes #1222

## Test Plan

<!-- How will this be tested prior to merging.-->
- [ ] :muscle: Manual
- [x] :zap: Unit test
- [ ] :green_heart: E2E
---
 src/internal/connector/graph_connector_test.go | 10 ++--------
 1 file changed, 2 insertions(+), 8 deletions(-)

diff --git a/src/internal/connector/graph_connector_test.go b/src/internal/connector/graph_connector_test.go
index 5f0a6b5b7..356f41f68 100644
--- a/src/internal/connector/graph_connector_test.go
+++ b/src/internal/connector/graph_connector_test.go
@@ -333,16 +333,10 @@ func (suite *GraphConnectorIntegrationSuite) TestMailFetch() {
 		scope       selectors.ExchangeScope
 		folderNames map[string]struct{}
 	}{
-		{
-			name:  "Mail Iterative Check",
-			scope: sel.MailFolders([]string{userID}, selectors.Any())[0],
-			folderNames: map[string]struct{}{
-				exchange.DefaultMailFolder: {},
-				"Sent Items":               {},
-			},
-		},
 		{
 			name: "Folder Iterative Check Mail",
+			// Only select specific folders so the test doesn't flake when the CI
+			// cleanup task deletes things.
 			scope: sel.MailFolders(
 				[]string{userID},
 				[]string{exchange.DefaultMailFolder},