Skip to content

fix: resolve pytest warnings in test_git_tools.py#196

Closed
ezyang wants to merge 2 commits into
gh/ezyang/148/basefrom
gh/ezyang/148/head
Closed

fix: resolve pytest warnings in test_git_tools.py#196
ezyang wants to merge 2 commits into
gh/ezyang/148/basefrom
gh/ezyang/148/head

Conversation

@ezyang
Copy link
Copy Markdown
Owner

@ezyang ezyang commented Mar 24, 2025

Stack from ghstack (oldest at bottom):

Fix these pytest warnings.

e57db7c  (Base revision)
4e3d96d  Remove @pytest.mark.asyncio decorator and convert asyncSetUp to sync setUp
2959325  Add asyncio import
31c6249  Implement setUp_async method using asyncio.run
ae6b763  Convert asyncTearDown to standard tearDown
6093c63  Fix async_setup to include git init command
2b14e4b  Convert test_git_log to use asyncio.run with inner async function
fe9708f  Convert test_git_diff to use asyncio.run with inner async function
a83cf90  Convert test_git_show to use asyncio.run with inner async function
ceac4c3  Convert test_git_blame to use asyncio.run with inner async function
f177a40  Convert test_invalid_path to use asyncio.run with inner async function
65d5b29  Convert test_command_failure to use asyncio.run with inner async function
60d29cd  Fix mocking in test_invalid_path to use correct import paths
5bce016  Auto-commit format changes
HEAD     Auto-commit lint changes

codemcp-id: 206-fix-resolve-pytest-warnings-in-test-git-tools-py

[ghstack-poisoned]
ezyang added a commit that referenced this pull request Mar 24, 2025
Fix these pytest warnings.

```git-revs
e57db7c  (Base revision)
4e3d96d  Remove pytest.mark.asyncio decorator and convert asyncSetUp to sync setUp
2959325  Add asyncio import
31c6249  Implement setUp_async method using asyncio.run
ae6b763  Convert asyncTearDown to standard tearDown
6093c63  Fix async_setup to include git init command
2b14e4b  Convert test_git_log to use asyncio.run with inner async function
fe9708f  Convert test_git_diff to use asyncio.run with inner async function
a83cf90  Convert test_git_show to use asyncio.run with inner async function
ceac4c3  Convert test_git_blame to use asyncio.run with inner async function
f177a40  Convert test_invalid_path to use asyncio.run with inner async function
65d5b29  Convert test_command_failure to use asyncio.run with inner async function
60d29cd  Fix mocking in test_invalid_path to use correct import paths
5bce016  Auto-commit format changes
HEAD     Auto-commit lint changes
```

codemcp-id: 206-fix-resolve-pytest-warnings-in-test-git-tools-py
ghstack-source-id: 08fde94
Pull-Request-resolved: #196
[ghstack-poisoned]
ezyang added a commit that referenced this pull request Mar 24, 2025
Fix these pytest warnings.

```git-revs
e57db7c  (Base revision)
4e3d96d  Remove pytest.mark.asyncio decorator and convert asyncSetUp to sync setUp
2959325  Add asyncio import
31c6249  Implement setUp_async method using asyncio.run
ae6b763  Convert asyncTearDown to standard tearDown
6093c63  Fix async_setup to include git init command
2b14e4b  Convert test_git_log to use asyncio.run with inner async function
fe9708f  Convert test_git_diff to use asyncio.run with inner async function
a83cf90  Convert test_git_show to use asyncio.run with inner async function
ceac4c3  Convert test_git_blame to use asyncio.run with inner async function
f177a40  Convert test_invalid_path to use asyncio.run with inner async function
65d5b29  Convert test_command_failure to use asyncio.run with inner async function
60d29cd  Fix mocking in test_invalid_path to use correct import paths
5bce016  Auto-commit format changes
742efb7  Auto-commit lint changes
7bfe13a  Completely rewrite test_git_tools.py to use MCPEndToEndTestCase
36e047a  Fix test_git_blame to use the correct author name set by MCPEndToEndTestCase
HEAD     Auto-commit format changes
```

codemcp-id: 206-fix-resolve-pytest-warnings-in-test-git-tools-py
ghstack-source-id: 1b5724d
Pull-Request-resolved: #196
@ezyang ezyang closed this in 3cd1000 Mar 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant