automated tests #503
automated tests #503
Conversation
… each potential values of the report_type argument) - 6 reports run to completion and are printed in CLI
| docx_filename = os.path.join(output_dir, f"{report_type}.docx") | ||
|
|
||
| # Check if the PDF and DOCX files are created | ||
| # assert os.path.exists(pdf_filename), f"PDF file not found for report type: {report_type}" |
There was a problem hiding this comment.
@ElishaKay why is this commented? Is this test final?
There was a problem hiding this comment.
@assafelovic Current Tests dont write the report to docs or pdf.
Probably because they're based on the code samples from the pip documentation which merely return a string of the report
There was a problem hiding this comment.
For those 6 use cases in the commit message above, correct, the tests are final and pass.
I needed to make a change in agent.py for those tests to pass (initialize report = "" at top of a function. If that doesn't break anything feel free to merge. Else we can continue writing tests for more use cases here.
Food for thought:
- What are the potential parameters and values that can be passed to the GPTResearcher Class?
- For end to end tests (fronted to backend and back), we'd be better positioned leveraging a React Testing Library
There was a problem hiding this comment.
Just got me thinking that GPT Researcher should be able to expose the writing functions
Run tests with:
pytest -s test_gpt_researcher.py