Skip to content
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

Add recording smoke test #775

Open
abrichr opened this issue Jun 18, 2024 · 6 comments · May be fixed by #803
Open

Add recording smoke test #775

abrichr opened this issue Jun 18, 2024 · 6 comments · May be fixed by #803
Labels
💎 Bounty enhancement New feature or request

Comments

@abrichr
Copy link
Member

abrichr commented Jun 18, 2024

Feature request

We would like to add an automated test that:

  • calls openadapt.record
  • waits for the recording.started signal to be emitted (fails after e.g. 30s if no such signal is emitted)
  • stops the recording by emitting the stop sequence on the keyboard (re-use openadapt.playback)
  • asserts something about the state of the database (e.g. that the recording has been created) and filesystem (the video and performance plots have been created)

Motivation

Avoid regressions like in:

@abrichr abrichr added the enhancement New feature or request label Jun 18, 2024
@abrichr
Copy link
Member Author

abrichr commented Jun 18, 2024

/bounty $100

Copy link

algora-pbc bot commented Jun 18, 2024

💎 $100 bounty • OpenAdaptAI

Steps to solve:

  1. Start working: Comment /attempt #775 with your implementation plan
  2. Submit work: Create a pull request including /claim #775 in the PR body to claim the bounty
  3. Receive payment: 100% of the bounty is received 2-5 days post-reward. Make sure you are eligible for payouts

Thank you for contributing to OpenAdaptAI/OpenAdapt!

Add a bountyShare on socials

Attempt Started (GMT+0) Solution
🟢 @onyedikachi-david Jun 22, 2024, 8:59:18 AM #803

@onyedikachi-david
Copy link

onyedikachi-david commented Jun 22, 2024

/attempt #775

Algora profile Completed bounties Tech Active attempts Options
@onyedikachi-david 2 bounties from 1 project
JavaScript, Shell
﹟764
Cancel attempt

@unworld11
Copy link

@abrichr can i get assigned?

@abrichr
Copy link
Member Author

abrichr commented Jun 24, 2024

@unworld11 feel free to submit an attempt, we will merge whoever submits working code that meets the required standard first.

Copy link

algora-pbc bot commented Jun 25, 2024

💡 @onyedikachi-david submitted a pull request that claims the bounty. You can visit your bounty board to reward.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💎 Bounty enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants