fbe 0.21.2 → 0.22.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f6593e4483941c4ff99de6f85aab33f62832d4293b1efb2a8e84d0b78ad79147
-  data.tar.gz: 134ceee60e4eb50c33b8c71194ffa0464c822e07884bd230776e5c59f025e2f7
+  metadata.gz: e5a39208fa372c54abc221588d0a96f234cfd7ba3c56a65fa1c9d71a5eb119e9
+  data.tar.gz: 97ef98fd304ce75ec5558a8915d3a8f95482cf0d203d3ede0882021b7bc036ea
 SHA512:
-  metadata.gz: a43a4fb0ef720d9e459d348437c7d3824625fae9aad064c768885fc945d25e0ba68b0034132c97dd0835bf93edbcd960d64d4aa9f9bbd31af5c5589f2121ef44
-  data.tar.gz: 5f76047d167636f220b5bcab707b2f69bbe47a471b5954a4dbe324055fb08556cec5470ac7d1d9bb22da436b91d69056b5205bf485b4ca3b63ea212eb0f649ad
+  metadata.gz: e17fbbbc95d55c4e0a9ec6611ba5ab1fcee8e42fb9e126a3d46bdb3e20bbad7d0c140d58f519275f71ce11d4a2d6399a211868638e208324b42dc5e577a988e1
+  data.tar.gz: d1fac9af3a2e773f23ad7a9392499feb01af4b7786654cf2fdb7140744e2fec4386d5ce45b49ea37432a9f31b9d9ca97fcec3b7c8be5b638b833ce6215b898f0

data/Gemfile.lock

@@ -180,7 +180,7 @@ GEM
     regexp_parser (2.10.0)
     retries (0.0.5)
     rexml (3.4.1)
-    rubocop (1.76.2)
+    rubocop (1.77.0)
       json (~> 2.3)
       language_server-protocol (~> 3.17.0.2)
       lint_roller (~> 1.1.0)

data/fbe.gemspec

@@ -18,7 +18,7 @@ Gem::Specification.new do |s|
     'manipulate the facts and create new ones'
   s.authors = ['Yegor Bugayenko']
   s.email = 'yegor256@gmail.com'
-  s.homepage = 'http://github.com/zerocracy/fbe'
+  s.homepage = 'https://github.com/zerocracy/fbe'
   s.files = `git ls-files`.split($RS)
   s.rdoc_options = ['--charset=UTF-8']
   s.extra_rdoc_files = ['README.md', 'LICENSE.txt']

data/lib/fbe/middleware/sqlite_store.rb

@@ -51,8 +51,9 @@ class Fbe::Middleware::SqliteStore
   # @param version [String] Version identifier for cache compatibility
   # @param loog [Loog] Logger instance (optional, defaults to Loog::NULL)
   # @param maxsize [Integer] Maximum database size in bytes (optional, defaults to 10MB)
+  # @param maxvsize [Integer] Maximum size in bytes of a single value (optional, defaults to 10Kb)
   # @raise [ArgumentError] If path is nil/empty, directory doesn't exist, or version is nil/empty
-  def initialize(path, version, loog: Loog::NULL, maxsize: 10 * 1024 * 1024)
+  def initialize(path, version, loog: Loog::NULL, maxsize: 10 * 1024 * 1024, maxvsize: 10 * 1024)
     raise ArgumentError, 'Database path cannot be nil or empty' if path.nil? || path.empty?
     dir = File.dirname(path)
     raise ArgumentError, "Directory #{dir} does not exist" unless File.directory?(dir)
@@ -61,6 +62,7 @@ class Fbe::Middleware::SqliteStore
     @version = version
     @loog = loog
     @maxsize = maxsize
+    @maxvsize = maxvsize
   end
 
   # Read a value from the cache.
@@ -91,10 +93,10 @@ class Fbe::Middleware::SqliteStore
   def write(key, value)
     return if value.is_a?(Array) && value.any? do |vv|
       req = JSON.parse(vv[0])
-      req['url'].include?('?') || req['method'] != 'get'
+      req['method'] != 'get'
     end
     value = JSON.dump(value)
-    return if value.bytesize > 10_000
+    return if value.bytesize > @maxvsize
     perform do |t|
       t.execute(<<~SQL, [key, value, Time.now.utc.iso8601])
         INSERT INTO cache(key, value, touched_at) VALUES(?1, ?2, ?3)

data/lib/fbe/octo.rb

@@ -83,7 +83,8 @@ def Fbe.octo(options: $options, global: $global, loog: $loog)
             )
             if options.sqlite_cache
               maxsize = options.sqlite_cache_maxsize || (10 * 1024 * 1024)
-              store = Fbe::Middleware::SqliteStore.new(options.sqlite_cache, Fbe::VERSION, loog:, maxsize:)
+              maxvsize = options.sqlite_cache_maxxsize || (10 * 1024)
+              store = Fbe::Middleware::SqliteStore.new(options.sqlite_cache, Fbe::VERSION, loog:, maxsize:, maxvsize:)
               loog.info(
                 "Using HTTP cache in SQLite file: #{store.path} (" \
                 "#{File.exist?(store.path) ? "#{File.size(store.path)} bytes" : 'file is absent'}, " \
@@ -176,7 +177,9 @@ def Fbe.octo(options: $options, global: $global, loog: $loog)
         end
       o =
         others(o:) do |*args|
-          raise "We are off-quota (remaining: #{@o.rate_limit.remaining})" if args.first != :off_quota? && @o.off_quota?
+          if args.first != :off_quota? && args.first != :print_trace! && @o.off_quota?
+            raise "We are off-quota (remaining: #{@o.rate_limit.remaining})"
+          end
           @o.__send__(*args)
         end
       o

data/lib/fbe.rb

@@ -10,5 +10,5 @@
 # License:: MIT
 module Fbe
   # Current version of the gem (changed by +.rultor.yml+ on every release)
-  VERSION = '0.21.2' unless const_defined?(:VERSION)
+  VERSION = '0.22.0' unless const_defined?(:VERSION)
 end

data/test/fbe/middleware/test_sqlite_store.rb

@@ -143,8 +143,8 @@ class SqliteStoreTest < Fbe::Test
       Fbe::Middleware::SqliteStore.new(f, '0.0.1').then do |store|
         store.write('a', 'a' * 9_997)
         store.write('b', 'b' * 9_998)
-        store.write('c', 'c' * 9_999)
-        store.write('d', 'd' * 10_000)
+        store.write('c', 'c' * 19_999)
+        store.write('d', 'd' * 30_000)
         assert_equal('a' * 9_997, store.read('a'))
         assert_equal('b' * 9_998, store.read('b'))
         assert_nil(store.read('c'))

data/test/fbe/test_octo.rb

@@ -56,8 +56,7 @@ class TestOcto < Fbe::Test
     stub_request(:get, 'https://api.github.com/rate_limit').to_return(
       { body: '{}', headers: { 'X-RateLimit-Remaining' => '222' } }
     )
-    global = {}
-    o = Fbe.octo(loog: Loog::NULL, global:, options: Judges::Options.new)
+    o = Fbe.octo(loog: Loog::NULL, global: {}, options: Judges::Options.new)
     stub_request(:get, 'https://api.github.com/user/42').to_return(
       body: { login: 'Dude56' }.to_json, headers: { 'Content-Type': 'application/json' }
     )
@@ -65,6 +64,24 @@ class TestOcto < Fbe::Test
     assert_equal('dude56', nick)
   end
 
+  def test_fails_user_request_when_off_quota
+    WebMock.disable_net_connect!
+    stub_request(:get, 'https://api.github.com/rate_limit').to_return(
+      { body: '{}', headers: { 'X-RateLimit-Remaining' => '3' } }
+    )
+    o = Fbe.octo(loog: Loog::NULL, global: {}, options: Judges::Options.new)
+    assert_raises(StandardError) { o.user(42) }
+  end
+
+  def test_no_failure_on_printing_when_off_quota
+    WebMock.disable_net_connect!
+    stub_request(:get, 'https://api.github.com/rate_limit').to_return(
+      { body: '{}', headers: { 'X-RateLimit-Remaining' => '3' } }
+    )
+    o = Fbe.octo(loog: Loog::NULL, global: {}, options: Judges::Options.new)
+    o.print_trace!
+  end
+
   def test_reads_repo_name_by_id
     WebMock.disable_net_connect!
     stub_request(:get, 'https://api.github.com/rate_limit').to_return(

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: fbe
 version: !ruby/object:Gem::Version
-  version: 0.21.2
+  version: 0.22.0
 platform: ruby
 authors:
 - Yegor Bugayenko
@@ -358,7 +358,7 @@ files:
 - test/fbe/test_who.rb
 - test/test__helper.rb
 - test/test_fbe.rb
-homepage: http://github.com/zerocracy/fbe
+homepage: https://github.com/zerocracy/fbe
 licenses:
 - MIT
 metadata: