rename test files

[philippemiron]

Fix #511 for me.

I'm using uv to create the environment (uv sync --extra all) and my vscode/settings.json is:

{
    "python.defaultInterpreterPath": "./.venv/bin/python",
    "python.testing.pytestArgs": [
        "tests"
    ],
    "python.testing.unittestEnabled": false,
    "python.testing.pytestEnabled": true,
    "editor.formatOnSave": true,
    "[python]": {
        "editor.defaultFormatter": "charliermarsh.ruff",
    }
}

Hopefully, it works with conda too, but someone should confirmed.

[selipot]

What uv? A new environment management solution?

[selipot]

I just had another look: I have tests running again in vs code, see screenshot. My json setting file is as below. Can someone remind me if we are using pytest or unittest?

{
    "git.confirmSync": false,
    "git.autofetch": true,
    "terminal.integrated.inheritEnv": false,
    "[python]": {
        "editor.formatOnType": true
    },
    "jupyter.interactiveWindow.creationMode": "perFile",
    "githubPullRequests.pullBranch": "never",
    "notebook.output.scrolling": true,
    "githubPullRequests.fileListLayout": "tree",
    "security.workspace.trust.untrustedFiles": "open",
    "window.autoDetectColorScheme": true,
    "terminal.integrated.enableMultiLinePasteWarning": false,
    "explorer.confirmDelete": false,
    "workbench.colorTheme": "Default Light Modern",
    "python.testing.pytestEnabled": true,
    "interactiveWindow.executeWithShiftEnter": false,
    "jupyter.interactiveWindow.textEditor.executeSelection": true,
    "jupyter.widgetScriptSources": [
        "jsdelivr.com",
        "unpkg.com"
    ]
}

[selipot]

The tests that failed in my case were re-run and passed after regenerating my local andro and glad datasets

[selipot]

@philippemiron this seems to run in VS Code for me, see screenshot. What is this setting doing?

    "python.testing.pytestArgs": [
        "tests"
    ],

[philippemiron]

It just specify the folders of the tests (it's running pytest tests/), which is tests/ by default so shouldn't make a difference.

@philippemiron this seems to run in VS Code for me, see screenshot. What is this setting doing?

    "python.testing.pytestArgs": [
        "tests"
    ],

[philippemiron]

What uv? A new environment management solution?

It is! It has many advantages. Would be easier to zoom to talk about it if you want more info. It's from the same people at Astral that built ruff.

https://github.com/astral-sh/uv

[selipot]

This looks good to me. Not sure wht the Lint/mypy test is failing?

[Copilot]

Pull Request Overview

This PR standardizes test file naming to the *_test.py pattern and aligns code/tests with the change, while also broadening type support.

• Update test filename globs in CONTRIBUTING.md
• Adjust existing tests to match new naming and suppress a type-checker warning
• Enhance unpack type hints to accept both np.ndarray and xarray.DataArray

Reviewed Changes

Copilot reviewed 20 out of 20 changed files in this pull request and generated 1 comment.

File	Description
tests/plotting_test.py	Suppress type checker warning on cartopy.crs import with # type: ignore
tests/kinematics_test.py	Convert boolean arrays to integer flags to match new test expectations
clouddrift/ragged.py	Extend unpack signature to support xarray.DataArray in parameters and return
CONTRIBUTING.md	Update pytest/coverage globs to use *_test.py naming

Comments suppressed due to low confidence (2)

clouddrift/ragged.py:792

• The default of None for rows isn’t covered by the current type annotation. Consider using rows: int | Iterable[int] | None or Optional[int | Iterable[int]].

rows: int | Iterable[int] = None,

tests/kinematics_test.py:27

• Replacing boolean literals with 1/0 may introduce unintended type mismatches when downstream code expects booleans. Consider using True/False or explicitly converting the array dtype.

[1, 1, 1, 0, 0],

[Copilot]

[nitpick] Adding a brief comment explaining why the type check is ignored here could help future maintainers understand the suppressed warning.

Suggested change

	import cartopy.crs as ccrs # type: ignore
	import cartopy.crs as ccrs # type: ignore # cartopy does not provide complete type stubs

[philippemiron]

There were exceptions in pyproject.toml to make mypy pass. I forgot to rename them.

[selipot]

@KevinShuman Shall we merge?