Merge bitcoin/bitcoin#36160: refactor: Minor improvements to HTTP unit tests
f04b0c31627adda1d54b7da4507cfe30a63f7e5e refactor(test): Make test method use Uppercase, shorten enum values (Hodlinator)
6eea8e313cce96be15fc0b5bb95c58aa9b77202b refactor(test): Simplify HTTP response check (Hodlinator)
Pull request description:
Improves recently added unit tests.
* Simplify HTTP response check in `http_server_socket_tests`, it was incorrectly referring to `unordered_map`
* Make `http_request_state_tests` unit test method use Uppercase as per developer-notes.md, shorten `enum` values for readability
ACKs for top commit:
winterrdog:
tACK f04b0c31627adda1d54b7da4507cfe30a63f7e5e
janb84:
ACK f04b0c31627adda1d54b7da4507cfe30a63f7e5e
pinheadmz:
ACK f04b0c31627adda1d54b7da4507cfe30a63f7e5e
sedited:
ACK f04b0c31627adda1d54b7da4507cfe30a63f7e5e
Tree-SHA512: fd2ecb2f6803b2eafbcc9dd4881b8370be3b91ae6680a5d390af4da674b6c43fbc67ba4724762f9e71a5c455828ab07708ee880ca30f62947d6850dc8d9669cf M
merge-script committed
a687188101af0c797ee190fc3811d915d48dc0aa