fbe 0.25.2 → 0.26.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e3de49b140508afd1a989a717220dc48363172e87a1e505cd4389681a506978b
-  data.tar.gz: 408c55e983349bcc82055d06922f20748d332b0beafac507340ee2745fac895a
+  metadata.gz: 4d139bdf4702ebd62afe4fe4e0edd82b25b1842955c4e233be08c323eb6f8375
+  data.tar.gz: 35dc6d0bc91574b1b7f967892232c4b55f9096156538de5eacb51411ee20dcc7
 SHA512:
-  metadata.gz: e2f7b6c2b768012d2e600c03953053c756905a77e25b84b11be796efa678dc6a89ebd99d573761528c8295acf93a1e8f3c554ce755061ff771767625053ed195
-  data.tar.gz: '0990166f5c5283d87687e2bb512ed2ae14e2afa35404ff81f9d02517c45ecac6ec187a8f3c2a05fb3bda9cb7a4220b6e0130aa94e35d069306e2224124980961'
+  metadata.gz: 3b65d4e592f10cfef206ff69a3f630cec44851382cfc87c5ca092bb31be7181a430241da91cb2e74628ad2cf3137f2b7034b746a319f940a4378ad3e997dd2f0
+  data.tar.gz: 9f894315a98b5972ba20be68cb6848d2c7743a4e64990b48e4cf33504cf9dd5716f054dc081cc99f5b5a8083d257057d527006d00f8464cd663f7f79b6e6bf52

data/Gemfile.lock

@@ -47,7 +47,7 @@ GEM
     ast (2.4.3)
     backtrace (0.4.1)
     base64 (0.3.0)
-    baza.rb (0.9.11)
+    baza.rb (0.9.12)
       backtrace (~> 0.4)
       elapsed (~> 0.0)
       faraday (~> 2.13)
@@ -70,13 +70,13 @@ GEM
     decoor (0.1.0)
     docile (1.4.1)
     drb (2.2.3)
-    elapsed (0.1.0)
-      loog (> 0)
-      tago (> 0)
+    elapsed (0.2.0)
+      loog (~> 0.6)
+      tago (~> 0.1)
     ellipsized (0.3.0)
     ethon (0.16.0)
       ffi (>= 1.15.0)
-    factbase (0.14.0)
+    factbase (0.14.3)
       backtrace (~> 0.4)
       decoor (~> 0.0)
       ellipsized (~> 0.3)
@@ -87,7 +87,7 @@ GEM
       others (~> 0.0)
       tago (~> 0.0)
       yaml (~> 0.3)
-    faraday (2.13.3)
+    faraday (2.13.4)
       faraday-net_http (>= 2.0, < 3.5)
       json
       logger
@@ -120,8 +120,8 @@ GEM
     intercepted (0.2.0)
     iri (0.11.2)
     joined (0.4.0)
-    json (2.13.0)
-    judges (0.51.1)
+    json (2.13.2)
+    judges (0.52.1)
       backtrace (~> 0.4)
       baza.rb (~> 0.5)
       concurrent-ruby (~> 1.2)
@@ -173,7 +173,7 @@ GEM
     ostruct (0.6.3)
     others (0.1.1)
     parallel (1.27.0)
-    parser (3.3.8.0)
+    parser (3.3.9.0)
       ast (~> 2.4.1)
       racc
     prism (1.4.0)
@@ -189,7 +189,7 @@ GEM
     regexp_parser (2.10.0)
     retries (0.0.5)
     rexml (3.4.1)
-    rubocop (1.78.0)
+    rubocop (1.79.1)
       json (~> 2.3)
       language_server-protocol (~> 3.17.0.2)
       lint_roller (~> 1.1.0)
@@ -197,7 +197,7 @@ GEM
       parser (>= 3.3.0.2)
       rainbow (>= 2.2.2, < 4.0)
       regexp_parser (>= 2.9.3, < 3.0)
-      rubocop-ast (>= 1.45.1, < 2.0)
+      rubocop-ast (>= 1.46.0, < 2.0)
       ruby-progressbar (~> 1.7)
       unicode-display_width (>= 2.4.0, < 4.0)
     rubocop-ast (1.46.0)

data/lib/fbe/octo.rb

@@ -185,6 +185,7 @@ def Fbe.octo(options: $options, global: $global, loog: $loog)
             raise 'The name of the repo is nil' if name.nil?
             json = @origin.repository(name)
             id = json[:id]
