Skip to content

Commit 2c09a78

Browse files
arzonusinstamitch
authored andcommitted
test: add DescribeCluster integration test (cadence-workflow#7344)
<!-- Describe what has changed in this PR --> **What changed?** Integration test for DescribeCluster has been added. <!-- Tell your future self why have you made these changes --> **Why?** The lack of this test caused this #[6338](cadence-workflow#6338), and was fixed in cadence-workflow#6350 The responses' variables depend on each run, so they cannot be asserted. The test asserts only that the API can return the response without errors. <!-- How have you verified this change? Tested locally? Added a unit test? Checked in staging env? --> **How did you test it?** Run integration tests <!-- Assuming the worst case, what can be broken when deploying this change to production? --> **Potential risks** No risks Signed-off-by: Seva Kaloshin <[email protected]> Signed-off-by: Mitch O'Dea <[email protected]>
1 parent 4c294ab commit 2c09a78

File tree

1 file changed

+10
-0
lines changed

1 file changed

+10
-0
lines changed

host/integration_test.go

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4470,3 +4470,13 @@ func (s *IntegrationSuite) sendSignal(domainName string, execution *types.Workfl
44704470
Identity: identity,
44714471
})
44724472
}
4473+
4474+
// TestDescribeCluster tests that DescribeCluster API returns a valid response without error
4475+
func (s *IntegrationSuite) TestDescribeCluster() {
4476+
ctx, cancel := createContext()
4477+
defer cancel()
4478+
4479+
response, err := s.AdminClient.DescribeCluster(ctx)
4480+
s.Require().NoError(err)
4481+
s.Require().NotNil(response)
4482+
}

0 commit comments

Comments
 (0)