raindrops RubyGem user+dev discussion/patches/pulls/bugs/help
 help / color / mirror / code / Atom feed
Search results ordered by [date|relevance]  view[summary|nested|Atom feed]
thread overview below | download mbox.gz: |
* [PATCH 1/4] tests: support running tests without rack||aggregate
  2023-12-29 17:50  6% [PATCH 0/4] lighter tests + rack 3 dependency fix Eric Wong
@ 2023-12-29 17:50  7% ` Eric Wong
  0 siblings, 0 replies; 2+ results
From: Eric Wong @ 2023-12-29 17:50 UTC (permalink / raw)
  To: raindrops-public; +Cc: Eric Wong

From: Eric Wong <e@80x24.org>

Diskspace and bandwidth are expensive, and we can make
rack+aggregate optional in tests, too.
---
 test/rack_unicorn.rb        | 3 +--
 test/test_last_data_recv.rb | 8 +++++++-
 test/test_watcher.rb        | 4 ++--
 3 files changed, 10 insertions(+), 5 deletions(-)

diff --git a/test/rack_unicorn.rb b/test/rack_unicorn.rb
index 779e8bf..0ecbd42 100644
--- a/test/rack_unicorn.rb
+++ b/test/rack_unicorn.rb
@@ -1,11 +1,10 @@
 # -*- encoding: binary -*-
 require "test/unit"
 require "raindrops"
-require "rack"
-require "rack/lobster"
 require "open-uri"
 begin
   require "unicorn"
+  require "rack"
   require "rack/lobster"
 rescue LoadError => e
   warn "W: #{e} skipping test since Rack or Unicorn was not found"
diff --git a/test/test_last_data_recv.rb b/test/test_last_data_recv.rb
index ef84e05..b1a5ac6 100644
--- a/test/test_last_data_recv.rb
+++ b/test/test_last_data_recv.rb
@@ -1,3 +1,9 @@
+begin
+  require 'aggregate'
+  have_aggregate = true
+rescue LoadError => e
+  warn "W: #{e} skipping #{__FILE__}"
+end
 require 'test/unit'
 require 'raindrops'
 require 'io/wait'
@@ -40,4 +46,4 @@ def test_accept_nonblock_one
     assert_equal 1, s.raindrops_aggregate.size
     assert_raise(IO::WaitReadable) { s.accept_nonblock }
   end
-end if RUBY_PLATFORM =~ /linux/
+end if RUBY_PLATFORM =~ /linux/ && have_aggregate
diff --git a/test/test_watcher.rb b/test/test_watcher.rb
index 28ac49b..e5d1fa2 100644
--- a/test/test_watcher.rb
+++ b/test/test_watcher.rb
@@ -1,9 +1,9 @@
 # -*- encoding: binary -*-
 require "test/unit"
-require "rack"
 require "raindrops"
 begin
   require 'aggregate'
+  require 'rack'
 rescue LoadError => e
   warn "W: #{e} skipping #{__FILE__}"
 end
@@ -183,4 +183,4 @@ def test_peaks
     assert_equal queued_before, headers["X-Last-Peak-At"], "should not change"
     assert_equal start, headers["X-First-Peak-At"]
   end
-end if RUBY_PLATFORM =~ /linux/ && defined?(Aggregate)
+end if RUBY_PLATFORM =~ /linux/ && defined?(Aggregate) && defined?(Rack)

^ permalink raw reply related	[relevance 7%]

* [PATCH 0/4] lighter tests + rack 3 dependency fix
@ 2023-12-29 17:50  6% Eric Wong
  2023-12-29 17:50  7% ` [PATCH 1/4] tests: support running tests without rack||aggregate Eric Wong
  0 siblings, 1 reply; 2+ results
From: Eric Wong @ 2023-12-29 17:50 UTC (permalink / raw)
  To: raindrops-public

Some things I noticed while preparing a release in the wake of
Ruby 3.3.

Eric Wong (4):
  tests: support running tests without rack||aggregate
  test/test_last_data_recv: don't require posix_mq
  extconf: disable optimization to speed up checks by ~3%
  gemspec: support rack 3.x

 ext/raindrops/extconf.rb    |  2 ++
 raindrops.gemspec           |  2 +-
 test/rack_unicorn.rb        |  3 +--
 test/test_last_data_recv.rb | 16 +++++++++++++++-
 test/test_watcher.rb        |  4 ++--
 5 files changed, 21 insertions(+), 6 deletions(-)

^ permalink raw reply	[relevance 6%]

Results 1-2 of 2 | reverse | options above
-- pct% links below jump to the message on this page, permalinks otherwise --
2023-12-29 17:50  6% [PATCH 0/4] lighter tests + rack 3 dependency fix Eric Wong
2023-12-29 17:50  7% ` [PATCH 1/4] tests: support running tests without rack||aggregate Eric Wong

Code repositories for project(s) associated with this public inbox

	https://yhbt.net/raindrops.git/

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).