test(convert): add tests for convert_to_json

[GonzaFP]

Contributor checklist

• This pull request is on a separate branch and not the main branch
• I have tested my code with the pytest command as directed in the testing section of the contributing guide

---

Description

Added tests for convert_to_json in test/cli/convert/wrapper.py to improve test coverage.

Related issue

• [Multiple assignees possible] Add missing tests to Scribe-Data #623

[andrewtavis]

Thanks for the PR, @GonzaFP! We'll try to get to the review soon :)

[andrewtavis]

Quick check in here, @GonzaFP 👋 Do we need to add all of these tests, or are these functionalities being covered by src/scribe_data/cli/convert/to_json.py? Just checking as by the looks of this this covers one extra line of code, and 120 lines of tests for covering one line of code is a bit much. Feel free to simplify them down to just calling the JSON conversion from the wrapper! Looks like we also need a similar test that we can do convert_to_csv_or_tsv from the wrapper and then that file would be 100%.

Happy to keep these changes if they're needed, but it'd be great if you could verify if some of the tests are repeated :)

[GonzaFP]

Thanks @andrewtavis for your response.
In wrapper.py, there are two conditions at the start of the function convert_wrapper that check if the output_dir is None and if input_path is None. I wrote two tests to test the conditions.
The convert_to_json function takes a language parameter of type string so I tested that too.

Should I combine all three tests into one?
PS: The prior tests of convert_to_sqlite served as an inspiration. I see that separate tests were written for each scenario.
I will work on convert_to_csv_or_tsv after convert_to_json is finished.