Dciborow/reduce test cases

[dciborow]

Description

Testing

Additional context

[github-actions]

Summary by GPT-4

The changes in this commit remove some unused code and update the test functions to use the correct arguments. Specifically, it removes the MockStorageContext class and its usage, as well as the MockRepoReader class and its related functions. It also updates the test_cli_gpt_cli and test_cli_gpt_module functions to pass the correct argument to the cli_test function.

Suggestions

Here are some suggestions for improving the changes in this PR:

1. Remove unused imports and classes:

In tests/conftest.py, you have removed the usage of SimpleDirectoryReader, MockStorageContext, MockRepoReader, and mock_load_data_from_branch. However, you have not removed their import statements and class definitions. To clean up the code, remove these unused imports and classes.

2. Update test cases to reflect changes:

Since you have made changes to the code, it is important to update the test cases accordingly. Make sure that all test cases are still valid and cover the new changes made in this PR.

3. Add comments explaining the changes:

It is always a good practice to add comments explaining why certain changes were made in a PR. This helps other developers understand the context behind those changes and makes it easier for them to review your code.

Overall, these suggestions should help improve the quality of your PR and make it easier for others to review your code.