Skip to content
Snippets Groups Projects
Verified Commit 88a0c984 authored by Yorick Peterse's avatar Yorick Peterse
Browse files

Fixed downtime check label colouring

The colours were incorrect: offline was green and online was red,
instead of the opposite.
parent d1da2e81
No related branches found
No related tags found
No related merge requests found
Loading
@@ -3,8 +3,8 @@ module Gitlab
Loading
@@ -3,8 +3,8 @@ module Gitlab
class Message class Message
attr_reader :path, :offline, :reason attr_reader :path, :offline, :reason
   
OFFLINE = "\e[32moffline\e[0m" OFFLINE = "\e[31moffline\e[0m"
ONLINE = "\e[31monline\e[0m" ONLINE = "\e[32monline\e[0m"
   
# path - The file path of the migration. # path - The file path of the migration.
# offline - When set to `true` the migration will require downtime. # offline - When set to `true` the migration will require downtime.
Loading
Loading
Loading
@@ -5,13 +5,13 @@ describe Gitlab::DowntimeCheck::Message do
Loading
@@ -5,13 +5,13 @@ describe Gitlab::DowntimeCheck::Message do
it 'returns an ANSI formatted String for an offline migration' do it 'returns an ANSI formatted String for an offline migration' do
message = described_class.new('foo.rb', true, 'hello') message = described_class.new('foo.rb', true, 'hello')
   
expect(message.to_s).to eq("[\e[32moffline\e[0m]: foo.rb: hello") expect(message.to_s).to eq("[\e[31moffline\e[0m]: foo.rb: hello")
end end
   
it 'returns an ANSI formatted String for an online migration' do it 'returns an ANSI formatted String for an online migration' do
message = described_class.new('foo.rb') message = described_class.new('foo.rb')
   
expect(message.to_s).to eq("[\e[31monline\e[0m]: foo.rb") expect(message.to_s).to eq("[\e[32monline\e[0m]: foo.rb")
end end
end end
end end
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment