castor

Create test cases in cpa package to show bugs of ParseTreeWalker

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. Text File
    patch-C3116-20110627.txt
    27/Jun/11 5:51 PM
    5 kB
    Ralf Joachim
  2. Text File
    patch-tests-TreeWalker-110625.txt
    25/Jun/11 2:52 PM
    48 kB
    Johannes Venzke
  3. Text File
    patch-tests-TreeWalker-110627.txt
    27/Jun/11 5:28 AM
    10 kB
    Johannes Venzke

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
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
Permalink
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
Permalink
Johannes Venzke added a comment -

Recreated patches according to SVN changes.

Show
Johannes Venzke added a comment - Recreated patches according to SVN changes.
Hide
Permalink
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
Permalink
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
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: