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

Bump rack-test from 1.1.0 to 2.1.0 #12715

Merged
merged 0 commits into from
May 31, 2023

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github May 18, 2023

Bumps rack-test from 1.1.0 to 2.1.0.

Changelog

Sourced from rack-test's changelog.

2.1.0 / 2023-03-14

  • Breaking changes:

    • Digest authentication support, deprecated in 2.0.0, has been removed (Jeremy Evans #307)
    • requiring rack/mock_session, deprecated in 2.0.0, has been removed (Jeremy Evans #307)
  • Minor enhancements:

    • The original_filename for Rack::Test::UploadedFile can now be set even if the content of the file comes from a file path (Stuart Chinery #314)
    • Add Rack::Test::Session#restore_state, for executing a block and restoring current state (last request, last response, and cookies) after the block (Jeremy Evans #316)
    • Make Rack::Test::Methods support default_host method similar to app, which will set the default host used for requests to the app (Jeremy Evans #317 #318)
    • Allow responses to set cookie paths not matching the current request URI. Such cookies will only be sent for paths matching the cookie path (Chris Waters #322)
    • Ignore leading dot for cookie domains, per RFC 6265 (Stephen Crosby #329)
    • Avoid creating empty multipart body if params is empty in Rack::Test::Session#env_for (Ryunosuke Sato #331)

2.0.2 / 2022-06-28

  • Bug fixes:
    • Fix additional incompatible character encodings error when building uploaded bodies (Jeremy Evans #311)

2.0.1 / 2022-06-27

  • Bug fixes:
    • Fix incompatible character encodings error when building uploaded file bodies (Jeremy Evans #308 #309)

2.0.0 / 2022-06-24

  • Breaking changes:
    • Digest authentication support is now deprecated, as it relies on digest authentication support in rack, which has been deprecated (Jeremy Evans #294)
    • Rack::Test::Utils.build_primitive_part no longer handles array values (Jeremy Evans #292)
    • Rack::Test::Utils module methods other than build_nested_query and build_multipart are now private methods (Jeremy Evans #297)
    • Rack::MockSession has been combined into Rack::Test::Session, and remains as an alias to Rack::Test::Session, but to keep some

... (truncated)

Commits
  • 867a6f1 Release 2.1.0
  • 44e8be4 Update History.md
  • 5a2a3bf Don't raise error with empty params and multipart header
  • febbea4 Update CHANGELOG
  • 79c5077 Fix tests when rack master branch is used
  • 70f4db2 Ignore leading dot when merging cookies
  • 73c7174 Bump cgi version in Ruby 3.1 CI test
  • 8fb4fb2 Add CI for Ruby 3.2
  • 10e7040 Test JRuby 9.4 in CI
  • 2036831 change order of arguments for assert_equal example in README
  • Additional commits viewable in compare view

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

@dependabot dependabot bot requested review from a team as code owners May 18, 2023 09:59
@dependabot dependabot bot added the dependencies Pull requests that update a dependency file label May 18, 2023
@va-vsp-bot va-vsp-bot requested a deployment to dependabot/bundler/rack-test-2.1.0/main/main May 18, 2023 10:00 In progress
@va-vfs-bot va-vfs-bot temporarily deployed to dependabot/bundler/rack-test-2.1.0/main/main May 18, 2023 10:13 Inactive
ryan-mcneil
ryan-mcneil previously approved these changes May 25, 2023
Copy link
Contributor

@ryan-mcneil ryan-mcneil left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Jk, issues in tests. Pulling down to investigate...

@ryan-mcneil ryan-mcneil enabled auto-merge (squash) May 25, 2023 19:33
@va-vsp-bot va-vsp-bot requested a deployment to dependabot/bundler/rack-test-2.1.0/main/main May 25, 2023 19:34 In progress
@ryan-mcneil ryan-mcneil dismissed their stale review May 25, 2023 20:07

failing tests

@ryan-mcneil ryan-mcneil requested a review from a team as a code owner May 25, 2023 21:20
@va-vsp-bot va-vsp-bot requested a deployment to dependabot/bundler/rack-test-2.1.0/main/main May 25, 2023 21:20 In progress
@ryan-mcneil ryan-mcneil disabled auto-merge May 25, 2023 21:20
@va-vsp-bot va-vsp-bot requested a deployment to dependabot/bundler/rack-test-2.1.0/main/main May 25, 2023 21:21 In progress
@ryan-mcneil ryan-mcneil self-assigned this May 26, 2023
@va-vsp-bot va-vsp-bot requested a deployment to dependabot/bundler/rack-test-2.1.0/main/main May 26, 2023 19:01 In progress
@va-vsp-bot va-vsp-bot requested a deployment to dependabot/bundler/rack-test-2.1.0/main/main May 31, 2023 14:40 In progress
@va-vsp-bot va-vsp-bot requested a deployment to dependabot/bundler/rack-test-2.1.0/main/main May 31, 2023 17:21 In progress
@va-vsp-bot va-vsp-bot requested a deployment to dependabot/bundler/rack-test-2.1.0/main/main May 31, 2023 17:22 In progress
@@ -52,7 +52,7 @@ def documents
@documents ||= params.slice(*document_keys).values.map do |document|
case document
when String
decode_document(document)
document.blank? ? nil : decode_document(document)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

rack-test was converting a nil attachment to a blank string, which was causing a failure down the road here

@@ -101,7 +101,7 @@ def self.get_partname(headers)
def self.get_content_type(headers)
headers.each do |header|
name, _, value = header.partition(':')
return value.split(';')[0].strip if name == 'Content-Type'
return value.split(';')[0].strip if name.downcase == 'content-type'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

rack-test start sending headers downcased, so I just made these comparisons agnostic of the case of the header

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

related: rack/rack#1592

@ryan-mcneil ryan-mcneil merged commit 26a79b4 into master May 31, 2023
@ryan-mcneil ryan-mcneil deleted the dependabot/bundler/rack-test-2.1.0 branch May 31, 2023 20:53
ryan-mcneil added a commit that referenced this pull request Dec 11, 2023
* Bump rack-test from 1.1.0 to 2.1.0

Bumps [rack-test](https://github.com/rack/rack-test) from 1.1.0 to 2.1.0.
- [Release notes](https://github.com/rack/rack-test/releases)
- [Changelog](https://github.com/rack/rack-test/blob/main/History.md)
- [Commits](rack/rack-test@v1.1.0...v2.1.0)

---
updated-dependencies:
- dependency-name: rack-test
  dependency-type: direct:development
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>

* fix capitalization issue with rack-test

* add case for empty string (rack-test)

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Ryan McNeil <ryan.mcneil@oddball.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
console-services-review dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants