diff options
author | Timothy Andrew <mail@timothyandrew.net> | 2016-11-24 13:07:22 +0530 |
---|---|---|
committer | Timothy Andrew <mail@timothyandrew.net> | 2016-12-16 16:29:31 +0530 |
commit | 4d6da770de94f4bf140507cdf43461b67269ce28 (patch) | |
tree | d637ccdf6af0475af83b01e9f8371c5f06f6f880 /spec/services/access_token_validation_service_spec.rb | |
parent | ac9835c602f1c9b5a35ef40df079faf1d4b91f7b (diff) | |
download | gitlab-ce-4d6da770de94f4bf140507cdf43461b67269ce28.tar.gz |
Implement minor changes from @dbalexandre's review.
- Mainly whitespace changes.
- Require the migration adding the `scope` column to the
`personal_access_tokens` table to have downtime, since API calls will
fail if the new code is in place, but the migration hasn't run.
- Minor refactoring - load `@scopes` in a `before_action`, since we're
doing it in three different places.
Diffstat (limited to 'spec/services/access_token_validation_service_spec.rb')
-rw-r--r-- | spec/services/access_token_validation_service_spec.rb | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/spec/services/access_token_validation_service_spec.rb b/spec/services/access_token_validation_service_spec.rb index 8808934fa24..332e745aa36 100644 --- a/spec/services/access_token_validation_service_spec.rb +++ b/spec/services/access_token_validation_service_spec.rb @@ -1,7 +1,6 @@ require 'spec_helper' describe AccessTokenValidationService, services: true do - describe ".sufficient_scope?" do it "returns true if the required scope is present in the token's scopes" do token = double("token", scopes: [:api, :read_user]) |