zold 0.31.7 → 0.31.9

data/test/commands/test_taxes.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-# Copyright (c) 2018-2023 Zerocracy
+# Copyright (c) 2018-2024 Zerocracy
 #
 # Permission is hereby granted, free of charge, to any person obtaining a copy
 # of this software and associated documentation files (the 'Software'), to deal
@@ -34,11 +34,11 @@ require_relative '../../lib/zold/commands/taxes'
 
 # TAXES test.
 # Author:: Yegor Bugayenko (yegor256@gmail.com)
-# Copyright:: Copyright (c) 2018 Yegor Bugayenko
+# Copyright:: Copyright (c) 2018-2024 Zerocracy
 # License:: MIT
 class TestTaxes < Zold::Test
   def test_pays_taxes
-    FakeHome.new(log: test_log).run do |home|
+    FakeHome.new(log: fake_log).run do |home|
       wallets = home.wallets
       wallet = home.create_wallet
       fund = Zold::Amount.new(zld: 19.99)
@@ -65,7 +65,7 @@ class TestTaxes < Zold::Test
       before = wallet.balance
       tax = Zold::Tax.new(wallet, ignore_score_weakness: true)
       debt = tax.debt
-      Zold::Taxes.new(wallets: wallets, remotes: remotes, log: test_log).run(
+      Zold::Taxes.new(wallets: wallets, remotes: remotes, log: fake_log).run(
         ['taxes', '--private-key=fixtures/id_rsa', '--ignore-score-weakness', 'pay', wallet.id.to_s]
       )
       wallet.flush

data/test/fake_home.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-# Copyright (c) 2018-2023 Zerocracy
+# Copyright (c) 2018-2024 Zerocracy
 #
 # Permission is hereby granted, free of charge, to any person obtaining a copy
 # of this software and associated documentation files (the 'Software'), to deal
@@ -33,7 +33,7 @@ require_relative '../lib/zold/remotes'
 
 # Fake home dir.
 # Author:: Yegor Bugayenko (yegor256@gmail.com)
-# Copyright:: Copyright (c) 2018 Yegor Bugayenko
+# Copyright:: Copyright (c) 2018-2024 Zerocracy
 # License:: MIT
 class FakeHome
   attr_reader :dir

data/test/node/fake_entrance.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-# Copyright (c) 2018-2023 Zerocracy
+# Copyright (c) 2018-2024 Zerocracy
 #
 # Permission is hereby granted, free of charge, to any person obtaining a copy
 # of this software and associated documentation files (the 'Software'), to deal
@@ -22,7 +22,7 @@
 
 # Fake entrance.
 # Author:: Yegor Bugayenko (yegor256@gmail.com)
-# Copyright:: Copyright (c) 2018 Yegor Bugayenko
+# Copyright:: Copyright (c) 2018-2024 Zerocracy
 # License:: MIT
 class FakeEntrance
   def initialize

data/test/node/fake_node.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-# Copyright (c) 2018-2023 Zerocracy
+# Copyright (c) 2018-2024 Zerocracy
 #
 # Permission is hereby granted, free of charge, to any person obtaining a copy
 # of this software and associated documentation files (the 'Software'), to deal
@@ -31,58 +31,75 @@ require_relative '../../lib/zold/node/front'
 
 # Fake node.
 # Author:: Yegor Bugayenko (yegor256@gmail.com)
-# Copyright:: Copyright (c) 2018 Yegor Bugayenko
+# Copyright:: Copyright (c) 2018-2024 Zerocracy
 # License:: MIT
 class FakeNode
   def initialize(log: Zold::Log::NULL)
     @log = log
   end
 
+  # This is a pretty weird situation: we have to acquire a single port
+  # number an reuse it for all tests. If we use different port numbers,
+  # acquiring them for each instance of FakeNode, we get port collisions,
+  # sometimes. Maybe this can be fixed sometimes.
+  # rubocop:disable Style/ClassVars
+  @@port = RandomPort::Pool::SINGLETON.acquire
+  # rubocop:enable Style/ClassVars
+
   def run(args = ['--standalone', '--no-metronome'])
     WebMock.allow_net_connect!
     FakeHome.new(log: @log).run do |home|
-      RandomPort::Pool::SINGLETON.acquire do |port|
-        node = Thread.new do
-          Thread.current.name = 'fake_node'
-          Thread.current.abort_on_exception = true
-          Zold::VerboseThread.new(@log).run do
-            require_relative '../../lib/zold/commands/node'
-            Zold::Node.new(wallets: home.wallets, remotes: home.remotes, copies: home.copies.root, log: @log).run(
-              [
-                '--home', home.dir,
-                '--network=test',
-                '--port', port.to_s,
-                '--host=localhost',
-                '--bind-port', port.to_s,
-                '--threads=1',
-                '--dump-errors',
-                '--strength=2',
-                '--halt-code=test',
-                '--routine-immediately',
-                '--invoice=NOPREFIX@ffffffffffffffff'
-              ] + args
-            )
-          end
+      node = Thread.new do
+        Thread.current.name = 'fake_node'
+        Thread.current.abort_on_exception = false
+        Zold::VerboseThread.new(@log).run do
+          require_relative '../../lib/zold/commands/node'
+          Zold::Node.new(wallets: home.wallets, remotes: home.remotes, copies: home.copies.root, log: @log).run(
+            [
+              '--home', home.dir,
+              '--network=test',
+              '--port', @@port.to_s,
+              '--host=localhost',
+              '--bind-port', @@port.to_s,
+              '--threads=1',
+              '--dump-errors',
+              '--strength=2',
+              '--halt-code=test',
+              '--routine-immediately',
+              '--invoice=NOPREFIX@ffffffffffffffff'
+            ] + args
+          )
+        end
+      end
+      uri = "http://localhost:#{@@port}/"
+      attempt = 0
+      loop do
+        ping = Zold::Http.new(uri: uri).get
+        unless ping.status == 599 && node.alive?
+          @log.debug("The URL #{uri} is probably alive, after #{attempt} attempts")
+          break
         end
-        uri = "http://localhost:#{port}/"
-        attempt = 0
-        loop do
-          ping = Zold::Http.new(uri: uri).get
-          break unless ping.status == 599 && node.alive?
-          @log.info("Waiting for #{uri} (attempt no.#{attempt}): ##{ping.status}...")
-          sleep 0.5
-          attempt += 1
-          break if attempt > 10
+        unless node.alive?
+          @log.debug("The URL #{uri} is dead, after #{attempt} attempts")
+          break
         end
-        raise "The node is dead at #{uri}" unless node.alive?
-        begin
-          yield port
-        ensure
-          Zold::Http.new(uri: "#{uri}?halt=test").get
-          node.join
-          sleep 0.1 # stupid sleep to make sure all threads are terminated
+        @log.debug("Waiting for #{uri} (attempt no.#{attempt}): ##{ping.status}...")
+        sleep 0.5
+        attempt += 1
+        if attempt > 10
+          @log.error("Waiting for too long for #{uri} (#{attempt} attempts)")
+          break
         end
       end
+      raise "The node is dead at #{uri}" unless node.alive?
+      begin
+        yield @@port
+      ensure
+        Zold::Http.new(uri: "#{uri}?halt=test").get
+        node.join
+        sleep 0.1 # stupid sleep to make sure all threads are terminated
+      end
+      @log.debug("Thread with fake node stopped: #{node.alive?}")
     end
   end
 end

data/test/node/test_async_entrance.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-# Copyright (c) 2018-2023 Zerocracy
+# Copyright (c) 2018-2024 Zerocracy
 #
 # Permission is hereby granted, free of charge, to any person obtaining a copy
 # of this software and associated documentation files (the 'Software'), to deal
@@ -30,25 +30,25 @@ require_relative 'fake_entrance'
 
 # AsyncEntrance test.
 # Author:: Yegor Bugayenko (yegor256@gmail.com)
-# Copyright:: Copyright (c) 2018 Yegor Bugayenko
+# Copyright:: Copyright (c) 2018-2024 Zerocracy
 # License:: MIT
 class TestAsyncEntrance < Zold::Test
   def test_renders_json
-    FakeHome.new(log: test_log).run do |home|
-      Zold::AsyncEntrance.new(FakeEntrance.new, home.dir, log: test_log).start do |e|
+    FakeHome.new(log: fake_log).run do |home|
+      Zold::AsyncEntrance.new(FakeEntrance.new, home.dir, log: fake_log).start do |e|
         assert_equal(0, e.to_json[:queue])
       end
     end
   end
 
   def test_sends_through_once
-    FakeHome.new(log: test_log).run do |home|
+    FakeHome.new(log: fake_log).run do |home|
       wallet = home.create_wallet
       amount = Zold::Amount.new(zld: 39.99)
       key = Zold::Key.new(file: 'fixtures/id_rsa')
       wallet.sub(amount, "NOPREFIX@#{Zold::Id.new}", key)
       basic = CountingEntrance.new
-      Zold::AsyncEntrance.new(basic, File.join(home.dir, 'a/b/c'), log: test_log).start do |e|
+      Zold::AsyncEntrance.new(basic, File.join(home.dir, 'a/b/c'), log: fake_log).start do |e|
         e.push(wallet.id, File.read(wallet.path))
         assert_equal_wait(1) { basic.count }
       end
@@ -56,9 +56,9 @@ class TestAsyncEntrance < Zold::Test
   end
 
   def test_sends_through
-    FakeHome.new(log: test_log).run do |home|
+    FakeHome.new(log: fake_log).run do |home|
       basic = CountingEntrance.new
-      Zold::AsyncEntrance.new(basic, File.join(home.dir, 'a/b/c'), log: test_log, queue_limit: 1000).start do |e|
+      Zold::AsyncEntrance.new(basic, File.join(home.dir, 'a/b/c'), log: fake_log, queue_limit: 1000).start do |e|
         Threads.new(20).assert do
           wallet = home.create_wallet
           amount = Zold::Amount.new(zld: 39.99)
@@ -72,11 +72,11 @@ class TestAsyncEntrance < Zold::Test
   end
 
   def test_handles_broken_entrance_gracefully
-    FakeHome.new(log: test_log).run do |home|
+    FakeHome.new(log: fake_log).run do |home|
       wallet = home.create_wallet
       id = wallet.id
       body = File.read(wallet.path)
-      Zold::AsyncEntrance.new(BrokenEntrance.new, home.dir, log: test_log).start do |e|
+      Zold::AsyncEntrance.new(BrokenEntrance.new, home.dir, log: fake_log).start do |e|
         e.push(id, body)
       end
     end

data/test/node/test_entrance.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-# Copyright (c) 2018-2023 Zerocracy
+# Copyright (c) 2018-2024 Zerocracy
 #
 # Permission is hereby granted, free of charge, to any person obtaining a copy
 # of this software and associated documentation files (the 'Software'), to deal
@@ -34,16 +34,16 @@ require_relative '../../lib/zold/commands/pay'
 
 # ENTRANCE test.
 # Author:: Yegor Bugayenko (yegor256@gmail.com)
-# Copyright:: Copyright (c) 2018 Yegor Bugayenko
+# Copyright:: Copyright (c) 2018-2024 Zerocracy
 # License:: MIT
 class TestEntrance < Zold::Test
   def test_pushes_wallet
     sid = Zold::Id::ROOT
     tid = Zold::Id.new
-    body = FakeHome.new(log: test_log).run do |home|
+    body = FakeHome.new(log: fake_log).run do |home|
       source = home.create_wallet(sid)
       target = home.create_wallet(tid)
-      Zold::Pay.new(wallets: home.wallets, copies: home.dir, remotes: home.remotes, log: test_log).run(
+      Zold::Pay.new(wallets: home.wallets, copies: home.dir, remotes: home.remotes, log: fake_log).run(
         [
           'pay', '--force', '--private-key=fixtures/id_rsa',
           source.id.to_s, target.id.to_s, '19.99', 'testing'
@@ -51,14 +51,14 @@ class TestEntrance < Zold::Test
       )
       File.read(source.path)
     end
-    FakeHome.new(log: test_log).run do |home|
+    FakeHome.new(log: fake_log).run do |home|
       source = home.create_wallet(sid)
       target = home.create_wallet(tid)
       ledger = File.join(home.dir, 'ledger.csv')
       e = Zold::Entrance.new(
         home.wallets,
         Zold::Pipeline.new(home.remotes, home.copies(source).root, 'x', ledger: ledger),
-        log: test_log
+        log: fake_log
       )
       modified = e.push(source.id, body)
       assert_equal(2, modified.count)
@@ -71,9 +71,9 @@ class TestEntrance < Zold::Test
   end
 
   def test_renders_json
-    FakeHome.new(log: test_log).run do |home|
+    FakeHome.new(log: fake_log).run do |home|
       wallet = home.create_wallet
-      e = Zold::Entrance.new(home.wallets, Zold::Pipeline.new(home.remotes, home.copies.root, 'x'), log: test_log)
+      e = Zold::Entrance.new(home.wallets, Zold::Pipeline.new(home.remotes, home.copies.root, 'x'), log: fake_log)
       e.push(wallet.id, File.read(wallet.path))
       assert(e.to_json[:history].include?(wallet.id.to_s))
       assert(!e.to_json[:speed].negative?)

data/test/node/test_farm.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-# Copyright (c) 2018-2023 Zerocracy
+# Copyright (c) 2018-2024 Zerocracy
 #
 # Permission is hereby granted, free of charge, to any person obtaining a copy
 # of this software and associated documentation files (the 'Software'), to deal
@@ -30,7 +30,7 @@ require_relative '../../lib/zold/node/farm'
 class FarmTest < Zold::Test
   def test_renders_in_json
     Dir.mktmpdir do |dir|
-      farm = Zold::Farm.new('NOPREFIX6@ffffffffffffffff', File.join(dir, 'f'), log: test_log, strength: 2)
+      farm = Zold::Farm.new('NOPREFIX6@ffffffffffffffff', File.join(dir, 'f'), log: fake_log, strength: 2)
       farm.start('localhost', 80, threads: 2) do
         assert_wait { !farm.best.empty? && !farm.best[0].value.zero? }
         count = 0
@@ -42,7 +42,7 @@ class FarmTest < Zold::Test
 
   def test_renders_in_text
     Dir.mktmpdir do |dir|
-      farm = Zold::Farm.new('NOPREFIX7@ffffffffffffffff', File.join(dir, 'f'), log: test_log, strength: 1)
+      farm = Zold::Farm.new('NOPREFIX7@ffffffffffffffff', File.join(dir, 'f'), log: fake_log, strength: 1)
       farm.start('localhost', 80, threads: 2) do
         assert(!farm.to_text.nil?)
       end
@@ -52,7 +52,7 @@ class FarmTest < Zold::Test
   def test_makes_many_scores
     Dir.mktmpdir do |dir|
       farm = Zold::Farm.new('NOPREFIX6@ffffffffffffffff', File.join(dir, 'f'),
-        log: test_log, lifetime: 10, farmer: Zold::Farmers::Plain.new, strength: 1)
+        log: fake_log, lifetime: 10, farmer: Zold::Farmers::Plain.new, strength: 1)
       farm.start('localhost', 80, threads: 4) do
         assert_wait { farm.best.length == 4 }
       end
@@ -67,7 +67,7 @@ class FarmTest < Zold::Test
   #  that it's usually a few microseconds, but sometimes over 200ms.
   def test_reads_scores_at_high_speed
     Dir.mktmpdir do |dir|
-      farm = Zold::Farm.new('NOPREFIX6@ffffffffffffffff', File.join(dir, 'f'), log: test_log, strength: 4)
+      farm = Zold::Farm.new('NOPREFIX6@ffffffffffffffff', File.join(dir, 'f'), log: fake_log, strength: 4)
       farm.start('localhost', 80, threads: 4) do
         assert_wait { !farm.best.empty? && !farm.best[0].value.zero? }
         cycles = 100
@@ -76,14 +76,14 @@ class FarmTest < Zold::Test
           farm.best
           Time.now - start
         end.inject(&:+) / cycles
-        test_log.info("Average speed is #{(speed * 1000).round(2)}ms in #{cycles} cycles")
+        fake_log.info("Average speed is #{(speed * 1000).round(2)}ms in #{cycles} cycles")
       end
     end
   end
 
   def test_makes_best_score_in_background
     Dir.mktmpdir do |dir|
-      farm = Zold::Farm.new('NOPREFIX1@ffffffffffffffff', File.join(dir, 'f'), log: test_log, strength: 3)
+      farm = Zold::Farm.new('NOPREFIX1@ffffffffffffffff', File.join(dir, 'f'), log: fake_log, strength: 3)
       farm.start('localhost', 80, threads: 1) do
         assert_wait { !farm.best.empty? && farm.best[0].value >= 3 }
         score = farm.best[0]
@@ -95,7 +95,7 @@ class FarmTest < Zold::Test
 
   def test_correct_score_from_empty_farm
     Dir.mktmpdir do |dir|
-      farm = Zold::Farm.new('NOPREFIX2@cccccccccccccccc', File.join(dir, 'f'), log: test_log, strength: 1)
+      farm = Zold::Farm.new('NOPREFIX2@cccccccccccccccc', File.join(dir, 'f'), log: fake_log, strength: 1)
       farm.start('example.com', 8080, threads: 0) do
         score = farm.best[0]
         assert(!score.expired?)
@@ -109,7 +109,7 @@ class FarmTest < Zold::Test
   def test_pre_loads_history
     Dir.mktmpdir do |dir|
       cache = File.join(dir, 'a/b/c/cache')
-      farm = Zold::Farm.new('NOPREFIX3@cccccccccccccccc', cache, log: test_log, strength: 1)
+      farm = Zold::Farm.new('NOPREFIX3@cccccccccccccccc', cache, log: fake_log, strength: 1)
       farm.start('example.com', 8080, threads: 0) do
         score = farm.best[0]
         assert(!score.nil?, 'The list of best scores can\'t be empty!')
@@ -132,7 +132,7 @@ class FarmTest < Zold::Test
         strength: 6
       )
       File.write(cache, score.to_s)
-      farm = Zold::Farm.new('NOPREFIX4@ffffffffffffffff', cache, log: test_log, strength: score.strength)
+      farm = Zold::Farm.new('NOPREFIX4@ffffffffffffffff', cache, log: fake_log, strength: score.strength)
       farm.start(score.host, score.port, threads: 1) do
         100.times do
           sleep(0.1)
@@ -170,7 +170,7 @@ class FarmTest < Zold::Test
   end
 
   def test_terminates_farm_entirely
-    Zold::Farm.new('NOPREFIX4@ffffffffffffffff', log: test_log, strength: 10).start('localhost', 4096, threads: 1) do
+    Zold::Farm.new('NOPREFIX4@ffffffffffffffff', log: fake_log, strength: 10).start('localhost', 4096, threads: 1) do
       sleep 1
     end
   end

data/test/node/test_farmers.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-# Copyright (c) 2018-2023 Zerocracy
+# Copyright (c) 2018-2024 Zerocracy
 #
 # Permission is hereby granted, free of charge, to any person obtaining a copy
 # of this software and associated documentation files (the 'Software'), to deal
@@ -38,7 +38,7 @@ class FarmersTest < Zold::Test
   def test_calculates_next_score
     before = Zold::Score.new(host: 'some-host', port: 9999, invoice: 'NOPREFIX4@ffffffffffffffff', strength: 3)
     TYPES.each do |farmer_class|
-      farmer = farmer_class.new(log: test_log)
+      farmer = farmer_class.new(log: fake_log)
       after = farmer.up(before)
       assert_equal(1, after.value)
       assert(!after.expired?)
@@ -49,7 +49,7 @@ class FarmersTest < Zold::Test
 
   def test_calculates_large_score
     TYPES.each do |type|
-      log = TestLogger.new(test_log)
+      log = TestLogger.new(fake_log)
       thread = Thread.start do
         farmer = type.new(log: log)
         farmer.up(Zold::Score.new(host: 'a', port: 1, invoice: 'NOPREFIX4@ffffffffffffffff', strength: 20))
@@ -62,9 +62,9 @@ class FarmersTest < Zold::Test
 
   def test_kills_farmer
     TYPES.each do |type|
-      farmer = type.new(log: test_log)
+      farmer = type.new(log: fake_log)
       thread = Thread.start do
-        Zold::VerboseThread.new(test_log).run do
+        Zold::VerboseThread.new(fake_log).run do
           farmer.up(Zold::Score.new(host: 'some-host', invoice: 'NOPREFIX4@ffffffffffffffff', strength: 32))
         end
       end

data/test/node/test_front.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-# Copyright (c) 2018-2023 Zerocracy
+# Copyright (c) 2018-2024 Zerocracy
 #
 # Permission is hereby granted, free of charge, to any person obtaining a copy
 # of this software and associated documentation files (the 'Software'), to deal
@@ -45,7 +45,7 @@ class FrontTest < Zold::Test
   def test_memory_leakage
     skip
     report = MemoryProfiler.report(top: 10) do
-      FakeNode.new(log: test_log).run(opts('--network=foo')) do |port|
+      FakeNode.new(log: fake_log).run(opts('--network=foo')) do |port|
         100.times do
           Zold::Http.new(uri: "http://localhost:#{port}/", network: 'foo').get
         end
@@ -55,7 +55,7 @@ class FrontTest < Zold::Test
   end
 
   def test_renders_front_json
-    FakeNode.new(log: test_log).run(opts('--network=foo')) do |port|
+    FakeNode.new(log: fake_log).run(opts('--network=foo')) do |port|
       res = Zold::Http.new(uri: "http://localhost:#{port}/", network: 'foo').get
       json = JSON.parse(res.body)
       assert_equal(Zold::VERSION, json['version'])
@@ -74,7 +74,7 @@ class FrontTest < Zold::Test
   end
 
   def test_renders_public_pages
-    FakeNode.new(log: test_log).run(opts) do |port|
+    FakeNode.new(log: fake_log).run(opts) do |port|
       {
         200 => [
           '/robots.txt',
@@ -117,7 +117,7 @@ class FrontTest < Zold::Test
   end
 
   def test_updates_list_of_remotes
-    FakeNode.new(log: test_log).run(['--no-metronome', '--ignore-score-weakness', '--no-cache']) do |port|
+    FakeNode.new(log: fake_log).run(['--no-metronome', '--ignore-score-weakness', '--no-cache']) do |port|
       (Zold::Remotes::MAX_NODES + 5).times do |i|
         score = Zold::Score.new(
           host: 'localhost', port: i + 1, invoice: 'NOPREFIX@ffffffffffffffff', strength: 1
@@ -138,9 +138,9 @@ class FrontTest < Zold::Test
   end
 
   def test_increments_score
-    FakeNode.new(log: test_log).run(opts('--threads=1')) do |port|
+    FakeNode.new(log: fake_log).run(opts('--threads=1')) do |port|
       3.times do |i|
-        assert_equal_wait(true, max: 60) do
+        assert_equal_wait(true, max: 120) do
           response = Zold::Http.new(uri: "http://localhost:#{port}/").get
           assert_equal(200, response.status, response.body)
           score = Zold::Score.parse_json(Zold::JsonPage.new(response.body).to_hash['score'])
@@ -159,8 +159,8 @@ class FrontTest < Zold::Test
   ].each do |p|
     method = "test_wallet_page_#{p.gsub(/[^a-z]/, '_')}"
     define_method(method) do
-      FakeHome.new(log: test_log).run do |home|
-        FakeNode.new(log: test_log).run(opts) do |port|
+      FakeHome.new(log: fake_log).run do |home|
+        FakeNode.new(log: fake_log).run(opts) do |port|
           wallet = home.create_wallet(txns: 2)
           base = "http://localhost:#{port}"
           response = Zold::Http.new(uri: "#{base}/wallet/#{wallet.id}").put(wallet.path)
@@ -173,8 +173,8 @@ class FrontTest < Zold::Test
   end
 
   def test_renders_wallets_page
-    FakeHome.new(log: test_log).run do |home|
-      FakeNode.new(log: test_log).run(opts) do |port|
+    FakeHome.new(log: fake_log).run do |home|
+      FakeNode.new(log: fake_log).run(opts) do |port|
         wallet = home.create_wallet(txns: 2)
         base = "http://localhost:#{port}"
         response = Zold::Http.new(uri: "#{base}/wallet/#{wallet.id}").put(wallet.path)
@@ -188,8 +188,8 @@ class FrontTest < Zold::Test
   end
 
   def test_fetch_in_multiple_threads
-    FakeNode.new(log: test_log).run(opts) do |port|
-      FakeHome.new(log: test_log).run do |home|
+    FakeNode.new(log: fake_log).run(opts) do |port|
+      FakeHome.new(log: fake_log).run do |home|
         wallet = home.create_wallet
         base = "http://localhost:#{port}"
         Zold::Http.new(uri: "#{base}/wallet/#{wallet.id}").put(wallet.path)
@@ -209,8 +209,8 @@ class FrontTest < Zold::Test
   end
 
   def test_pushes_twice
-    FakeNode.new(log: test_log).run(opts) do |port|
-      FakeHome.new(log: test_log).run do |home|
+    FakeNode.new(log: fake_log).run(opts) do |port|
+      FakeHome.new(log: fake_log).run do |home|
         wallet = home.create_wallet
         base = "http://localhost:#{port}"
         assert_equal(
@@ -227,9 +227,9 @@ class FrontTest < Zold::Test
   end
 
   def test_pushes_many_wallets
-    FakeNode.new(log: test_log).run(opts) do |port|
+    FakeNode.new(log: fake_log).run(opts) do |port|
       base = "http://localhost:#{port}"
-      FakeHome.new(log: test_log).run do |home|
+      FakeHome.new(log: fake_log).run do |home|
         Threads.new(5).assert do
           wallet = home.create_wallet
           Zold::Http.new(uri: "#{base}/wallet/#{wallet.id}").put(wallet.path)
@@ -273,7 +273,7 @@ class FrontTest < Zold::Test
   end
 
   def test_gzip
-    FakeNode.new(log: test_log).run(opts) do |port|
+    FakeNode.new(log: fake_log).run(opts) do |port|
       response = Zold::Http.new(uri: URI("http://localhost:#{port}/version")).get
       assert_equal(200, response.status, response)
       assert_operator(300, :>, response.body.length.to_i, 'Expected the content to be small')
@@ -282,7 +282,7 @@ class FrontTest < Zold::Test
 
   def test_performance
     times = Queue.new
-    FakeNode.new(log: test_log).run(opts('--threads=4', '--strength=6')) do |port|
+    FakeNode.new(log: fake_log).run(opts('--threads=4', '--strength=6')) do |port|
       Threads.new(10).assert(100) do
         start = Time.now
         Zold::Http.new(uri: URI("http://localhost:#{port}/")).get
@@ -291,7 +291,7 @@ class FrontTest < Zold::Test
     end
     all = []
     all << times.pop(true) until times.empty?
-    test_log.info("Average response time is #{all.inject(&:+) / all.count}")
+    fake_log.info("Average response time is #{all.inject(&:+) / all.count}")
   end
 
   # The score exposed via the HTTP header must be reduced to the value of 16.
@@ -299,7 +299,7 @@ class FrontTest < Zold::Test
   # HTTP request. This value is enough to identify a valueable node, and filter
   # out those that are too weak.
   def test_score_is_reduced
-    FakeNode.new(log: test_log).run(opts('--threads=1', '--strength=1', '--farmer=plain')) do |port|
+    FakeNode.new(log: fake_log).run(opts('--threads=1', '--strength=1', '--farmer=plain')) do |port|
       scores = []
       50.times do
         res = Zold::Http.new(uri: URI("http://localhost:#{port}/")).get
@@ -311,7 +311,7 @@ class FrontTest < Zold::Test
   end
 
   def test_headers_are_being_set_correctly
-    FakeNode.new(log: test_log).run(opts('--expose-version=9.9.9')) do |port|
+    FakeNode.new(log: fake_log).run(opts('--expose-version=9.9.9')) do |port|
       response = Zold::Http.new(uri: URI("http://localhost:#{port}/")).get
       assert_equal('no-cache', response.headers['Cache-Control'])
       assert_equal('close', response.headers['Connection'])
@@ -325,7 +325,7 @@ class FrontTest < Zold::Test
 
   def test_alias_parameter
     name = SecureRandom.hex(4)
-    FakeNode.new(log: test_log).run(opts("--alias=#{name}")) do |port|
+    FakeNode.new(log: fake_log).run(opts("--alias=#{name}")) do |port|
       uri = URI("http://localhost:#{port}/")
       response = Zold::Http.new(uri: uri).get
       assert_match(
@@ -337,7 +337,7 @@ class FrontTest < Zold::Test
   end
 
   def test_default_alias_parameter
-    FakeNode.new(log: test_log).run(opts) do |port|
+    FakeNode.new(log: fake_log).run(opts) do |port|
       uri = URI("http://localhost:#{port}/")
       response = Zold::Http.new(uri: uri).get
       assert_match(
@@ -349,8 +349,9 @@ class FrontTest < Zold::Test
   end
 
   def test_invalid_alias
+    skip
     exception = assert_raises RuntimeError do
-      FakeNode.new(log: test_log).run(opts('--alias=invalid-alias')) do |port|
+      FakeNode.new(log: fake_log).run(opts('--alias=invalid-alias')) do |port|
         uri = URI("http://localhost:#{port}/")
         Zold::Http.new(uri: uri).get
       end
@@ -360,8 +361,8 @@ class FrontTest < Zold::Test
 
   def test_push_fetch_in_multiple_threads
     key = Zold::Key.new(text: File.read('fixtures/id_rsa'))
-    FakeNode.new(log: test_log).run(opts) do |port|
-      FakeHome.new(log: test_log).run do |home|
+    FakeNode.new(log: fake_log).run(opts) do |port|
+      FakeHome.new(log: fake_log).run do |home|
         wallet = home.create_wallet(Zold::Id::ROOT)
         base = "http://localhost:#{port}"
         Zold::Http.new(uri: "#{base}/wallet/#{wallet.id}").put(wallet.path)
@@ -380,7 +381,7 @@ class FrontTest < Zold::Test
   end
 
   def test_checksum_in_json
-    FakeNode.new(log: test_log).run(opts) do |port|
+    FakeNode.new(log: fake_log).run(opts) do |port|
       uri = URI("http://localhost:#{port}/")
       response = Zold::Http.new(uri: uri).get
       assert(

data/test/node/test_nodup_entrance.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-# Copyright (c) 2018-2023 Zerocracy
+# Copyright (c) 2018-2024 Zerocracy
 #
 # Permission is hereby granted, free of charge, to any person obtaining a copy
 # of this software and associated documentation files (the 'Software'), to deal
@@ -29,13 +29,13 @@ require_relative 'fake_entrance'
 
 # NoDupEntrance test.
 # Author:: Yegor Bugayenko (yegor256@gmail.com)
-# Copyright:: Copyright (c) 2018 Yegor Bugayenko
+# Copyright:: Copyright (c) 2018-2024 Zerocracy
 # License:: MIT
 class TestNoDupEntrance < Zold::Test
   def test_ignores_dup
-    FakeHome.new(log: test_log).run do |home|
+    FakeHome.new(log: fake_log).run do |home|
       wallet = home.create_wallet
-      Zold::NoDupEntrance.new(RealEntrance.new, home.wallets, log: test_log).start do |e|
+      Zold::NoDupEntrance.new(RealEntrance.new, home.wallets, log: fake_log).start do |e|
         assert(e.push(wallet.id, File.read(wallet.path)).empty?)
       end
     end