Bad practice to send requests twice using the same connection in the same test?

If I were doing this, I’d probably do:

result_conn = put conn, ...

# do stuff

result_conn = get conn, ...

That way conn is never rebound and is always the fresh conn provided to the test. However, I agree that making two requests in the same test is probably a code smell.

Yeah, I’ll make sure to use the markdown in the future.

I inserted conn = recycle(conn) right before the second (conn, path, params), but the headers were not recycled into the new conn. Also, after discussing with my coworker, I am deciding to limit each test to 1 http request. Thanks for the tip though, I’ll consider using recycle in the future for similar issues.

It’s usually fine to reuse the conn, perhaps recycle helps?

If not, I think the big is in your implementation rather than in the tests.

Ps: it’s much easier to read if you actually copy and paste code and annotate it using standard markdown, than to put a screenshot, which is bad for smaller bandwidths or data plans.