github-mirrors/logstash
1
0
Fork 0
mirror of https://github.com/elastic/logstash.git synced 2025-06-27 17:08:55 -04:00
Code Issues Projects Releases Packages Wiki Activity

Fix QA failure introduced by Health API changes and update rspec dependency of the QA package. (#16521) (#16522)

Browse source 
* Update rspec dependency of the QA package.

* Update qa/Gemfile

Align on rspec 3.13.x

Co-authored-by: Ry Biesemeyer <yaauie@users.noreply.github.com>

* Fix the QA test failure caused after reflecting Health Report status to the Node stats.

---------

Co-authored-by: Ry Biesemeyer <yaauie@users.noreply.github.com>
(cherry picked from commit 1e5105fcd8)

Co-authored-by: Mashhur <99575341+mashhurs@users.noreply.github.com>
This commit is contained in:
github-actions[bot] 2024-10-09 15:28:02 -07:00 • committed by GitHub
parent a4eddb8a2a
commit dc24f02972
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 3 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

2
qa/Gemfile
View file

@ -1,5 +1,5 @@
source "https://rubygems.org"
gem "rspec", "~> 3.1.0"
gem "rspec", "~> 3.13"
gem "rake"
gem "stud"
gem "pry", :group => :test

3
qa/docker/spec/spec_helper.rb
View file

@ -38,7 +38,8 @@ end
def wait_for_logstash(container)
Stud.try(40.times, [NoMethodError, Docker::Error::ConflictError, RSpec::Expectations::ExpectationNotMetError, TypeError]) do
expect(logstash_available?(container)).to be true
expect(get_logstash_status(container)).to eql 'green'
# unknown or red status may be also meaningful while testing
expect(%w(unknown green yellow red).include?(get_logstash_status(container))).to be true
end
end