tests(#2406): add test_deck.py

[bruno-portfolio]

Summary

Adds unit tests for the deck module as requested in #2406.

Changes

• 19 tests covering Map initialization, layers, and method availability
• Uses ee_initialize=False to avoid Earth Engine dependency
• Single test class following project conventions

Verification

python -m unittest discover -v tests test_deck.py

All 19 tests pass locally.

---

Continuation of #2575, following the incremental PR approach as by @schwehr.

[schwehr]

I don't think these hasattr and issubclass are providing useful test coverage. Why are they needed?

[bruno-portfolio]

You're right, they're not needed, the other tests already exercise those classes and methods. I'll remove them.

[schwehr]

Please combine all the tests that only have this one line into test_map_init_default_params:

m = deck.Map(ee_initialize=False)

That looks to be test_map_default_center, test_map_default_zoom, test_map_default_height, test_map_default_map_style, test_map_layers_initially_empty

[schwehr]

For future changes, please just do additional commits on top of what is reviewed. That way the changes are more obvious to the reviewer. I will do a squash and merge at the end so extra commits will not appear in the final history.

I think after this one last change, the PR should be good to go.

Thank you!

[schwehr]

Remove the self.assertIsNotNone(m) from here to the end. If for some odd reason deck.Map(...) returns None, the first access of m. will catch that. It will be a slightly messier error message, but that will keep the test code easier to follow as it grows.