From 3209cd1abfb2a0ed8124bd40259858a4c0896dd5 Mon Sep 17 00:00:00 2001
From: Vaibhav Kamra <vkamra@alcion.ai>
Date: Fri, 16 Dec 2022 10:35:55 -0800
Subject: [PATCH] Revert "temporarily skip e2e smoke tests" (#1843)

Reverts alcionai/corso#1835 now that #1841 is in. That should help with the flakiness.

I will follow up with another PR after though that moves these tests to a post-CI job (we don't need to run these on every commit)
---
 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 deletions(-)

diff --git a/src/cli/backup/exchange_integration_test.go b/src/cli/backup/exchange_integration_test.go
index 0fd8994b7..d4d301375 100644
--- a/src/cli/backup/exchange_integration_test.go
+++ b/src/cli/backup/exchange_integration_test.go
@@ -50,8 +50,6 @@ type NoBackupExchangeIntegrationSuite struct {
 }
 
 func TestNoBackupExchangeIntegrationSuite(t *testing.T) {
-	t.Skip("temporary skip on e2e smoke tests")
-
 	if err := tester.RunOnAny(
 		tester.CorsoCITests,
 		tester.CorsoCLITests,
@@ -141,8 +139,6 @@ type BackupExchangeIntegrationSuite struct {
 }
 
 func TestBackupExchangeIntegrationSuite(t *testing.T) {
-	t.Skip("temporary skip on e2e smoke tests")
-
 	if err := tester.RunOnAny(
 		tester.CorsoCITests,
 		tester.CorsoCLITests,
@@ -240,8 +236,6 @@ type PreparedBackupExchangeIntegrationSuite struct {
 }
 
 func TestPreparedBackupExchangeIntegrationSuite(t *testing.T) {
-	t.Skip("temporary skip on e2e smoke tests")
-
 	if err := tester.RunOnAny(
 		tester.CorsoCITests,
 		tester.CorsoCLITests,
@@ -478,8 +472,6 @@ 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 1363493e5..956f9f075 100644
--- a/src/cli/backup/onedrive_integration_test.go
+++ b/src/cli/backup/onedrive_integration_test.go
@@ -40,8 +40,6 @@ type NoBackupOneDriveIntegrationSuite struct {
 }
 
 func TestNoBackupOneDriveIntegrationSuite(t *testing.T) {
-	t.Skip("temporary skip on e2e smoke tests")
-
 	if err := tester.RunOnAny(
 		tester.CorsoCITests,
 		tester.CorsoCLITests,
@@ -131,8 +129,6 @@ 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 ad698cd44..4abb510d1 100644
--- a/src/cli/backup/sharepoint_integration_test.go
+++ b/src/cli/backup/sharepoint_integration_test.go
@@ -40,8 +40,6 @@ type NoBackupSharePointIntegrationSuite struct {
 }
 
 func TestNoBackupSharePointIntegrationSuite(t *testing.T) {
-	t.Skip("temporary skip on e2e smoke tests")
-
 	if err := tester.RunOnAny(
 		tester.CorsoCITests,
 		tester.CorsoCLITests,
@@ -131,8 +129,6 @@ 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 d3e62e4f2..b7b3fa45f 100644
--- a/src/internal/operations/backup_test.go
+++ b/src/internal/operations/backup_test.go
@@ -314,8 +314,6 @@ 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,8 +459,6 @@ 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()
 
@@ -496,8 +492,6 @@ 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 0e800121c..e09f6febe 100644
--- a/src/internal/operations/restore_test.go
+++ b/src/internal/operations/restore_test.go
@@ -260,8 +260,6 @@ func (suite *RestoreOpIntegrationSuite) TestNewRestoreOperation() {
 }
 
 func (suite *RestoreOpIntegrationSuite) TestRestore_Run() {
-	suite.T().Skip("temporary skip on e2e smoke tests")
-
 	ctx, flush := tester.NewContext()
 	defer flush()