From 144d047568c9190ea114135ae3025eebbe95a44c Mon Sep 17 00:00:00 2001
From: Keepers <ryanfkeepers@gmail.com>
Date: Fri, 16 Dec 2022 00:51:07 -0700
Subject: [PATCH] temporarily skip e2e smoke tests (#1835)

## Description

These tests have the highes rate of timeout
based failures, while providing the most loose
gurantees about system behavior.  This will
temporarily disable them while we consider
separating our suite between integration
and e2e smoke sets.

## Does this PR need a docs update or release note?

- [x] :no_entry: No

## Type of change

- [x] :robot: Test

## Test Plan

- [x] :green_heart: E2E
---
 src/cli/backup/exchange_integration_test.go   | 8 ++++++++
 src/cli/backup/onedrive_integration_test.go   | 4 ++++
 src/cli/backup/sharepoint_integration_test.go | 4 ++++
 src/internal/operations/backup_test.go        | 6 ++++++
 src/internal/operations/restore_test.go       | 2 ++
 5 files changed, 24 insertions(+)

diff --git a/src/cli/backup/exchange_integration_test.go b/src/cli/backup/exchange_integration_test.go
index d4d301375..0fd8994b7 100644
--- a/src/cli/backup/exchange_integration_test.go
+++ b/src/cli/backup/exchange_integration_test.go
@@ -50,6 +50,8 @@ type NoBackupExchangeIntegrationSuite struct {
 }
 
 func TestNoBackupExchangeIntegrationSuite(t *testing.T) {
+	t.Skip("temporary skip on e2e smoke tests")
+
 	if err := tester.RunOnAny(
 		tester.CorsoCITests,
 		tester.CorsoCLITests,
@@ -139,6 +141,8 @@ type BackupExchangeIntegrationSuite struct {
 }
 
 func TestBackupExchangeIntegrationSuite(t *testing.T) {
+	t.Skip("temporary skip on e2e smoke tests")
+
 	if err := tester.RunOnAny(
 		tester.CorsoCITests,
 		tester.CorsoCLITests,
@@ -236,6 +240,8 @@ type PreparedBackupExchangeIntegrationSuite struct {
 }
 
 func TestPreparedBackupExchangeIntegrationSuite(t *testing.T) {
+	t.Skip("temporary skip on e2e smoke tests")
+
 	if err := tester.RunOnAny(
 		tester.CorsoCITests,
 		tester.CorsoCLITests,
@@ -472,6 +478,8 @@ type BackupDeleteExchangeIntegrationSuite struct {
 }
 
 func TestBackupDeleteExchangeIntegrationSuite(t *testing.T) {
+	t.Skip("temporary skip on e2e smoke tests")
+
 	if err := tester.RunOnAny(
 		tester.CorsoCITests,
 		tester.CorsoCLITests,
diff --git a/src/cli/backup/onedrive_integration_test.go b/src/cli/backup/onedrive_integration_test.go
index 956f9f075..1363493e5 100644
--- a/src/cli/backup/onedrive_integration_test.go
+++ b/src/cli/backup/onedrive_integration_test.go
@@ -40,6 +40,8 @@ type NoBackupOneDriveIntegrationSuite struct {
 }
 
 func TestNoBackupOneDriveIntegrationSuite(t *testing.T) {
+	t.Skip("temporary skip on e2e smoke tests")
+
 	if err := tester.RunOnAny(
 		tester.CorsoCITests,
 		tester.CorsoCLITests,
@@ -129,6 +131,8 @@ type BackupDeleteOneDriveIntegrationSuite struct {
 }
 
 func TestBackupDeleteOneDriveIntegrationSuite(t *testing.T) {
+	t.Skip("temporary skip on e2e smoke tests")
+
 	if err := tester.RunOnAny(
 		tester.CorsoCITests,
 		tester.CorsoCLITests,
diff --git a/src/cli/backup/sharepoint_integration_test.go b/src/cli/backup/sharepoint_integration_test.go
index 4abb510d1..ad698cd44 100644
--- a/src/cli/backup/sharepoint_integration_test.go
+++ b/src/cli/backup/sharepoint_integration_test.go
@@ -40,6 +40,8 @@ type NoBackupSharePointIntegrationSuite struct {
 }
 
 func TestNoBackupSharePointIntegrationSuite(t *testing.T) {
+	t.Skip("temporary skip on e2e smoke tests")
+
 	if err := tester.RunOnAny(
 		tester.CorsoCITests,
 		tester.CorsoCLITests,
@@ -129,6 +131,8 @@ type BackupDeleteSharePointIntegrationSuite struct {
 }
 
 func TestBackupDeleteSharePointIntegrationSuite(t *testing.T) {
+	t.Skip("temporary skip on e2e smoke tests")
+
 	if err := tester.RunOnAny(
 		tester.CorsoCITests,
 		tester.CorsoCLITests,
diff --git a/src/internal/operations/backup_test.go b/src/internal/operations/backup_test.go
index aacac38ea..b56d19b0b 100644
--- a/src/internal/operations/backup_test.go
+++ b/src/internal/operations/backup_test.go
@@ -316,6 +316,8 @@ func (suite *BackupOpIntegrationSuite) TestNewBackupOperation() {
 // TestBackup_Run ensures that Integration Testing works
 // for the following scopes: Contacts, Events, and Mail
 func (suite *BackupOpIntegrationSuite) TestBackup_Run_exchange() {
+	suite.T().Skip("temporary skip on e2e smoke tests")
+
 	ctx, flush := tester.NewContext()
 	defer flush()
 
@@ -461,6 +463,8 @@ func (suite *BackupOpIntegrationSuite) TestBackup_Run_exchange() {
 }
 
 func (suite *BackupOpIntegrationSuite) TestBackup_Run_oneDrive() {
+	suite.T().Skip("temporary skip on e2e smoke tests")
+
 	ctx, flush := tester.NewContext()
 	defer flush()
 
@@ -494,6 +498,8 @@ func (suite *BackupOpIntegrationSuite) TestBackup_Run_oneDrive() {
 }
 
 func (suite *BackupOpIntegrationSuite) TestBackup_Run_sharePoint() {
+	suite.T().Skip("temporary skip on e2e smoke tests")
+
 	ctx, flush := tester.NewContext()
 	defer flush()
 
diff --git a/src/internal/operations/restore_test.go b/src/internal/operations/restore_test.go
index 67f02946d..e04bff348 100644
--- a/src/internal/operations/restore_test.go
+++ b/src/internal/operations/restore_test.go
@@ -260,6 +260,8 @@ func (suite *RestoreOpIntegrationSuite) TestNewRestoreOperation() {
 }
 
 func (suite *RestoreOpIntegrationSuite) TestRestore_Run() {
+	suite.T().Skip("temporary skip on e2e smoke tests")
+
 	ctx, flush := tester.NewContext()
 	defer flush()