Arvados

Reviewing 95fd4dd

• In the view, you can just call @object.editable? to check if the user has write permission. That covers all the corner cases and will save you some typing.
• In the new test, the name has a typo "avaialble."
• In the test, please go directly to the project page: visit page_with_token("active", "/projects/#{my_project['uuid']"}). In general, being more conscientious about what pages we visit in integration tests will help keep test runtimes proportional what we're testing.
• The two assert_no_selector lines are redundant. Just checking li is fine; it will find li.disabled if that exists. Given this, it might make sense to have all the tests look for li.

Thanks.

Applied in changeset arvados|commit:46aee7d3bec5676e054bddc38633b2775f3a0fd6.