Remove unnecessary comments from grants tests

- Remove explanatory comments that don't add value
- Code is self-explanatory without extra commentary
- Cleaner and more concise implementation

Author: sfc-gh-daniszewski

tests_integration/test_streamlit.py

@@ -142,7 +142,6 @@ def test_streamlit_grants_flow(
     alter_snowflake_yml,
 ):
     """Test that streamlit grants are properly applied during deployment."""
-    # Use the session's default role (main CI role)
     test_role = snowflake_session.role
     entity_id = "app_1"
 
@@ -170,7 +169,6 @@ def test_streamlit_grants_experimental_flow(
     alter_snowflake_yml,
 ):
     """Test that streamlit grants are properly applied during experimental deployment."""
-    # Use the session's default role (main CI role)
     test_role = snowflake_session.role
     entity_id = "app_1"
 

tests_integration/testing_utils/streamlit_utils.py

@@ -184,15 +184,13 @@ def assert_proper_url_is_returned(
         assert message.endswith(create_expected_url_suffix(entity_id, session))
 
     def verify_grants_applied(self, entity_id: str, test_role: str):
-        # Switch to test role and verify grants work
         self.setup.sql_test_helper.execute_single_sql(f"USE ROLE {test_role}")
         streamlits_with_role = self.setup.sql_test_helper.execute_single_sql(
             f"SHOW STREAMLITS LIKE '{entity_id}'"
         )
         assert (
             len(streamlits_with_role) == 1
         ), f"Role {test_role} should have USAGE access to the streamlit"
-        # No need to switch back since this is at the end of the test
 
 
 def create_expected_url_suffix(entity_id: str, session: SnowflakeConnection):