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

        Carlos Vara created issue -
        Hide
        Carlos Vara added a comment -

        Attached patch with extra test.

        Show
        Carlos Vara added a comment - Attached patch with extra test.
        Carlos Vara made changes -
        Field Original Value New Value
        Attachment bval-jsr303.patch [ 12443269 ]
        Donald Woods made changes -
        Assignee Donald Woods [ drwoods ]
        Hide
        Donald Woods added a comment -

        committed as r939818

        Show
        Donald Woods added a comment - committed as r939818
        Donald Woods made changes -
        Resolution Fixed [ 1 ]
        Status Open [ 1 ] Resolved [ 5 ]
        Fix Version/s 0.1-incubating [ 12314849 ]
        Hide
        Matt Benson added a comment -

        closing issues associated with existing releases

        Show
        Matt Benson added a comment - closing issues associated with existing releases
        Matt Benson made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development