Skip to content

Commit

Permalink
Remove mention of grouped-update key
Browse files Browse the repository at this point in the history
Co-authored-by: Barry Gordon <896971+brrygrdn@users.noreply.github.com>
  • Loading branch information
Nishnha and brrygrdn committed Apr 25, 2023
1 parent 5529ce8 commit f6916ee
Showing 1 changed file with 1 addition and 6 deletions.
7 changes: 1 addition & 6 deletions updater/spec/dependabot/api_client_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -175,13 +175,12 @@
end

context "grouped updates" do
it "does not include the grouped-update or dependency-group keys by default" do
it "does not include the dependency-group key by default" do
client.create_pull_request(dependency_change, base_commit)

expect(WebMock).
to(have_requested(:post, create_pull_request_url).
with do |req|
expect(req.body).not_to include("grouped-update")
expect(req.body).not_to include("dependency-group")
end)
end
Expand All @@ -202,10 +201,6 @@
to(have_requested(:post, create_pull_request_url).
with do |req|
data = JSON.parse(req.body)["data"]
# FIXME: Once we fully remove the `group-all` prototype behavior, we can also remove the
# `grouped-update` expectation since dependency-groups will require rules and only
# return the `dependency-group` key
expect(data["grouped-update"]).to be nil
expect(data["dependency-group"]).to eq({ "name" => "dummy_group_name" })
end)
end
Expand Down

0 comments on commit f6916ee

Please sign in to comment.