wants to merge 0 commits
Now that the /groups URL handles user permissions there was no need for /permissions URL. However, the template project_permissions.html has not been deleted
I think this section isn't being used after all:
def groups(self, **kw):
Because at the beginning of the class it has self.groups = GroupsController() and that is what is being used. So you can delete those 3 lines and also the project_permissions.html file then.
self.groups = GroupsController()
Regarding the test_subproject_permissions test that is removed: we don't have any existing test covering a subproject and the "/groups" URL. Do you want to try adding the test_subproject_permissions test back in but changing it to use /groups instead of /permissions URL? The self.app.get line will have to change, and the lines after it would have to change quite a lot. Or you could remove those lines, and the test would only test accessing the page and not making changes. (I think that would be ok since the page should work the same for regular projects and subprojects, and we already have tests for it on a regular project)
FYI this commit got merged to master, a bit unintentionally, as part of my mongo 3.4 fixes. Its okay though since the commit itself is good, but the followup changes I mentioned in the previous comment I think we still should do.
Log in to post a comment.