On Fri, Apr 20, 2012 at 10:30 AM, Afkham Azeez <[email protected]> wrote:

>
>
> On Fri, Apr 20, 2012 at 8:50 AM, Vijitha Kumara <[email protected]> wrote:
>
>> Yes, perhaps those were written with the Junit in mind :)
>>
>
> Exactly, in JUnit it is the other way about. I too realized this when I
> read their API docs and then corrected some of my mistakes. I guess there
> may be tons of incorrectly written assert statements :(
>

Yes Azeez, you're quite right.
I also checked with few other product integration tests as well. And there
are loads of them in testng migrated integration tests. But they didn't get
noticed until an assert fails and you find hard to understand the assertion
failure message. But we have to fix them all.

I have started refactoring them for GReg and there're quite a many to fix
:(.


>
>
>>
>>
>> On Thu, Apr 19, 2012 at 8:18 PM, Afkham Azeez <[email protected]> wrote:
>>
>>>
>>>
>>> On Thu, Apr 19, 2012 at 1:47 PM, Vijitha Kumara <[email protected]>wrote:
>>>
>>>>
>>>> This seems bit strange, the real test case expects the length to be 2
>>>> but the test failure complains it expects 3...
>>>
>>>
>>> The way the parameters have been passed to that test is incorrect. It
>>> should be;
>>>
>>> *assertEquals(schemas.length, 2);*
>>>
>>>
>>>>
>>>>
>>>> -----------------------------------------------------------------------------------------------------------------
>>>> build 18-Apr-2012 17:01:13 Failed tests:
>>>> build 18-Apr-2012 17:01:13  
>>>> testAddSchema(org.wso2.carbon.registry.governance.api.test.SchemaTestCase):
>>>> expected:<3> but was:<2>
>>>> build 18-Apr-2012 17:01:13
>>>> build 18-Apr-2012 17:01:13 Tests run: 233, Failures: 1, Errors: 0,
>>>> Skipped: 0
>>>>
>>>> -----------------------------------------------------------------------------------------------------------------
>>>>
>>>> Test case:
>>>>
>>>> -----------------------------------------------------------------------------------------------------------------
>>>> Schema[] schemas = schemaManager.findSchemas(new SchemaFilter() {
>>>>             public boolean matches(Schema schema) throws
>>>> GovernanceException {
>>>>                 if (schema.getAttribute("version").equals("0.01")) {
>>>>                     return true;
>>>>                 }
>>>>                 return false;
>>>>             }
>>>>         });
>>>>         *assertEquals(2, schemas.length);*
>>>>         assertEquals(newSchema.getId(), schemas[0].getId());
>>>>
>>>> -----------------------------------------------------------------------------------------------------------------
>>>>
>>>> Regards,
>>>>
>>>> On Thu, Apr 19, 2012 at 10:56 AM, Subash Chaturanga <[email protected]>wrote:
>>>>
>>>>>
>>>>>
>>>>> On Thu, Apr 19, 2012 at 10:41 AM, Subash Chaturanga 
>>>>> <[email protected]>wrote:
>>>>>
>>>>>>
>>>>>>
>>>>>> On Thu, Apr 19, 2012 at 10:19 AM, Ishan Jayawardena 
>>>>>> <[email protected]>wrote:
>>>>>>
>>>>>>> Hi, please have a look at this error. Attached Log.
>>>>>>>
>>>>>>
>>>>>> Hi,  already saw this on bamboo logs and working on it, this is not
>>>>>> only due to a testng failure. But product-governance-services also fails.
>>>>>>
>>>>>> - governance stratos fails.
>>>>>>
>>>>> There was patch for this fix, and not yet commited. I think Dimuthu is
>>>>> looking in to this to commit the patch.
>>>>>
>>>>>
>>>>>> - 1 test failure and I am locally trying to reproduce this.(This was
>>>>>> a randomly failing test and we intentionally add this(+ two other tests) 
>>>>>> to
>>>>>> see the behavior of this, because we couldn't reproduce this locally in
>>>>>> anyway).
>>>>>>
>>>>>> Excluded SchemaTest until get fixed.
>>>>>
>>>>>>  Thanks
>>>>>>
>>>>>>
>>>>>>> Thanks,
>>>>>>> Ishan.
>>>>>>> --
>>>>>>> Ishan Jayawardena
>>>>>>> Software Engineer
>>>>>>> Integration Technologies Team
>>>>>>> WSO2, Inc. http://wso2.com/
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>> _______________________________________________
>>>>>>> Dev mailing list
>>>>>>> [email protected]
>>>>>>> http://wso2.org/cgi-bin/mailman/listinfo/dev
>>>>>>>
>>>>>>>
>>>>>>
>>>>>>
>>>>>> --
>>>>>>
>>>>>> Subash Chaturanga
>>>>>> Software Engineer
>>>>>> WSO2 Inc. http://wso2.com
>>>>>>
>>>>>> email - [email protected]
>>>>>> phone - 077 2225922
>>>>>>
>>>>>>
>>>>>
>>>>>
>>>>> --
>>>>>
>>>>> Subash Chaturanga
>>>>> Software Engineer
>>>>> WSO2 Inc. http://wso2.com
>>>>>
>>>>> email - [email protected]
>>>>> phone - 077 2225922
>>>>>
>>>>>
>>>>> _______________________________________________
>>>>> Dev mailing list
>>>>> [email protected]
>>>>> http://wso2.org/cgi-bin/mailman/listinfo/dev
>>>>>
>>>>>
>>>>
>>>>
>>>> --
>>>> Vijitha Kumara
>>>> Senior Software Engineer; WSO2, Inc.;  http://wso2.com/
>>>> email: [email protected]
>>>>
>>>> Lean . Enterprise . Middleware
>>>>
>>>>
>>>> _______________________________________________
>>>> Dev mailing list
>>>> [email protected]
>>>> http://wso2.org/cgi-bin/mailman/listinfo/dev
>>>>
>>>>
>>>
>>>
>>> --
>>> *Afkham Azeez*
>>> Director of Architecture; WSO2, Inc.; http://wso2.com
>>> Member; Apache Software Foundation; http://www.apache.org/
>>> * <http://www.apache.org/>**
>>> email: **[email protected]* <[email protected]>* cell: +94 77 3320919
>>> blog: **http://blog.afkham.org* <http://blog.afkham.org>*
>>> twitter: **http://twitter.com/afkham_azeez*<http://twitter.com/afkham_azeez>
>>> *
>>> linked-in: **http://lk.linkedin.com/in/afkhamazeez*
>>>
>>> *
>>> *
>>> *Lean . Enterprise . Middleware*
>>>
>>>
>>
>>
>> --
>> Vijitha Kumara
>> Senior Software Engineer; WSO2, Inc.;  http://wso2.com/
>> email: [email protected]
>>
>> Lean . Enterprise . Middleware
>>
>>
>
>
> --
> *Afkham Azeez*
> Director of Architecture; WSO2, Inc.; http://wso2.com
> Member; Apache Software Foundation; http://www.apache.org/
> * <http://www.apache.org/>**
> email: **[email protected]* <[email protected]>* cell: +94 77 3320919
> blog: **http://blog.afkham.org* <http://blog.afkham.org>*
> twitter: **http://twitter.com/afkham_azeez*<http://twitter.com/afkham_azeez>
> *
> linked-in: **http://lk.linkedin.com/in/afkhamazeez*
> *
> *
> *Lean . Enterprise . Middleware*
>
>
> _______________________________________________
> Dev mailing list
> [email protected]
> http://wso2.org/cgi-bin/mailman/listinfo/dev
>
>


-- 

Subash Chaturanga
Software Engineer
WSO2 Inc. http://wso2.com

email - [email protected]
phone - 077 2225922
_______________________________________________
Dev mailing list
[email protected]
http://wso2.org/cgi-bin/mailman/listinfo/dev

Reply via email to