Hi Michael and Marius,

Thanks for the feedback and suggestions! This is what I did:

On 10/13/10 15:37 , Michael Howitz wrote:
> Am 13.10.2010 um 13:50 schrieb Jan-Wijbrand Kolman:
>> Then there's no real need for a browser extra, as the browser
>> subpackage does not really have any code (only registrations). Or
>> maybe you meant something different?
>
> When someone wants to use the browser package he could use the
> "browser" extra to get all the packages needed for the ZCML
> registrations successfully load. Otherwise he has to find out himself
> where the ZCML directives are declared.

I created two extra_requires: a browser and a test_browser. The first 
lists the dependencies for the browser subpackage to work. The second 
for the corresponding tests to run.

On 10/13/10 14:42 , Marius Gedminas wrote:
> On Wed, Oct 13, 2010 at 01:50:39PM +0200, Jan-Wijbrand Kolman wrote:
>>> Ah, no it wouldn't help there, as the testrunner would find the
>>> tests in the browser subpackage and it would try to run them
>>> regardless.
> You could always conditionally disable them: in each test*.py file
> have
>
> def test_suite(): suite = unittest.TestSuite() if some condition:
>     suite.addTest(unittest.makeSuite(...))
>     suite.addTest(doctest.DocTestSuite(...))
>     suite.addTest(doctest.DocFileSuite(...)) return suite

I also made the tests.py in the browser subpackage only add tests to the 
test suite whenever the test_browser dependencies indeed are available. 
This fixes the situation where a test runner (in my case the test runner 
of the grok toolkit) finds these tests and would try to run them.

The CHANGES.txt contains instructions on how to enable the ZMI views for 
your project in case you need them.

I propose to release this as 1.5.0, which would be a major version 
release that would indicate: "Watch out, potentially backwards 
incompatible changes ahead!".

Right?

Thanks again for your feedback!

kind regards, jw

_______________________________________________
Zope-Dev maillist  -  Zope-Dev@zope.org
https://mail.zope.org/mailman/listinfo/zope-dev
**  No cross posts or HTML encoding!  **
(Related lists - 
 https://mail.zope.org/mailman/listinfo/zope-announce
 https://mail.zope.org/mailman/listinfo/zope )

Reply via email to