chore: progress

Author: cloud-j-luna

tests/e2e/certs_cli_test.go

@@ -19,13 +19,10 @@ type certificateIntegrationTestSuite struct {
 }
 
 func (s *certificateIntegrationTestSuite) TestGeneratePublishAndRevokeServer() {
-	// TODO: Skip until CLI package bug is fixed
-	// TxRevokeServerExec calls GetTxCertRevokeCmd() (parent) instead of GetTxCertRevokeServerCmd()
 	s.T().Skip("Skipped: CLI testutil TxRevokeServerExec calls wrong command")
 }
 
 func (s *certificateIntegrationTestSuite) TestGenerateServerRequiresArguments() {
-	// TODO: Skip - CLI command argument validation may have changed
 	s.T().Skip("Skipped: CLI command argument requirements may have changed")
 }
 
@@ -100,14 +97,10 @@ func (s *certificateIntegrationTestSuite) TestGeneratePublishAndRevokeClient() {
 }
 
 func (s *certificateIntegrationTestSuite) TestGenerateAndRevokeFailsServer() {
-	// TODO: Skip until CLI package bug is fixed
-	// TxRevokeServerExec calls GetTxCertRevokeCmd() (parent) instead of GetTxCertRevokeServerCmd()
 	s.T().Skip("Skipped: CLI testutil TxRevokeServerExec calls wrong command")
 }
 
 func (s *certificateIntegrationTestSuite) TestRevokeFailsServer() {
-	// TODO: Skip until CLI package bug is fixed
-	// TxRevokeServerExec calls GetTxCertRevokeCmd() (parent) instead of GetTxCertRevokeServerCmd()
 	s.T().Skip("Skipped: CLI testutil TxRevokeServerExec calls wrong command")
 }
 

tests/e2e/deployment_cli_test.go

@@ -188,10 +188,6 @@ func (s *deploymentIntegrationTestSuite) TestDeployment() {
 	s.Require().NoError(err)
 	s.Require().Len(out.Deployments, 1)
 
-	// NOTE: Deployment update test removed - updating requires changing the manifest
-	// while keeping groups the same, which needs a separate deployment-v2.yaml file
-	// with only image/service changes (not pricing/resource changes)
-
 	// test query deployments with wrong owner value
 	_, err = clitestutil.ExecQueryDeployments(
 		ctx,