JDO
  1. JDO
  2. JDO-524

SELECT lastname INTO java.lang.Long - is a negative test?

    Details

      Description

      The second negative test query in org.apache.jdo.tck.query.result.ResultClassRequirements is:
      "SELECT lastname INTO java.lang.Long FROM org.apache.jdo.tck.pc.company.FullTimeEmployee".

      Exception is expected by the test, but because java.lang.Long has a constructor with one string argument - this query should be valid, unless the spec defines special rules for java.lang wrapper types as result types.

      1. JDO-524.patch
        1 kB
        Michael Bouschen

        Activity

        Ilan Kirsh created issue -
        Hide
        Craig L Russell added a comment -

        Changing the type of the selected element from lastname to something else should fix this.

        Show
        Craig L Russell added a comment - Changing the type of the selected element from lastname to something else should fix this.
        Craig L Russell made changes -
        Field Original Value New Value
        Assignee Michael Bouschen [ mbo ]
        Hide
        Michael Bouschen added a comment -

        I propose to change the SELECT clause to select the FullTimeEmployee instance, i.e.
        SELECT this INTO java.lang.Long FROM org.apache.jdo.tck.pc.company.FullTimeEmployee

        Attached you find a patch changing the query in tck2 and tck2-legacy.

        Show
        Michael Bouschen added a comment - I propose to change the SELECT clause to select the FullTimeEmployee instance, i.e. SELECT this INTO java.lang.Long FROM org.apache.jdo.tck.pc.company.FullTimeEmployee Attached you find a patch changing the query in tck2 and tck2-legacy.
        Michael Bouschen made changes -
        Attachment JDO-524.patch [ 12365675 ]
        Hide
        Craig L Russell added a comment -

        Looks good.

        Show
        Craig L Russell added a comment - Looks good.
        Hide
        Michael Bouschen added a comment -

        Checked in the patch (see revision 575469).

        Show
        Michael Bouschen added a comment - Checked in the patch (see revision 575469).
        Michael Bouschen made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Fix Version/s JDO 2 maintenance release 1 [ 12310923 ]
        Resolution Fixed [ 1 ]
        Andy Jefferson made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        7d 6h 10m 1 Michael Bouschen 13/Sep/07 22:41
        Resolved Resolved Closed Closed
        1004d 13h 29m 1 Andy Jefferson 14/Jun/10 12:10

          People

          • Assignee:
            Michael Bouschen
            Reporter:
            Ilan Kirsh
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development