-
Notifications
You must be signed in to change notification settings - Fork 43
Open
Description
We didn't have python integration tests when we originally added aliases. We should add some sort of tests to ensure python can call them.
At the moment, we test only one version. In this case below, we actually test the alias and not the original function ST_GeomFromWKT anywhere.
| def test_st_geomfromtext(eng, wkt, expected): |
Options I see:
- Parametrize the existing test functions so all aliases are called for the entire test suite of that function
@pytest.mark.parametrize(func_name, ["ST_GeomFromText", "ST_GeomFromWKT"])
...
def test_st_geomfromwkt()- Add some separate tests, just make sure the aliases are working
def test_st_geomfromtext_alias()
# < a single query > Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels