BVal
  1. BVal
  2. BVAL-31

A ConstraintDefinitionException should be thrown when the resolved validator is not compatible with the annotation type

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.1-incubating
    • Fix Version/s: 0.1-incubating
    • Component/s: jsr303
    • Labels:
      None

      Description

      The spec is a bit vague in this, so the fix is the minimum required to pass it. However, in the future we might be able to be more lenient and filter the list of validators by annotation type before applying the resolution filtering.

      An additional test ConstraintCompositionTest#testOverriddenAttributesMustMatchInType() in the TCK passes, but I would say the test has a bug in the definition of FrenchZipcodeWithInvalidOverride which is making it test this scenario instead of the overridden attributes. Maybe we should report the bug to JBoss people.

        Activity

        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        12h 24m 1 Donald Woods 30/Apr/10 22:20
        Resolved Resolved Closed Closed
        707d 1h 8m 1 Matt Benson 06/Apr/12 23:29
        Matt Benson made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Hide
        Matt Benson added a comment -

        closing issues associated with existing releases

        Show
        Matt Benson added a comment - closing issues associated with existing releases
        Donald Woods made changes -
        Resolution Fixed [ 1 ]
        Status Open [ 1 ] Resolved [ 5 ]
        Fix Version/s 0.1-incubating [ 12314849 ]
        Hide
        Donald Woods added a comment -

        committed as r939818

        Show
        Donald Woods added a comment - committed as r939818
        Donald Woods made changes -
        Assignee Donald Woods [ drwoods ]
        Carlos Vara made changes -
        Field Original Value New Value
        Attachment bval-jsr303.patch [ 12443269 ]
        Hide
        Carlos Vara added a comment -

        Attached patch with extra test.

        Show
        Carlos Vara added a comment - Attached patch with extra test.
        Carlos Vara created issue -

          People

          • Assignee:
            Donald Woods
            Reporter:
            Carlos Vara
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development