diff options
author | Grzegorz Bizon <grzesiek.bizon@gmail.com> | 2017-07-04 15:50:03 +0200 |
---|---|---|
committer | Grzegorz Bizon <grzesiek.bizon@gmail.com> | 2017-07-04 15:50:03 +0200 |
commit | 22d8460b5d9926d7608d23aeb58e20d9035efa92 (patch) | |
tree | b0c54448933f7a737765cb1d1d467ba768e043c2 /app/models/concerns/has_status.rb | |
parent | a17c90b2a7331a7427813684b04095b55c4b3cc1 (diff) | |
download | gitlab-ce-22d8460b5d9926d7608d23aeb58e20d9035efa92.tar.gz |
Add some validations to persisted stage model
Diffstat (limited to 'app/models/concerns/has_status.rb')
-rw-r--r-- | app/models/concerns/has_status.rb | 12 |
1 files changed, 4 insertions, 8 deletions
diff --git a/app/models/concerns/has_status.rb b/app/models/concerns/has_status.rb index 235196cae13..8ea5a007f76 100644 --- a/app/models/concerns/has_status.rb +++ b/app/models/concerns/has_status.rb @@ -12,6 +12,10 @@ module HasStatus failed: 4, canceled: 5, skipped: 6, manual: 7 } class_methods do + def enumerated_status! + enum status: HasStatus::STATUSES_ENUM + end + def status_sql scope_relevant = respond_to?(:exclude_ignored) ? exclude_ignored : all scope_warnings = respond_to?(:failed_but_allowed) ? failed_but_allowed : none @@ -56,14 +60,6 @@ module HasStatus def all_state_names state_machines.values.flat_map(&:states).flat_map { |s| s.map(&:name) } end - - private - - def enumerated_status! - @status_strategy = :enumerator - - enum status: HasStatus::STATUSES_ENUM - end end included do |