Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Stop hiding exception raised by composer #9657

Merged
merged 22 commits into from
May 15, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
22 commits
Select commit Hold shift + click to select a range
adff15b
Stop hiding exception raised by composer
robaiken May 2, 2024
5da1fe1
Log error, rather than raise it
robaiken May 3, 2024
be91073
lint fix
robaiken May 3, 2024
e3bf86d
Merge branch 'main' into robaiken/dont-suppress-composer-error
robaiken May 3, 2024
ff8c892
Merge branch 'main' into robaiken/dont-suppress-composer-error
robaiken May 3, 2024
df0af0f
Adding test for missing native extension
robaiken May 7, 2024
4925f2c
remove debugger
robaiken May 7, 2024
bf84d05
lint
robaiken May 7, 2024
f763edd
added test for missing native extension logs an error
robaiken May 8, 2024
b7ca065
lint
robaiken May 8, 2024
fc0b827
Merge branch 'main' into robaiken/dont-suppress-composer-error
robaiken May 8, 2024
667da5b
updating test descriptions
robaiken May 8, 2024
b86369b
Merge branch 'main' into robaiken/dont-suppress-composer-error
robaiken May 8, 2024
6a4ac2e
Merge branch 'main' into robaiken/dont-suppress-composer-error
robaiken May 9, 2024
312af66
updating test to check logger is called
robaiken May 10, 2024
8328573
Merge remote-tracking branch 'origin/robaiken/dont-suppress-composer-…
robaiken May 10, 2024
398e260
Removing fixture
robaiken May 14, 2024
d977648
Improving spys
robaiken May 14, 2024
78cf509
lint
robaiken May 14, 2024
500e7b5
Merge branch 'main' into robaiken/dont-suppress-composer-error
robaiken May 14, 2024
532da13
Merge branch 'main' into robaiken/dont-suppress-composer-error
robaiken May 15, 2024
2f82f30
Merge branch 'main' into robaiken/dont-suppress-composer-error
robaiken May 15, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -307,7 +307,10 @@ def handle_composer_errors(error)

# If there *is* a lockfile we can't confidently distinguish between
# cases where we can't install and cases where we can't update. For
# now, we therefore just ignore the dependency.
# now, we therefore just ignore the dependency and log the error.

Dependabot.logger.error(error.message)
robaiken marked this conversation as resolved.
Show resolved Hide resolved
error.backtrace.each { |line| Dependabot.logger.error(line) }
nil
elsif error.message.include?("URL required authentication") ||
error.message.include?("403 Forbidden")
Expand Down
15 changes: 14 additions & 1 deletion composer/spec/dependabot/composer/update_checker_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -513,7 +513,7 @@
end
end

context "with an update that can't resolve" do
context "with an update that can't resolve due to a version conflict" do
let(:project_name) { "version_conflict_on_update" }
let(:dependency_name) { "longman/telegram-bot" }
let(:dependency_version) { "2.1.5" }
Expand All @@ -528,6 +528,19 @@

it { is_expected.to be_nil }

it "logs an error" do
allow(Dependabot.logger).to receive(:error)

is_expected.to be_nil
expect(Dependabot.logger).to have_received(:error).with(
a_string_starting_with("Your requirements could not be resolved to an installable set of packages.")
).once

expect(Dependabot.logger).to have_received(:error).with(
a_string_starting_with("/home/dependabot/")
).at_least(:once)
end

context "and there is no lockfile" do
let(:project_name) { "version_conflict_on_update_without_lockfile" }

Expand Down