I think the "column" parameter needs to be added to the "r.to.vect" module call, but the test failed after adding it with the error below.

r.to.vect input=a_1@PERMANENT type=area column=values output=test_2001_01@PERMANENT --o --q
DBMI-SQLite driver error:
Error in sqlite3_prepare():
near "values": syntax error

The test passed after I changed the column values in test_to_vect.py to "value". But I'm unsure about modifying unittest code, so I need some advice.

"values" is a reserved sqlite keyword so when you fixed the tool, the specified column "values" started to be used, hence the error. Given there is a default specified, I would simply remove the column parameter from the test.

It is also possible to remove line 166 theoretically, but I didn't do so since Flake8 didn't complain about F841. Line 166 returns None or an object, so it could be used to determine whether the process is continuing or terminating.

    # Check the new stvds
    new_sp = tgis.check_new_stds(output, "stvds", dbif=dbif, overwrite=overwrite)

I would keep the call, but remove the return value, the check function should throw an error.

Should there be an update in the .flake8 file as well?