Skip to content
This repository has been archived by the owner on Jan 25, 2022. It is now read-only.

Commit

Permalink
Revert "Increase timeout when waiting for log messages"
Browse files Browse the repository at this point in the history
This commit caused the tests of our produciton integration with
AppDirect to fail. I'm reverting it until I find a fix.

This reverts commit c25c9b5.
  • Loading branch information
Matt Royal committed Jan 1, 2014
1 parent c25c9b5 commit 394b606
Show file tree
Hide file tree
Showing 3 changed files with 10 additions and 12 deletions.
2 changes: 1 addition & 1 deletion Gemfile
Expand Up @@ -6,7 +6,7 @@ gem 'ci_reporter'
gem 'cfoundry', '~> 4.0.4.rc2'
gem 'cf-uaa-lib'
gem 'httparty'
gem 'blue-shell', '~> 0.3.0'
gem 'blue-shell'
gem 'parallel_tests'

group :monitoring do
Expand Down
4 changes: 2 additions & 2 deletions Gemfile.lock
Expand Up @@ -7,7 +7,7 @@ GEM
activesupport (3.2.14)
i18n (~> 0.6, >= 0.6.4)
multi_json (~> 1.0)
blue-shell (0.3.0)
blue-shell (0.2.2)
rspec
builder (3.0.4)
cf-uaa-lib (1.3.10)
Expand Down Expand Up @@ -49,7 +49,7 @@ PLATFORMS
ruby

DEPENDENCIES
blue-shell (~> 0.3.0)
blue-shell
cf-uaa-lib
cfoundry (~> 4.0.4.rc2)
ci_reporter
Expand Down
16 changes: 7 additions & 9 deletions spec/loggregator_spec.rb
Expand Up @@ -52,15 +52,13 @@ def check_loggregator_works
puts "starting #{__method__} (#{Time.now})"

BlueShell::Runner.run("gcf logs #{app_name}") do |runner|
runner.with_timeout 60 do
runner.should say "Connected, tailing logs for app #{app_name}"

# Hit twice to see that both router and app messages come through; order is not guaranteed
page_content
runner.should say "[RTR]"
page_content
runner.should say "[App/0]"
end
runner.should say "Connected, tailing logs for app #{app_name}"

# Hit twice to see that both router and app messages come through; order is not guaranteed
page_content
runner.should say "[RTR]"
page_content
runner.should say "[App/0]"
end
end

Expand Down

0 comments on commit 394b606

Please sign in to comment.