Skip to content

app_record: Add tests for Record and RECORDING_INFO. #86

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

InterLinked1
Copy link
Contributor

No description provided.

Copy link

Attention! This pull request may contain issues that could prevent it from being accepted. Please review the checklist below and take the recommended action. If you believe any of these are not applicable, just add a comment and let us know.

  • The are no cherry-pick-to headers in any comment in this PR. If the PR applies to more than just the branch it was submitted against, please add a comment with one or more cherry-pick-to: <branch> headers or a comment with cherry-pick-to: none to indicate that this PR shouldn't be cherry-picked to any other branch. See the Code Contribution documentation for more information.

Documentation:

Copy link

Workflow PRCheck completed successfully

@InterLinked1
Copy link
Contributor Author

This test should not be merged as is. This test passes, but for some reason it hangs after passing until killed after 50 seconds, and I haven't been able to figure out why exactly.

@jcolp
Copy link
Member

jcolp commented Apr 29, 2025

Fixing the channel lock may resolve that.

@jcolp
Copy link
Member

jcolp commented May 6, 2025

What's the new state of this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants