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일

또한 main 분기가 이제 곧 출시될 v2.0 릴리스를 가리키고 있음을 기억하십시오.
1.x 릴리스에서도 이것을 원하면 1.x 분기를 분기해야 합니다. 👍

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 댓글

관련 문제