Marshall/mock urlopen

[mpkrass7]

Essentially I just added a couple of web urls to the unit tests and validated that they work. If they aren't valid urls, they won't be tested. Please don't make me make a unit test to assess if the check for url validity is itself valid.

Notes:

• In one of the unit tests, I don't check the web urls because the unit test checks the name attribute of the resulting colormap. I wasn't sure what the name should be so I left them out of that test
• Hi Kevin

[mpkrass7]

Hmm I'm seeing that some checks are not successful here

[mpkrass7]

Yay I think we're passing all of the tests. Adding exception handling for OSError was the key

[arvkevi]

Let's not add requests as a package dependency, instead we could add it as a dev dependency like this:

[project.optional-dependencies]
dev = [
    "black",
    "pytest",
    "pytest-cov",
    "requests",
]

Also, add it to ci/requirements.txt, since that's what we actually install during CI.

[mpkrass7]

Ahh thought I had removed it from there..

[arvkevi]

LGTM! Thank you @mpkrass7, this was a great find!