Clarify release lookup error handling

Author: iangmaia

lib/fastlane/plugin/wpmreleasetoolkit/helper/github_helper.rb

@@ -254,6 +254,8 @@ def find_release(repository:, version:)
       def release_for_tag(repository:, version:)
         client.release_for_tag(repository, version)
       rescue Octokit::NotFound
+        # A 404 only means the fallback did not find a published release.
+        # Other API errors should be surfaced.
         nil
       end
 

spec/github_helper_spec.rb

@@ -745,6 +745,20 @@ def create_release(is_draft:, assets: [], name: nil)
       end
     end
 
+    it 'does not hide unexpected direct release lookup errors' do
+      allow(client).to receive(:releases).with(test_repo).and_return([])
+      allow(client).to receive(:release_for_tag).with(test_repo, test_version).and_raise(Octokit::TooManyRequests)
+
+      with_tmp_file(named: 'test-app.zip') do |file_path|
+        expect(client).not_to receive(:release_assets)
+        expect(client).not_to receive(:upload_asset)
+
+        expect do
+          upload_release_assets(assets: [file_path])
+        end.to raise_error(Octokit::TooManyRequests)
+      end
+    end
+
     it 'uploads one asset to the existing release' do
       with_tmp_file(named: 'test-app.zip') do |file_path|
         expect(client).to receive(:upload_asset).with(release_url, file_path, { content_type: 'application/octet-stream' })