+            raise "Repository #{name} not found" if id.nil?
             @loog.debug("GitHub repository #{name.inspect} has an ID: ##{id}")
             id
           end

data/lib/fbe/unmask_repos.rb

@@ -49,11 +49,12 @@ end
 # @param [Judges::Options] options Options containing 'repositories' field with masks
 # @param [Hash] global Global cache for storing API responses
 # @param [Loog] loog Logger for debug output
+# @param [quota_aware] Boolean Should we stop if quota is off?
 # @return [Array<String>] Shuffled list of repository full names (e.g., 'org/repo')
 # @raise [RuntimeError] If no repositories match the provided masks
 # @note Exclusion patterns must start with '-' (e.g., '-org/pattern*')
 # @note Results are shuffled to distribute load when processing
-def Fbe.unmask_repos(options: $options, global: $global, loog: $loog)
+def Fbe.unmask_repos(options: $options, global: $global, loog: $loog, quota_aware: true)
   raise 'Repositories mask is not specified' unless options.repositories
   raise 'Repositories mask is empty' if options.repositories.empty?
   repos = []
@@ -77,5 +78,12 @@ def Fbe.unmask_repos(options: $options, global: $global, loog: $loog)
   raise "No repos found matching: #{options.repositories.inspect}" if repos.empty?
   repos.shuffle!
   loog.debug("Scanning #{repos.size} repositories: #{repos.joined}...")
-  repos
+  return repos unless block_given?
+  repos.each do |repo|
+    if quota_aware && octo.off_quota?
+      $loog.info("No GitHub quota left, it is time to stop at #{repo}")
+      break
+    end
+    yield repo
+  end
 end

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.25.2' unless const_defined?(:VERSION)
+  VERSION = '0.26.1' unless const_defined?(:VERSION)
 end

data/test/fbe/test_octo.rb

@@ -64,6 +64,29 @@ class TestOcto < Fbe::Test
     assert_equal('dude56', nick)
   end
 
+  def test_reads_repo_id_by_name
+    WebMock.disable_net_connect!
+    stub_request(:get, 'https://api.github.com/rate_limit').to_return(
+      { body: '{}', headers: { 'X-RateLimit-Remaining' => '222' } }
+    )
+    o = Fbe.octo(loog: Loog::NULL, global: {}, options: Judges::Options.new)
+    stub_request(:get, 'https://api.github.com/repos/foo/bar').to_return(
+      body: { id: 42 }.to_json, headers: { 'Content-Type': 'application/json' }
+    )
+    id = o.repo_id_by_name('foo/bar')
+    assert_equal(42, id)
+  end
+
+  def test_reads_lost_repo_id_by_name
+    WebMock.disable_net_connect!
+    stub_request(:get, 'https://api.github.com/rate_limit').to_return(
+      { body: '{}', headers: { 'X-RateLimit-Remaining' => '222' } }
+    )
+    o = Fbe.octo(loog: Loog::NULL, global: {}, options: Judges::Options.new)
+    stub_request(:get, 'https://api.github.com/repos/foo/bar').to_return(status: 404)
+    assert_raises(StandardError) { o.repo_id_by_name('foo/bar') }
+  end
+
   def test_fails_user_request_when_off_quota
     WebMock.disable_net_connect!
     stub_request(:get, 'https://api.github.com/rate_limit').to_return(

data/test/fbe/test_unmask_repos.rb

@@ -25,6 +25,15 @@ class TestUnmaskRepos < Fbe::Test
     refute_includes(list, 'zerocracy/datum')
   end
 
+  def test_iterates_them
+    opts = Judges::Options.new({ 'testing' => true, 'repositories' => 'yegor256/tacit,zerocracy/*' })
+    list = []
+    Fbe.unmask_repos(options: opts, global: {}, loog: Loog::NULL) do |n|
+      list << n
+    end
+    assert_predicate(list.size, :positive?)
+  end
+
   def test_finds_case_insensitive
     opts = Judges::Options.new({ 'testing' => true, 'repositories' => 'Yegor256/*' })
     list = Fbe.unmask_repos(options: opts, global: {}, loog: Loog::NULL)

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: fbe
 version: !ruby/object:Gem::Version
-  version: 0.25.2
+  version: 0.26.1
 platform: ruby
 authors:
 - Yegor Bugayenko