Overriding user-agent when using cy.request or cy.visit

[AbdullahZN]

Added possibility to override user-agent headers for request.sendannd request.sendStream

Since headers can be provided by the client either in lowercase/uppercase/mix of both:

• I also added a method to lowercase all the provided headers

💡 I don't know where this kind of utils method should go, is it fine in the request.coffee ?

Closes #3873
Closes #3719

[CLAassistant]

All committers have signed the CLA.

[AbdullahZN]

Errors look unrelated to the changes. Can we try a rebuild ?

[jennifer-shehane]

Other issues regarding lowercase request headers:

• Stub request headers response control work only for lowercase headers names #3637
• Headers change from original case to all lowercase during cy.request() #2879

[AbdullahZN]

Hi,
Any update on this ?

[AbdullahZN]

Thanks for the review @flotwig, corrected 👍

[flotwig]

Looks good @AbdullahZN, but the same unit test is failing every time this PR runs, so that will need to be figured out before this can be merged.

[AbdullahZN]

I'll dig into that. It's hard to debug though: tests are failing on my local develop branch 🤔

[AbdullahZN]

@flotwig It's ok now, tests are green 🍏 :)

[flotwig]

I think this will also fix #3719, since cy.visit() goes through request. Unless there's some other place where user-agent gets written.

[AbdullahZN]

It will require a bit more work. I'll try to handle it in another PR

[flotwig]

@AbdullahZN I added a test in your branch for cy.visit user-agent, it appears to be passing with just the changes you've made: 3d1c832

We can close #3719 too then, nice