Skip to content

Conversation

Bibo-Joshi
Copy link
Member

Trying to get things for #4271 done in smaller chunks

@Bibo-Joshi Bibo-Joshi added the ⚙️ tests affected functionality: tests label Sep 19, 2024
@Bibo-Joshi Bibo-Joshi removed the request for review from harshil21 September 19, 2024 19:34
@Bibo-Joshi Bibo-Joshi marked this pull request as draft September 19, 2024 19:35
@Bibo-Joshi Bibo-Joshi marked this pull request as ready for review September 19, 2024 20:08
Copy link
Member

@harshil21 harshil21 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

well, it is my own commit so I have to approve it 😄

@@ -65,7 +65,7 @@ async def animated_sticker_set(bot):


@pytest.fixture(scope="session")
async def audio(bot, chat_id, aiolib):
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

these were from when I was testing a library similar to pytest-asyncio-cooperative, i.e. running the tests themselves concurrently

@Bibo-Joshi Bibo-Joshi merged commit 28d19c3 into master Sep 21, 2024
25 checks passed
@Bibo-Joshi Bibo-Joshi deleted the move-files-conftest branch September 21, 2024 09:33
@github-actions github-actions bot locked and limited conversation to collaborators Sep 29, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
⚙️ tests affected functionality: tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants