Skip to content

test_common.py: Add test_download_ee_image_tiles_parallel#2602

Draft
schwehr wants to merge 5 commits into
gee-community:masterfrom
schwehr:test-download_ee_image_tiles_parallel
Draft

test_common.py: Add test_download_ee_image_tiles_parallel#2602
schwehr wants to merge 5 commits into
gee-community:masterfrom
schwehr:test-download_ee_image_tiles_parallel

Conversation

@schwehr
Copy link
Copy Markdown
Collaborator

@schwehr schwehr commented Feb 12, 2026

Test that geemap can call joblib

Copy link
Copy Markdown
Member

@jdbcode jdbcode left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For all of the default arguments can ANY (from unittest.mock import ANY) be used?

@jdbcode jdbcode assigned schwehr and unassigned jdbcode Feb 13, 2026
@schwehr
Copy link
Copy Markdown
Collaborator Author

schwehr commented Feb 13, 2026

For all of the default arguments can ANY (from unittest.mock import ANY) be used?

I've not had much experience with ANY. I think I'll keep it as is, but if these types of tests are demonstrateably brittle, we should definitely consider it.

@schwehr
Copy link
Copy Markdown
Collaborator Author

schwehr commented Feb 13, 2026

I need to figure out what is up with the docs-build failure:

======================================================================
FAIL: test_download_ee_image_tiles_parallel (test_common.CommonTest.test_download_ee_image_tiles_parallel)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/runner/work/_temp/uv-python-dir/cpython-3.12.12-linux-x86_64-gnu/lib/python3.12/unittest/mock.py", line 1396, in patched
    return func(*newargs, **newkeywargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/runner/work/geemap/geemap/tests/test_common.py", line 631, in test_download_ee_image_tiles_parallel
    self.assertEqual(mock_download_ee_image.call_count, 2)
AssertionError: 0 != 2

@schwehr
Copy link
Copy Markdown
Collaborator Author

schwehr commented Feb 14, 2026

I can't figure out why this works locally for me, but not on github.

@schwehr schwehr marked this pull request as draft February 14, 2026 16:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants