[ 
https://issues.apache.org/jira/browse/DERBY-3724?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12609343#action_12609343
 ] 

Kathey Marsden commented on DERBY-3724:
---------------------------------------

>For testVarchar(), When creating the table, a SQLException is thrown with the 
>prompt >that 32767 is a invalid length. 
I think you should create the table with the maximum size  allowed and only 
insert rows that will fit.

>For testLongVarchar(), When inserting a big row with the length of 33000, a 
>>SQLException is thrown with the prompt that 33000 is a invalid length.

Again only insert colums that will fit up to the max.

>I have another question: I have 10 test***() methods and 1 suite() method in 
>the class >of BigDataTest. When I run this test, I have seen 20 test cases 
>run. What's the problem?
Yes you are testing with both embedded and network client so this is expected.


I will take a closer look at the patch later today.  A couple things I noticed 
glancing at it briefly is that you should remove big.sql from the old harness 
and add your new test to lang._Suite.  

Also we should rename testJCCQueryBlock to testDefaultQueryBlock and change the 
comments to indicate client instead of JCC.





> Converting big.sql  to JUnit 
> -----------------------------
>
>                 Key: DERBY-3724
>                 URL: https://issues.apache.org/jira/browse/DERBY-3724
>             Project: Derby
>          Issue Type: Task
>          Components: Test
>         Environment: windows XP Professional SP2, 
>            Reporter: Junjie Peng
>            Assignee: Junjie Peng
>             Fix For: 10.4.1.3
>
>         Attachments: patch.txt
>
>   Original Estimate: 168h
>  Remaining Estimate: 168h
>
> It is a task of "Converting old tests to JUnit ". It request coverting  
> "derdy/java/testing/org/apache/derbyTesting/functionTests/tests/lang/big.sql" 
> into a junit. Also, it is a task of 2008 GSoc.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to