[ckan-dev] Failing test

David Read david.read at okfn.org
Mon Aug 8 17:58:54 UTC 2011


Adria,

Seb wrote this comment, and he is now on another project, so I think
we have to work it out ourselves...
https://bitbucket.org/okfn/ckan/changeset/e5a35e329117

This was a rather difficult and hectic bit of work, so lots things
were tried and changed. He commented out the assert
(cset:a1751e7f9e0c) and then enabled it again (cset:e5a35e329117), I
assume when he got something fixed and he worked out what it was for,
which is: checking that if you don't have permissions to put a package
in a particular group then it won't advertise this on the package edit
form.

I get the impression it was working well before and after the time
that Seb did the upgrade to SQLAlchemy 5/6, and I'm not sure
_get_user_editable_groups logic was used for David Raznick's new form
system. Perhaps he can say more about this.

David

2011/8/8 Adrià Mercader <amercadero at gmail.com>:
> Hi,
> I pushed the latest changes on authorization with all tests passing
> except for this one:
>
> ckan.tests.functional.test_package.TestEdit.test_edit_700_groups_unauthorized
>
> There is a prior comment on the code that states that the assertion
> does indeed fail:
>
> https://bitbucket.org/okfn/ckan/src/ed847ce8ce2f/ckan/tests/functional/test_package.py#cl-1020
>
> Can I ask the comment's author his thoughts about this?
>
> Thanks!
>
> Adrià
>
> _______________________________________________
> ckan-dev mailing list
> ckan-dev at lists.okfn.org
> http://lists.okfn.org/mailman/listinfo/ckan-dev
>




More information about the ckan-dev mailing list