gitlabhq/spec
Felix Gilcher ce6436b98a Don't crash when removing a user that's not project member
The attempt to revoke project access for a user that was not member of the
project results in a 500 Internal Server error where it actually should
result in a 200 OK since after the operation, the user is not member of
the project. This turns the operation into an idempotent call that can
be repeated with no ill effects.

Updated the spec and changed the code accordingly. However, the result differs
slightly, as we can't return the users project access level if the user was not
member. I'm not aware if anybody relies on the result of this call.

Fixes #2832
2013-02-01 13:53:35 +00:00
..
contexts Improve CreateContext call. Fixed test 2013-01-18 20:21:13 +02:00
controllers remove unused autogenerated files 2013-01-24 22:31:48 +02:00
factories simple refactoring 2013-01-24 22:31:48 +02:00
helpers remove unused autogenerated files 2013-01-24 22:31:48 +02:00
lib fix tests 2013-01-28 17:53:01 +02:00
mailers Add optional signup. 2013-01-18 11:57:31 +01:00
models fix tests 2013-01-28 08:59:34 +02:00
observers fix key observer tests 2013-01-28 23:03:38 +02:00
requests Don't crash when removing a user that's not project member 2013-02-01 13:53:35 +00:00
routing Fixed dashboard show specs 2013-01-27 13:20:23 +02:00
support Async perform for add/remove team members 2013-01-28 17:39:02 +02:00
tasks/gitlab Fix routing issues when navigating over tree, commits etc 2012-12-25 06:14:05 +03:00
workers sidekiq with green tests 2013-01-09 09:14:05 +03:00
factories.rb user factory username over sequence 2013-01-28 17:46:24 +02:00
factories_spec.rb Factories spec refactoring after #1616 2012-10-03 13:33:31 +03:00
seed_project.tar.gz Rspec test repo replaced.\nMerge Requests improved 2012-03-13 23:54:49 +02:00
spec_helper.rb sidekiq with green tests 2013-01-09 09:14:05 +03:00