summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorKrasimir Angelov <kangelov@gitlab.com>2019-07-02 17:32:25 +0300
committerKrasimir Angelov <kangelov@gitlab.com>2019-07-02 17:32:25 +0300
commit107344ebed33936e802d01bc6d22e480935d1e33 (patch)
tree1f647a04df2162e2217f434c57e97c71c5d957d5
parente8b416a65fc29992ccc3f81051a8d7e4aaee4166 (diff)
downloadgitlab-ce-50228-deploy-tokens-custom-username.tar.gz
Improve deploy token related specs readability50228-deploy-tokens-custom-username
-rw-r--r--spec/controllers/projects/settings/repository_controller_spec.rb9
-rw-r--r--spec/services/deploy_tokens/create_service_spec.rb8
2 files changed, 16 insertions, 1 deletions
diff --git a/spec/controllers/projects/settings/repository_controller_spec.rb b/spec/controllers/projects/settings/repository_controller_spec.rb
index 7631eb1400f..7f67f67e775 100644
--- a/spec/controllers/projects/settings/repository_controller_spec.rb
+++ b/spec/controllers/projects/settings/repository_controller_spec.rb
@@ -34,7 +34,14 @@ describe Projects::Settings::RepositoryController do
end
describe 'POST create_deploy_token' do
- let(:deploy_token_params) { { name: "deployer_token", expires_at: 1.month.from_now.to_date.to_s, username: 'deployer', read_repository: "1" } }
+ let(:deploy_token_params) do
+ {
+ name: 'deployer_token',
+ expires_at: 1.month.from_now.to_date.to_s,
+ username: 'deployer',
+ read_repository: '1'
+ }
+ end
subject(:create_deploy_token) { post :create_deploy_token, params: { namespace_id: project.namespace, project_id: project, deploy_token: deploy_token_params } }
diff --git a/spec/services/deploy_tokens/create_service_spec.rb b/spec/services/deploy_tokens/create_service_spec.rb
index 4973494fe46..fbb66fe4cb7 100644
--- a/spec/services/deploy_tokens/create_service_spec.rb
+++ b/spec/services/deploy_tokens/create_service_spec.rb
@@ -40,6 +40,14 @@ describe DeployTokens::CreateService do
end
end
+ context 'when username is provided' do
+ let(:deploy_token_params) { attributes_for(:deploy_token, username: 'deployer') }
+
+ it 'keeps the provided username' do
+ expect(subject.read_attribute(:username)).to eq('deployer')
+ end
+ end
+
context 'when the deploy token is invalid' do
let(:deploy_token_params) { attributes_for(:deploy_token, read_repository: false, read_registry: false) }