Remove test_data_kind_fails because it's already covered by validate_data_input doctests

Author: seisman

pygmt/helpers/utils.py

@@ -79,6 +79,10 @@ def validate_data_input(
     Traceback (most recent call last):
         ...
     pygmt.exceptions.GMTInvalidInput: Too much data. Use either data or x/y/z.
+    >>> validate_data_input(data="infile", x=[1, 2, 3], y=[4, 5, 6])
+    Traceback (most recent call last):
+        ...
+    pygmt.exceptions.GMTInvalidInput: Too much data. Use either data or x/y/z.
     >>> validate_data_input(data="infile", z=[7, 8, 9])
     Traceback (most recent call last):
         ...

pygmt/tests/test_helpers.py

@@ -4,18 +4,11 @@
 
 from pathlib import Path
 
-import numpy as np
 import pytest
 import xarray as xr
 from pygmt import Figure
 from pygmt.exceptions import GMTInvalidInput
-from pygmt.helpers import (
-    GMTTempFile,
-    args_in_kwargs,
-    data_kind,
-    kwargs_to_strings,
-    unique_name,
-)
+from pygmt.helpers import GMTTempFile, args_in_kwargs, kwargs_to_strings, unique_name
 from pygmt.helpers.testing import load_static_earth_relief, skip_if_no
 
 
@@ -32,25 +25,6 @@ def test_load_static_earth_relief():
     assert isinstance(data, xr.DataArray)
 
 
-@pytest.mark.parametrize(
-    ("data", "x", "y"),
-    [
-        (None, None, None),
-        ("data.txt", np.array([1, 2]), np.array([4, 5])),
-        ("data.txt", np.array([1, 2]), None),
-        ("data.txt", None, np.array([4, 5])),
-        (None, np.array([1, 2]), None),
-        (None, None, np.array([4, 5])),
-    ],
-)
-def test_data_kind_fails(data, x, y):
-    """
-    Make sure data_kind raises exceptions when it should.
-    """
-    with pytest.raises(GMTInvalidInput):
-        data_kind(data=data, x=x, y=y)
-
-
 def test_unique_name():
     """
     Make sure the names are really unique.