Radhika Chippada [Wed, 4 Feb 2015 20:54:14 +0000 (15:54 -0500)]
Diagnostics testing: The test fails because there are 2 Run buttons in the Inputs tab right after choosing inputs. Click on Components tab before clicking on the Run button.
Radhika Chippada [Wed, 4 Feb 2015 14:22:05 +0000 (09:22 -0500)]
2659: Do not turn on anonymous config by default in test mode; instead make the tests set it as needed.
Also convert some of the simple integration tests into controller tests.
Tom Clegg [Tue, 3 Feb 2015 21:33:18 +0000 (16:33 -0500)]
2659: Check anon config at runtime instead of boot time in skip_before_filter. This makes it possible for test cases to toggle behavior by changing configuration on the fly.
Brett Smith [Tue, 3 Feb 2015 20:40:10 +0000 (15:40 -0500)]
5131: run-command consumes escape backslashes.
run-command has long recognized backslashes to inhibit
substitution, but it was leaving those backslashes in the string,
which could lead to incorrect commands. This commit makes sure
they're removed after they're parsed.
Radhika Chippada [Tue, 3 Feb 2015 15:20:45 +0000 (10:20 -0500)]
2659: Finally understood the mystery behind the "anonymous tests are failing when run with all other tests".
There is a permission link fixture for anonymous group. I needed to create read permission link for anonymous user.
(Tom thinks) the groups permission cache may be getting out of sync somehow.
Radhika Chippada [Tue, 3 Feb 2015 13:33:29 +0000 (08:33 -0500)]
2659: Expect redirect to login page when show method is executed in jobs controller etc when no anonymous configuration is set.
Something is overriding the configured value from application.default.yml when the test is executed along with all other tests!
Radhika Chippada [Mon, 2 Feb 2015 23:23:16 +0000 (18:23 -0500)]
2659: Do not require login for project tab_counts method as well.
Remove anonymous login tests for /users and /groups pages since we are no longer doing session establishment for anonymous user login.
These are like any other un-shared pages and show login page to user.
Radhika Chippada [Mon, 2 Feb 2015 22:15:55 +0000 (17:15 -0500)]
2659: load @prov_svg outside of the 'if current_user' block so that an anonymous user can see the provenance graph when accessing publicly accessible collection.
Radhika Chippada [Mon, 2 Feb 2015 01:22:42 +0000 (20:22 -0500)]
2659: (1) Update link_to_if_arvados_object method to display an object name as link for User type only when the user is active.
Otherwise, an anonymous user will always see 404 when such a link is clicked.
(2) In the unlikely event of an anonymous user traversing to /users or /groups pages, there will be an 'Add a new xxx' link,
which is undesirable (even though they do eventually fail when accessed). Hence, hide them for inactive users.
(3) Add test assertions for both the updates.
Radhika Chippada [Sun, 1 Feb 2015 20:09:23 +0000 (15:09 -0500)]
2659: It appears that a job's log file is stored in a user's Home project, instead of the project in which the job originated from.
Due to this, when an anonymous user accesses the job from the publicly accessible project, the Log tab is failing.
Hence, display job -> Log tab for active users only.
Radhika Chippada [Sun, 1 Feb 2015 16:30:40 +0000 (11:30 -0500)]
2659: Collection show_files shows selection dropdown (which has only create new collection from selected files option) when user is active.
Also, dashboard displays the 'Run a pipeline' selector for active users. Upated anonymous user test to verify dashboard access.
Brett Smith [Sun, 1 Feb 2015 16:22:42 +0000 (11:22 -0500)]
5037: Fix tearDown error handling in PySDK Websockets test.
I accidentally broke this in 51f9e6cb. That version would crash if
the test failed to assign self.ws. Refactor to be more robust in that
case. No issue #.
Brett Smith [Fri, 30 Jan 2015 20:09:34 +0000 (15:09 -0500)]
5037: Fix race condition in PySDK Websockets test.
This might just be bad luck, but for some reason when testing Tom's
changes I ran into a situation where this test failed because it
started processing the Websocket event about the new human before it
assigned self.h, so the assertion failed. Fix the race condition by
asserting against a thread-aware Queue.
Brett Smith [Tue, 27 Jan 2015 21:22:35 +0000 (16:22 -0500)]
5074: Improve discovery document caching in Ruby SDK.
* Report a warning if there's an error reading or parsing a cached
discovery document.
* Cache discovery documents to a tempfile and rename, to reduce the
chances that we'll write corrupted caches due to I/O errors.
* Make this extension a separate module that can be loaded by users
like the CLI SDK.
* Tighten exception handling throughout.
Brett Smith [Tue, 27 Jan 2015 23:09:20 +0000 (18:09 -0500)]
4878: Refactor Workbench "Re-run job" button for more code reuse.
* Avoid adding new routes: add the modal directly to the job show
page, and submit the new work with the existing job creation route.
* Fix id references; e.g., for labels.