Faraday: 機能リクエスト： `Faraday :: Adapter :: Test`で` Faraday :: FlatParamsEncoder`をサポート

作成日 2021年08月19日 · 4コメント · ソース: lostisland/faraday

1291年には導入された`strict_mode`で`Faraday::Adapter::Test::Stubs` 、と私はモードのためのより多くの機能欲しい：サポート`Faraday::FlatParamsEncoder` 。たとえば、同じクエリパラメータ`a`使用してHTTPリクエストを複数回呼び出したい場合、テストスタブが異なる値を持つ同じパラメータがすべて正しく渡されているかどうかを確認する必要があります。

class Client
  def initialize(conn)
    <strong i="10">@conn</strong> = conn
  end

  def sushi(jname, params: {})
    res = @conn.get("/#{jname}", params)
    data = JSON.parse(res.body)
    data['name']
  end
end

RSpec.describe Client do
  let(:stubs) { Faraday::Adapter::Test::Stubs.new(strict_mode: true) }
  let(:conn)   { Faraday.new(request: { params_encoder: Faraday::FlatParamsEncoder }) { |b| b.adapter(:test, stubs) } }
  let(:client) { Client.new(conn) }

  it 'handles the same multiple URL parameters' do
    # The same parameter `a` is expected to appear multiple times with different values.
    stubs.get('/ebi?a=x&a=y&a=z') { [200, { 'Content-Type' => 'application/json' }, '{"name": "shrimp"}'] }

    # Test should pass
    expect(client.sushi('ebi', params: { a: %w[x y z] })).to eq('shrimp')

    # Test should not pass due to the lack of "x" and "y" for `a`
    # expect(client.sushi('ebi', params: { a: %w[z] })).to eq('shrimp')
    stubs.verify_stubbed_calls
  end
end

どう思いますか？理にかなっている場合は、プルリクエストを開きたいです。

feature help wanted

ソース

yykamei

最も参考になるコメント

また、 mainブランチが次のv2.0リリースを指していることを忘れないでください。
1.xリリースでもこれが必要な場合は、 1.xブランチから分岐する必要があります👍

iMacTia 2021年08月19日

❤2

全てのコメント4件

こんにちは@yykamei 、上記は私には理にかなっています、私はPRを歓迎します。
好奇心から、上記のテストで現在何が起こっていますか？最後のものは合格ですか？

iMacTia 2021年08月19日

❤2

こんにちは@iMacTia 、お返事ありがとう

好奇心から、上記のテストで現在何が起こっていますか？最後のものは合格ですか？

ああ、それは最新の1.xブランチを渡さないようです。これは、 Stub#paramsがFaraday::Utils.parse_nested_query構成されているためだと思います。以前のクエリパラメータを上書きするように見えます。

Randomized with seed 45661

Client
  handles the same multiple URL parameters (FAILED - 1)

Failures:

  1) Client handles the same multiple URL parameters
     Failure/Error:
       raise Stubs::NotFound, "no stubbed request for #{env[:method]} "\
                              "#{normalized_path} #{env[:body]}"

     Faraday::Adapter::Test::Stubs::NotFound:
       no stubbed request for get /ebi?a=x
     # ./lib/faraday/adapter/test.rb:253:in `call'
     # ./lib/faraday/rack_builder.rb:154:in `build_response'
     # ./lib/faraday/connection.rb:511:in `run_request'
     # ./lib/faraday/connection.rb:200:in `get'
     # ./ok.rb:11:in `sushi'
     # ./ok.rb:30:in `block (2 levels) in <top (required)>'

Finished in 0.00391 seconds (files took 0.78136 seconds to load)
1 example, 1 failure

Failed examples:

しかし、このようにコードを変更すると、テストに合格します。問題の説明を書き直しました。

diff --git a/a.rb b/a.rb
index c4b7084..91d795d 100644
--- a/a.rb
+++ b/a.rb
@@ -27,7 +27,7 @@ RSpec.describe Client do
     expect(client.sushi('ebi', params: { a: %w[x y z] })).to eq('shrimp')

     # Test should not pass due to the lack of "y" and "z" for `a`
-    expect(client.sushi('ebi', params: { a: %w[x] })).to eq('shrimp')
+    expect(client.sushi('ebi', params: { a: %w[z] })).to eq('shrimp')
     stubs.verify_stubbed_calls
   end
 end

とにかくありがとう。変更を加えます💪

yykamei 2021年08月20日

🚀1

この問題は＃1316で解決されました。

ありがとうございました！

yykamei 2021年09月01日

このページは役に立ちましたか？

0 / 5 - 0 評価

Faraday: 機能リクエスト： `Faraday :: Adapter :: Test`で` Faraday :: FlatParamsEncoder`をサポート

最も参考になるコメント

全てのコメント4件

関連する問題