From cd41d2fbceae34734bc61ae6c2052ff689847633 Mon Sep 17 00:00:00 2001 From: ashmrtn <3891298+ashmrtn@users.noreply.github.com> Date: Mon, 25 Mar 2024 13:16:43 -0700 Subject: [PATCH] Reduce test to just a single site to avoid failure (#5274) #### Does this PR need a docs update or release note? - [ ] :white_check_mark: Yes, it's included - [ ] :clock1: Yes, but in a later PR - [x] :no_entry: No #### Type of change - [ ] :sunflower: Feature - [x] :bug: Bugfix - [ ] :world_map: Documentation - [x] :robot: Supportability/Tests - [ ] :computer: CI/Deployment - [ ] :broom: Tech Debt/Cleanup #### Test Plan - [x] :muscle: Manual - [ ] :zap: Unit test - [ ] :green_heart: E2E --- src/pkg/services/m365/sites_test.go | 19 +++++-------------- 1 file changed, 5 insertions(+), 14 deletions(-) diff --git a/src/pkg/services/m365/sites_test.go b/src/pkg/services/m365/sites_test.go index f2464ee96..0c3fc103d 100644 --- a/src/pkg/services/m365/sites_test.go +++ b/src/pkg/services/m365/sites_test.go @@ -76,20 +76,11 @@ func (suite *siteIntegrationSuite) TestSites_GetByID() { ctx, flush := tester.NewContext(t) defer flush() - sites, err := suite.cli.Sites(ctx, fault.New(true)) - assert.NoError(t, err, clues.ToCore(err)) - assert.NotEmpty(t, sites) - - for _, s := range sites { - suite.Run("site_"+s.ID, func() { - t := suite.T() - site, err := suite.cli.SiteByID(ctx, s.ID) - require.NoError(t, err, clues.ToCore(err)) - assert.NotEmpty(t, site.WebURL) - assert.NotEmpty(t, site.ID) - assert.NotEmpty(t, site.OwnerType) - }) - } + site, err := suite.cli.SiteByID(ctx, suite.m365.Site.ID) + require.NoError(t, err, clues.ToCore(err)) + assert.NotEmpty(t, site.WebURL) + assert.NotEmpty(t, site.ID) + assert.NotEmpty(t, site.OwnerType) } // ---------------------------------------------------------------------------