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

Update prepareFilesToUpload tests #1417

Merged
merged 4 commits into from
Jan 15, 2025
Merged

Update prepareFilesToUpload tests #1417

merged 4 commits into from
Jan 15, 2025

Conversation

Jona-Internxt
Copy link
Contributor

@Jona-Internxt Jona-Internxt commented Jan 14, 2025

Description

Updated prepareFilesToUpload tests. Ramdomly tests are failing, so I tried to improve the way to obtain the results

Related Issues

This issue is related to this task

Checklist

  • Changes have been tested locally.
  • Unit tests have been written or updated as necessary.
  • The code adheres to the repository's coding standards.
  • Relevant documentation has been added or updated.
  • No new warnings or errors have been introduced.
  • SonarCloud issues have been reviewed and addressed.
  • QA Passed

How Has This Been Tested?

Tests has been launched locally many times.

Additional Notes

When review, if possible, launch the tests locally several times and check if they pass.

@Jona-Internxt Jona-Internxt requested review from rafijv and xabg2 January 14, 2025 11:11
@Jona-Internxt Jona-Internxt self-assigned this Jan 14, 2025
Copy link

vercel bot commented Jan 14, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
drive-web ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 15, 2025 9:35am

@Jona-Internxt Jona-Internxt marked this pull request as draft January 14, 2025 11:20
Copy link

cloudflare-workers-and-pages bot commented Jan 14, 2025

Deploying drive-web with  Cloudflare Pages  Cloudflare Pages

Latest commit: e4c8ef5
Status:⚡️  Build in progress...

View logs

@Jona-Internxt Jona-Internxt marked this pull request as ready for review January 15, 2025 09:38
@CandelR CandelR merged commit d45548d into master Jan 15, 2025
18 of 19 checks passed
@CandelR CandelR deleted the fix/update-tests branch January 15, 2025 12:49
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.

3 participants