Faraday: फ़ीचर अनुरोध: 'फैराडे :: फ्लैट पैराम्स एन्कोडर' का समर्थन 'फैराडे :: एडेप्टर :: टेस्ट' में करें

को निर्मित 19 अग॰ 2021 · 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 19 अग॰ 2021

❤2

सभी 4 टिप्पणियाँ

हाय @yykamei , उपरोक्त मेरे लिए समझ में आता है, मैं एक पीआर का स्वागत करता हूं।
जिज्ञासा से बाहर, वर्तमान में उपरोक्त परीक्षणों के साथ क्या होता है? क्या आखिरी गुजर रहा है?

iMacTia 19 अग॰ 2021

❤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 20 अग॰ 2021

🚀1

यह समस्या #1316 के साथ हल हो गई थी।

शुक्रिया!

yykamei 1 सित॰ 2021

क्या यह पृष्ठ उपयोगी था?

0 / 5 - 0 रेटिंग्स

Faraday: फ़ीचर अनुरोध: 'फैराडे :: फ्लैट पैराम्स एन्कोडर' का समर्थन 'फैराडे :: एडेप्टर :: टेस्ट' में करें

सबसे उपयोगी टिप्पणी

सभी 4 टिप्पणियाँ

संबंधित मुद्दों