MPRester lazily get endpoint and api_key

[DanielYang59]

Summary

• Lazily get endpoint and api_key, to close MP_API_KEY is retrieved at import time #935
• Add more unit tests

[DanielYang59]

@tschaume This one is ready for review, thanks!

Also try to ping @janosh if you are interested in commenting :)

[janosh]

thanks @DanielYang59, looks great! esp. those new tests 👍

[DanielYang59]

The test failure is owing to restricted access to repo secret from a fork:

With the exception of GITHUB_TOKEN, secrets are not passed to the runner when a workflow is triggered from a forked repository.

And is passing locally:

=================================================== test session starts ====================================================
platform darwin -- Python 3.12.6, pytest-8.3.3, pluggy-1.5.0
rootdir: /Users/yang/developer/api
configfile: pyproject.toml
plugins: typeguard-4.3.0
collected 31 items                                                                                                         

tests/test_client.py ...............................                                                                 [100%]

[tschaume]

I think it looks good - I'll try to get it merged and released.

[tschaume]

@DanielYang59 release candidate is out for this 🎉 See https://pypi.org/project/mp-api/0.43.0rc0/

[DanielYang59]

Great! Thanks a lot for reviewing!