Don't try to delete test data in the test (#3547)
Data should be cleaned up by purge script when it runs so there's no need to risk test failures by trying to clean it up at the end of the test --- #### Does this PR need a docs update or release note? - [ ] ✅ Yes, it's included - [ ] 🕐 Yes, but in a later PR - [x] ⛔ No #### Type of change - [ ] 🌻 Feature - [x] 🐛 Bugfix - [ ] 🗺️ Documentation - [x] 🤖 Supportability/Tests - [ ] 💻 CI/Deployment - [ ] 🧹 Tech Debt/Cleanup #### Test Plan - [x] 💪 Manual - [ ] ⚡ Unit test - [ ] 💚 E2E
This commit is contained in:
parent
7cb01b1e27
commit
27d5889293
@ -151,10 +151,6 @@ func (suite *RestoreIntgSuite) TestRestoreEvent() {
|
||||
}
|
||||
}
|
||||
|
||||
type containerDeleter interface {
|
||||
DeleteContainer(context.Context, string, string) error
|
||||
}
|
||||
|
||||
// TestRestoreExchangeObject verifies path.Category usage for restored objects
|
||||
func (suite *RestoreIntgSuite) TestRestoreExchangeObject() {
|
||||
t := suite.T()
|
||||
@ -165,12 +161,6 @@ func (suite *RestoreIntgSuite) TestRestoreExchangeObject() {
|
||||
service, err := createService(m365)
|
||||
require.NoError(t, err, clues.ToCore(err))
|
||||
|
||||
deleters := map[path.CategoryType]containerDeleter{
|
||||
path.EmailCategory: suite.ac.Mail(),
|
||||
path.ContactsCategory: suite.ac.Contacts(),
|
||||
path.EventsCategory: suite.ac.Events(),
|
||||
}
|
||||
|
||||
userID := tester.M365UserID(suite.T())
|
||||
|
||||
tests := []struct {
|
||||
@ -379,10 +369,6 @@ func (suite *RestoreIntgSuite) TestRestoreExchangeObject() {
|
||||
fault.New(true))
|
||||
assert.NoError(t, err, clues.ToCore(err))
|
||||
assert.NotNil(t, info, "item info was not populated")
|
||||
assert.NotNil(t, deleters)
|
||||
|
||||
err = deleters[test.category].DeleteContainer(ctx, userID, destination)
|
||||
assert.NoError(t, err, clues.ToCore(err))
|
||||
})
|
||||
}
|
||||
}
|
||||
|
||||
Loading…
x
Reference in New Issue
Block a user