On Wed, May 4, 2016 at 6:15 AM, Mark Lavin <markdla...@gmail.com> wrote:
> I had assumed this was still a work in progress because there are missing > tests and some documentation. The build is passing but the unittest > coverage for the new modules seems low or at least not up to the standards > I expect for Django contributions. The same for the daphne and asgiref > packages which are new requirements. In previous discussions there were > talks about performance benchmarks to track potential regressions before > this would be accepted similar to the template-based widget rendering > change. I don't see any references here or in this work. What is the status > of those? > > - The documentation on Channels is considerably more than I would have accepted for a patch. What do you think is missing? - The channels branch is likely missing some tests around auth and session in particular, but a lot of the "main" stuff has tests; what would you need before accepting it? - asgiref quite literally has more tests than actual code we're using, so I don't see a problem there - Daphne is severely lacking in tests for WebSocket encoding/decoding support as writing a test harness for it is ridiculously hard and I was hoping to get some help on that now we finally have the MOSS money around. - There are no performance regression tests as by default the codepath in Django is the same as before (apart from runserver, which in my basic tests actually sped up); as long as you deploy using WSGI, you never even touch Channels code (which wasn't the original plan, but now is for sanity reasons). I would like some leeway on the external dependencies being fully tested considering that they are not beholden to Django's release cycle, but maybe because we're declaring them as core dependencies (I patched in install_requires into setup.py for them) we should be very strict? If this patch is going to be denied based on Daphne not having sufficient test coverage then I suspect we'll miss the deadline and have to foist this on the LTS instead, which I really don't want. Andrew -- You received this message because you are subscribed to the Google Groups "Django developers (Contributions to Django itself)" group. To unsubscribe from this group and stop receiving emails from it, send an email to django-developers+unsubscr...@googlegroups.com. To post to this group, send email to django-developers@googlegroups.com. Visit this group at https://groups.google.com/group/django-developers. To view this discussion on the web visit https://groups.google.com/d/msgid/django-developers/CAFwN1uqfj7720_qHbO2sggF076w4dCOzaRD%2B7qA%3DwrNG%2BcLhFA%40mail.gmail.com. For more options, visit https://groups.google.com/d/optout.
