Details

    • Type: Sub-task Sub-task
    • Status: Resolved Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: JDO queries, JDO tests
    • Labels:
      None
    • Number of attachments :
      3

      Description

      First there should be test cases, that are fast to start, and that display the current problems of ParseTreeWalker to indicate if the problems are gone. These tests should be in the cpa package.

      1. patch-C3116-20110627.txt
        5 kB
        Ralf Joachim
      2. patch-tests-TreeWalker-110625.txt
        48 kB
        Johannes Venzke
      3. patch-tests-TreeWalker-110627.txt
        10 kB
        Johannes Venzke

        Activity

        Hide
        Johannes Venzke added a comment -

        Created Tests for CastorParameters and COUNT() function.

        I wondered, if we shouldn't implement support for SELECT and COUNT queries. It it impossible at the moment, but maybe we should create Tests already.

        Show
        Johannes Venzke added a comment - Created Tests for CastorParameters and COUNT() function. I wondered, if we shouldn't implement support for SELECT and COUNT queries. It it impossible at the moment, but maybe we should create Tests already.
        Hide
        Ralf Joachim added a comment -

        Could you please update patch to current SVN head.

        Show
        Ralf Joachim added a comment - Could you please update patch to current SVN head.
        Hide
        Johannes Venzke added a comment -

        Recreated patches according to SVN changes.

        Show
        Johannes Venzke added a comment - Recreated patches according to SVN changes.
        Hide
        Ralf Joachim added a comment -

        Will commit this patch as is. It has the following modifications to the original patch of Johannes:

        • I prefer test to extend TestCase.
        • New tests are commented out as the required features are not supported yet.
        • EJBQL will never support COUNT or parameters starting with $. At CastorQL they are needed for backward compatibility.
        Show
        Ralf Joachim added a comment - Will commit this patch as is. It has the following modifications to the original patch of Johannes: I prefer test to extend TestCase. New tests are commented out as the required features are not supported yet. EJBQL will never support COUNT or parameters starting with $. At CastorQL they are needed for backward compatibility.
        Hide
        Ralf Joachim added a comment -

        To prevent mixing up features of CastorQl and EJBQL in the future we should work on them in seperate issues.

        Show
        Ralf Joachim added a comment - To prevent mixing up features of CastorQl and EJBQL in the future we should work on them in seperate issues.

          People

          • Assignee:
            Johannes Venzke
            Reporter:
            Johannes Venzke
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: