[FLINK-13525] Improve Expression validation test

• Type: Improvement
• Status: Reopened
• Priority: Not a Priority
• Resolution: Unresolved
• Component/s: Table SQL / Planner
• Assignee: Unassigned
• Reporter: Jark Wu
• Created: 01/Aug/19 06:58
• Updated: 29/Jul/21 07:35

Description

Currently, the expression validation tests under org.apache.flink.table.expressions.validation are using following pattern to verify an exception should be thrown:

@Test
  def testWrongKeyType(): Unit = {
    testAllApis('f2.at(12), "f2.at(12)", "f2[12]", "FAIL")
  }

However, this only covers the first expression 'f2.at(12) throw the expected exception. All the other expression will be ignored. We should improve this. And it's nice to test the expected exception message too.