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

        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.
        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.
        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).

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development