Broken unit tests

View: New views
5 Messages — Rating Filter:   Alert me  

Broken unit tests

by Chuck Hagenbuch :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

Kolab_Storage is currently breaking the Horde unit tests:

Fatal error: Class 'Horde_Kolab_Test_Server' not found in  
/.../horde/horde/framework/Kolab_Storage/lib/Horde/Kolab/Test/Storage.php on  
line 38

I'm not sure why there are PHPUnit base test classes in the lib/ dirs,  
either... ?

-chuck

--
Horde developers mailing list - Join the hunt: http://horde.org/bounties/
Frequently Asked Questions: http://horde.org/faq/
To unsubscribe, mail: dev-unsubscribe@...

Re: Broken unit tests

by Gunnar Wrobel-2 :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

Quoting Chuck Hagenbuch <chuck@...>:

> Kolab_Storage is currently breaking the Horde unit tests:
>
> Fatal error: Class 'Horde_Kolab_Test_Server' not found in  
> /.../horde/horde/framework/Kolab_Storage/lib/Horde/Kolab/Test/Storage.php on  
> line 38
>

I'm working on both Kolab_Storage and Kolab_FreeBusy at the moment. I  
should have used a separate branch for that work as it turned out to  
be somewhat more complex than I thought it would be.

> I'm not sure why there are PHPUnit base test classes in the lib/  
> dirs, either... ?

I did not find a decent alternative for this so far. The Unit test  
suite in Kolab_Storage requires some basic functionality I currently  
provide within Kolab_Server. Kolab_FreeBusy requires the extended  
function set provided within Kolab_Storage. Would it be better to  
place these TestCase extensions into the Horde_Test package?

Cheers,

Gunnar

>
> -chuck
>
> --
> Horde developers mailing list - Join the hunt: http://horde.org/bounties/
> Frequently Asked Questions: http://horde.org/faq/
> To unsubscribe, mail: dev-unsubscribe@...
>







--
Horde developers mailing list - Join the hunt: http://horde.org/bounties/
Frequently Asked Questions: http://horde.org/faq/
To unsubscribe, mail: dev-unsubscribe@...

attachment0 (204 bytes) Download Attachment

Re: Broken unit tests

by Chuck Hagenbuch :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

Quoting Gunnar Wrobel <p@...>:

> Quoting Chuck Hagenbuch <chuck@...>:
>
>> Kolab_Storage is currently breaking the Horde unit tests:
>>
>> Fatal error: Class 'Horde_Kolab_Test_Server' not found in  
>> /.../horde/horde/framework/Kolab_Storage/lib/Horde/Kolab/Test/Storage.php  
>> on line 38
>>
>
> I'm working on both Kolab_Storage and Kolab_FreeBusy at the moment.  
> I should have used a separate branch for that work as it turned out  
> to be somewhat more complex than I thought it would be.

Not too late to branch it off now if you want. :)

>> I'm not sure why there are PHPUnit base test classes in the lib/  
>> dirs, either... ?
>
> I did not find a decent alternative for this so far. The Unit test  
> suite in Kolab_Storage requires some basic functionality I currently  
> provide within Kolab_Server. Kolab_FreeBusy requires the extended  
> function set provided within Kolab_Storage. Would it be better to  
> place these TestCase extensions into the Horde_Test package?

I don't think so... what about loading them directly out of the  
Kolab_Sserver test/ dir? I'm not sure what opinions others have here...

-chuck

--
Horde developers mailing list - Join the hunt: http://horde.org/bounties/
Frequently Asked Questions: http://horde.org/faq/
To unsubscribe, mail: dev-unsubscribe@...

Re: Broken unit tests

by Gunnar Wrobel-2 :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

Hi Chuck,

Quoting Chuck Hagenbuch <chuck@...>:

> Quoting Gunnar Wrobel <p@...>:
>
>> Quoting Chuck Hagenbuch <chuck@...>:
>>
>>> Kolab_Storage is currently breaking the Horde unit tests:
>>>
>>> Fatal error: Class 'Horde_Kolab_Test_Server' not found in  
>>> /.../horde/horde/framework/Kolab_Storage/lib/Horde/Kolab/Test/Storage.php  
>>> on line 38

Fixed that in git. There are still tests failing but running the  
AllTests.php from horde-cvs did not break anymore.

>>>
>>
>> I'm working on both Kolab_Storage and Kolab_FreeBusy at the moment.  
>> I should have used a separate branch for that work as it turned out  
>> to be somewhat more complex than I thought it would be.
>
> Not too late to branch it off now if you want. :)
>
>>> I'm not sure why there are PHPUnit base test classes in the lib/  
>>> dirs, either... ?
>>
>> I did not find a decent alternative for this so far. The Unit test  
>> suite in Kolab_Storage requires some basic functionality I  
>> currently provide within Kolab_Server. Kolab_FreeBusy requires the  
>> extended function set provided within Kolab_Storage. Would it be  
>> better to place these TestCase extensions into the Horde_Test  
>> package?
>
> I don't think so... what about loading them directly out of the  
> Kolab_Sserver test/ dir? I'm not sure what opinions others have  
> here...
The problem is to differentiate between the dev situation and the  
installed situation. I want to be able to run these tests on the Kolab  
Server, too. But I hope I solved that now.

Cheers,

Gunnar

>
> -chuck
>
> --
> Horde developers mailing list - Join the hunt: http://horde.org/bounties/
> Frequently Asked Questions: http://horde.org/faq/
> To unsubscribe, mail: dev-unsubscribe@...
>







--
Horde developers mailing list - Join the hunt: http://horde.org/bounties/
Frequently Asked Questions: http://horde.org/faq/
To unsubscribe, mail: dev-unsubscribe@...

attachment0 (204 bytes) Download Attachment

Re: Broken unit tests

by Chuck Hagenbuch :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

Quoting Gunnar Wrobel <p@...>:

> Fixed that in git. There are still tests failing but running the  
> AllTests.php from horde-cvs did not break anymore.

Cool, thanks. I'll work on some of the warnings/failures now that  
there aren't fatal errors.

-chuck

--
Horde developers mailing list - Join the hunt: http://horde.org/bounties/
Frequently Asked Questions: http://horde.org/faq/
To unsubscribe, mail: dev-unsubscribe@...