diff options
author | Grzegorz Bizon <grzesiek.bizon@gmail.com> | 2018-01-16 12:22:09 +0100 |
---|---|---|
committer | Grzegorz Bizon <grzesiek.bizon@gmail.com> | 2018-01-16 12:22:09 +0100 |
commit | c0e9a46dd590084e42dd43754e9b2ef93d31d6a6 (patch) | |
tree | e0ad6ac68b331a6f2d557c566bf208dce3dcd271 | |
parent | 7951acfd158d3f3b1358acb49925c913c50de3a6 (diff) | |
download | gitlab-ce-c0e9a46dd590084e42dd43754e9b2ef93d31d6a6.tar.gz |
Fix running manual action tooltip
-rw-r--r-- | lib/gitlab/ci/status/build/action.rb | 5 | ||||
-rw-r--r-- | spec/lib/gitlab/ci/status/build/action_spec.rb | 8 |
2 files changed, 8 insertions, 5 deletions
diff --git a/lib/gitlab/ci/status/build/action.rb b/lib/gitlab/ci/status/build/action.rb index 45fd0d4aa07..6c9125647ad 100644 --- a/lib/gitlab/ci/status/build/action.rb +++ b/lib/gitlab/ci/status/build/action.rb @@ -2,6 +2,9 @@ module Gitlab module Ci module Status module Build + ## + # Extended status for playable manual actions. + # class Action < Status::Extended def label if has_action? @@ -12,7 +15,7 @@ module Gitlab end def self.matches?(build, user) - build.action? + build.playable? end end end diff --git a/spec/lib/gitlab/ci/status/build/action_spec.rb b/spec/lib/gitlab/ci/status/build/action_spec.rb index 8c25f72804b..d612d29e3e0 100644 --- a/spec/lib/gitlab/ci/status/build/action_spec.rb +++ b/spec/lib/gitlab/ci/status/build/action_spec.rb @@ -37,16 +37,16 @@ describe Gitlab::Ci::Status::Build::Action do describe '.matches?' do subject { described_class.matches?(build, user) } - context 'when build is an action' do - let(:build) { create(:ci_build, :manual) } + context 'when build is playable action' do + let(:build) { create(:ci_build, :playable) } it 'is a correct match' do expect(subject).to be true end end - context 'when build is not manual' do - let(:build) { create(:ci_build) } + context 'when build is not playable action' do + let(:build) { create(:ci_build, :non_playable) } it 'does not match' do expect(subject).to be false |