grass.jupyter: add test for utils.py functions#7070
grass.jupyter: add test for utils.py functions#7070ayushjariyal wants to merge 5 commits intoOSGeo:mainfrom
Conversation
github-actions
bot
added
Python
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
|
@gulshan-123, thanks for the suggestion. I’ve updated the code accordingly in this PR and and verified it locally by running pytest from a normal Linux terminal(Not from inside GRASS teminal and not from inside docker container). All tests are now passing. 
Can you please review this? |
echoix
changed the title
|
@petrasovaa , The CI is currently failing due to lint errors in super linters. I verify it locally and everything pass successfully on my system. |
|
@petrasovaa, Could you please review this PR? |
| IPython = pytest.importorskip("IPython", reason="IPython package not available") | ||
| ipywidgets = pytest.importorskip( | ||
| "ipywidgets", reason="ipywidgets package not available" | ||
| ) |
There was a problem hiding this comment.
@petrasovaa, they are not strictly needed for these tests. I added them to maintain consistency with other test files and for future work.
But for now I will remove them to keep test minimal
petrasovaa
left a comment
petrasovaa
left a comment
There was a problem hiding this comment.
You have typos in docstrings.
ayushjariyal
force-pushed
the
tests_utils
branch
from
07d0185 to
f8bb059
Compare
2 participants
This PR add tests for
get_regionandget_location_proj_string.In this PR I initialize GRASS session using existing fixtures and avoid relying of GRASS environment to ensure they run correctly outside GRASS shell and CI.
Tests were verify locally in clean Docker image outside GRASS shell.