Closed JackCollins91 closed 3 weeks ago
Hi @GalenReich, PR is updated per your request. I also updated the initial comment to reflect changes. The attached .txt file shows the results of the latest testing. I added some new test cases which are detailed there also.
Closes Issue #22
edgar-tool now supports parameters
inc_in
for 'incorporated_in' andpeo_in
for 'principal executive offices in.'both parameters takes a string like "NY" or "NY, OH." Any country code in
edgar_tool.constants.TEXT_SEARCH_LOCATIONS_MAPPING
is supported.Also, the country names as per TEXT_SEARCH_LOCATIONS_MAPPING.values() are accepted. Whitespace and upper/lower casing should be ignored, and so therefore, "mexico,ny" and "MexicO, Ny" will both convert to "NY,O5"
Malformed input will result in the dictionary for
edgar_tool.constants.TEXT_SEARCH_LOCATIONS_MAPPING
being printed in the console so the user can see what codes they can use.I tested the following commands for both correct, incorrect, and unrelated use cases, I have pasted the results of that run in the attached file.
test cli session.txt