fix: add data-testid attributes to key UI elements

[TanviHanda]

Close Issue number #56

Description

What changed

Added data-testid attributes to key UI elements for better automated testing support.

Updated components

• FileUpload
• VideoEditor
• ExportOverlay
• ExportSettings
• DownloadResult
• AudioSpeedControl

Purpose

This improves element targeting and reliability for end-to-end/UI tests.

[vercel]

Someone is attempting to deploy a commit to the magic-peach1's projects Team on Vercel.

A member of the Team first needs to authorize it.

[github-actions]

⚠️ PR Format Issues — @TanviHanda

Please fix the following before your PR can be reviewed:

• ⚠️ No linked issue found. Add Closes #<issue-number> to your PR description.

Push new commits after fixing — this comment will update automatically.

📖 CONTRIBUTING.md

[github-actions]

👋 Thanks for your PR, @TanviHanda!

Welcome to Reframe — a browser-based video editor built for everyone 🎬

🟠 GSSoC'26 PR detected — thanks for contributing under GirlScript Summer of Code 2026!

What happens next

1. 🤖 Automated checks — build & TypeScript typecheck will run automatically
2. ✅ Vercel preview — a preview deployment will be created (requires maintainer authorization for fork PRs)
3. 👀 Code review — a maintainer will review your changes
4. 🚀 Merge — once approved, your PR will be merged!

Quick checklist

• PR title follows Conventional Commits (e.g. feat: add dark mode)
• Linked the issue this PR closes (e.g. Closes #123)
• Tested the changes locally (bun run dev)
• Build passes (bun run build)

Useful links

• 📖 CONTRIBUTING.md
• 🐛 Open Issues
• ⭐ Star the repo to support the project!

Happy coding! 🎉

[magic-peach]

Hey @TanviHanda! Thanks for adding data-testid attributes — these are useful for testing. However, package-lock.json (7k lines) has been committed to this PR. This project uses bun.lock exclusively.

Please remove it:

rm package-lock.json
git add package-lock.json
git commit -m "chore: remove package-lock.json"
git push

Once removed and CI passes, the testid additions will be reviewed!

[magic-peach]

Hey @TanviHanda! Adding data-testid attributes for testing is a useful contribution. This PR has merge conflicts with main. Please rebase:

git fetch origin
git rebase origin/main
git push --force-with-lease

[magic-peach]

Hey @TanviHanda! This PR has merge conflicts with the latest main. Please rebase to resolve them and trigger fresh CI:

git fetch origin
git rebase origin/main
git push --force-with-lease

Once CI (build/lint/typecheck) passes, we'll review and merge!

[magic-peach]

Hey @TanviHanda! This PR looks good in terms of code structure, but the build/lint/typecheck CI hasn't run yet. Please rebase onto the latest main to trigger fresh checks:

git fetch upstream
git rebase upstream/main
git push --force-with-lease

Once CI passes, we'll review and merge!

[vercel]

Deployment failed with the following error:

Resource is limited - try again in 24 hours (more than 100, code: "api-deployments-free-per-day").

Learn More: https://vercel.com/magic-peach1s-projects?upgradeToPro=build-rate-limit

[magic-peach]

Hey @TanviHanda! 👋

We've added a new requirement for all PRs: a screen recording showing your changes working on your local machine must be attached before a PR can be merged.

Please add a recording to this PR that shows:

1. bun run dev running at http://localhost:3000
2. The full working flow of your change (demonstrate the feature/fix end-to-end)
3. Any tests passing — if your change touches logic with tests, show bun run lint and bunx tsc --noEmit completing without errors in the terminal

How to record:

• macOS: Cmd + Shift + 5 → Record Selected Portion, or QuickTime Player
• Windows: Win + G → Xbox Game Bar → Capture
• Linux: OBS Studio, GNOME Screenshot, or kazam
• Any OS: Loom (free, easy to share)

Once you have the recording, drag the file directly into a comment on this PR, or paste a Loom link. This is now a hard requirement — see CONTRIBUTING.md for full details.

Thanks for contributing to Reframe! 🎬

[TanviHanda]

Hey @magic-peach
https://github.com/user-attachments/assets/0d73f794-75fe-4099-9747-3470c05cc63d

Added the screen recording showing the app running locally, the implemented flow working end-to-end, and the required UI elements with data-testid attributes.

[TanviHanda]

Hi @magic-peach , I resolved the merge conflict and pushed the updated branch.

The remaining failed checks seem to be related to Vercel/Claude bot limits or external review/deployment issues. Please let me know if any code changes are needed from my side.

[magic-peach]

Hey @TanviHanda! This PR has merge conflicts with main. Please rebase to resolve them:

git fetch origin
git rebase origin/main
# resolve any conflicts
git push --force-with-lease

Once rebased and CI passes, this can be reviewed for merge.

[magic-peach]

@TanviHanda Adding data-testid attributes to key UI elements is a useful addition for testing infrastructure! Please resolve the merge conflicts with main:

git fetch origin
git rebase origin/main
git push --force-with-lease

Also please fill in the PR description with what specific elements got data-testid attributes and which issue this relates to. Once CI (build/lint/typecheck) passes and the description is filled in, this can be merged!

[TanviHanda]

Hi @magic-peach
Conflicts have been resolved and the PR description has been updated as requested.

The remaining failing check is Vercel deployment authorization, which requires maintainer/team permissions.

[magic-peach]

Hi @TanviHanda — thanks for this, and sorry for the wait.

I'd like to get this merged, but it replaces a cn() call with a template literal, which loses Tailwind class-conflict resolution.

To get it unblocked:

1. restore the cn() wrapper on those className props

Could you take a look within the next 14 days? If I don't hear back by then I'll close it to keep the queue manageable — but you're very welcome to reopen it any time after that. Thanks again for contributing to Reframe 🙏

[TanviHanda]

Hi @magic-peach
Okay, I’ll take a look and make the changes within 14 days. Thanks