Skip to content

Commit

Permalink
Update tests to exclude dependencies in tests_require
Browse files Browse the repository at this point in the history
  • Loading branch information
amazimbe committed Aug 8, 2024
1 parent 457e55d commit d1732fe
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 3 deletions.
1 change: 1 addition & 0 deletions python/lib/dependabot/python/metadata_finder.rb
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,7 @@ def homepage_url
def look_up_source
potential_source_urls = [
pypi_listing.dig("info", "project_urls", "Source"),
pypi_listing.dig("info", "project_urls", "Repository"),
pypi_listing.dig("info", "home_page"),
pypi_listing.dig("info", "download_url"),
pypi_listing.dig("info", "docs_url")
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -175,7 +175,7 @@
describe "parse" do
subject(:dependencies) { parser.dependency_set.dependencies }

its(:length) { is_expected.to eq(15) }
its(:length) { is_expected.to eq(13) }

describe "an install_requires dependencies" do
subject(:dependency) { dependencies.find { |d| d.name == "boto3" } }
Expand Down Expand Up @@ -216,7 +216,7 @@
describe "a tests_require dependencies" do
subject(:dependency) { dependencies.find { |d| d.name == "responses" } }

it "has the right details" do
it "has the right details", skip: "Issue: https://github.com/dependabot/dependabot-core/issues/10388" do
expect(dependency).to be_a(Dependabot::Dependency)
expect(dependency.name).to eq("responses")
expect(dependency.version).to eq("0.5.1")
Expand Down
2 changes: 1 addition & 1 deletion python/spec/dependabot/python/file_parser_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -996,7 +996,7 @@
)
end

its(:length) { is_expected.to eq(15) }
its(:length) { is_expected.to eq(13) }

describe "an install_requires dependencies" do
subject(:dependency) { dependencies.find { |d| d.name == "boto3" } }
Expand Down

0 comments on commit d1732fe

Please sign in to comment.