Skip to content

Commit 73eb4bc

Browse files
committed
Move queries within testCommentViewColumn for consistency
See the change in previous commit.
1 parent 119569e commit 73eb4bc

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

testing/trino-testing/src/main/java/io/trino/testing/BaseConnectorTest.java

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -3729,17 +3729,17 @@ public void testCommentViewColumn()
37293729
assertUpdate("COMMENT ON COLUMN " + view.getName() + "." + viewColumnName + " IS 'new region key comment'");
37303730
assertThat(getColumnComment(view.getName(), viewColumnName)).isEqualTo("new region key comment");
37313731

3732-
// comment updated
3732+
// comment deleted
3733+
assertUpdate("COMMENT ON COLUMN " + view.getName() + "." + viewColumnName + " IS NULL");
3734+
assertThat(getColumnComment(view.getName(), viewColumnName)).isEqualTo(null);
3735+
3736+
// comment set to non-empty value before verifying setting empty comment
37333737
assertUpdate("COMMENT ON COLUMN " + view.getName() + "." + viewColumnName + " IS 'updated region key comment'");
37343738
assertThat(getColumnComment(view.getName(), viewColumnName)).isEqualTo("updated region key comment");
37353739

37363740
// comment set to empty
37373741
assertUpdate("COMMENT ON COLUMN " + view.getName() + "." + viewColumnName + " IS ''");
37383742
assertThat(getColumnComment(view.getName(), viewColumnName)).isEqualTo("");
3739-
3740-
// comment deleted
3741-
assertUpdate("COMMENT ON COLUMN " + view.getName() + "." + viewColumnName + " IS NULL");
3742-
assertThat(getColumnComment(view.getName(), viewColumnName)).isEqualTo(null);
37433743
}
37443744
}
37453745

0 commit comments

Comments
 (0)