Faraday: Solicitud de función: soporte `Faraday :: FlatParamsEncoder` en` Faraday :: Adapter :: Test`

Creado en 19 ago. 2021 · 4Comentarios · Fuente: lostisland/faraday

1291 introdujo `strict_mode` en `Faraday::Adapter::Test::Stubs` , y quiero más funciones para el modo: compatible con `Faraday::FlatParamsEncoder` . Por ejemplo, si quiero llamar a una solicitud HTTP con el mismo parámetro de consulta `a` varias veces, espero que el código auxiliar de prueba verifique si los mismos parámetros con valores diferentes se pasaron correctamente.

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

¿Qué piensas? Quiero abrir una solicitud de extracción si tiene sentido.

feature help wanted

Fuente

yykamei

Comentario más útil

Además, recuerde que la rama main ahora apunta a la próxima versión v2.0.
Si también desea esto en una versión 1.x, deberá ramificar la rama 1.x 👍

iMacTia en 19 ago. 2021

❤2

Todos 4 comentarios

Hola @yykamei , lo anterior tiene sentido para mí, agradecería un PR.
Por curiosidad, ¿qué sucede actualmente con las pruebas anteriores? ¿Está pasando el último?

iMacTia en 19 ago. 2021

Además, recuerde que la rama main ahora apunta a la próxima versión v2.0.
Si también desea esto en una versión 1.x, deberá ramificar la rama 1.x 👍

iMacTia en 19 ago. 2021

❤2

Hola @iMacTia , gracias por tu respuesta.

Por curiosidad, ¿qué sucede actualmente con las pruebas anteriores? ¿Está pasando el último?

Oh, parece no pasar en la última rama 1.x . Creo que esto se debe a que Stub#params se construye con Faraday::Utils.parse_nested_query ; parece anular los parámetros de consulta anteriores.

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:

Pero, si cambio el código de esta manera, la prueba pasará. Reescribí la descripción del problema.

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

Gracias de cualquier manera. Voy a hacer algunos cambios 💪

yykamei en 20 ago. 2021

🚀1

Este problema se resolvió con # 1316.

¡Gracias!

yykamei en 1 sept. 2021

¿Fue útil esta página

0 / 5 - 0 calificaciones