Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Migration of localdb tests to async #5143

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 18 commits into from
Apr 10, 2025
Merged

Migration of localdb tests to async #5143

merged 18 commits into from
Apr 10, 2025

Conversation

gblaih
Copy link
Contributor

@gblaih gblaih commented Mar 20, 2025

Continued from #5011

@gblaih gblaih added the test label Mar 20, 2025
@netlify
Copy link

netlify bot commented Mar 20, 2025 •
edited
Loading

✅ Deploy Preview for cbioportalfrontend ready!

Name Link
🔨 Latest commit fbbe1a9
🔍 Latest deploy log https://app.netlify.com/sites/cbioportalfrontend/deploys/67f44aa129bada0008506f41
😎 Deploy Preview https://deploy-preview-5143.preview.cbioportal.org
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@gblaih gblaih force-pushed the localdb-async-migration branch 3 times, most recently from bd78719 to ce02d95 Compare April 3, 2025 20:51
@dippindots dippindots requested a review from alisman April 9, 2025 14:43
@gblaih gblaih force-pushed the localdb-async-migration branch 2 times, most recently from 8903d68 to 54afc34 Compare April 9, 2025 14:59
bemijonathan and others added 16 commits April 9, 2025 14:19
@bemijonathan
local git changes (cBioPortal#4)
8981ddc 
complete the migration of local tests from sync to async

* fix tolerance value
@bemijonathan
fix await keycloak
5204311
@bemijonathan
remove .only
6e4165c
fix async errors
02b09a6
fix issues with executeAsync calls
1fad3f3
fix and stabilize virtual study, namespace, and survival chart tests
fa84bb8
additional async fixes to namespace tests and utils
5c8cdea
fix remote spec affected by async utils changes
502c27b
fix checkbox async issues
889d5f9
additional test fixes
4fba4b4
update screenshot and namespace test
bf24bdf
stabilize genomic evolution test, remove onlys and not async utils re…
753f76a 
…ferences
stabilize breadcrumbs test and fix minor issues
8083c6c
update breadcrumbs screenshot
92761ac
stabilize breadcrumbs further by removing xy chart prior to test
9c3a922
stabilize reset colors on oncoprint test
3b4a6e6
@gblaih gblaih force-pushed the localdb-async-migration branch 2 times, most recently from b0e028d to 90be973 Compare April 9, 2025 19:44
@gblaih gblaih force-pushed the localdb-async-migration branch from 90be973 to 9f45a11 Compare April 9, 2025 19:47
@alisman alisman merged commit 6296770 into cBioPortal:master Apr 10, 2025
13 of 16 checks passed
@gblaih gblaih mentioned this pull request Apr 10, 2025
Closed
3 tasks
@gblaih gblaih deleted the localdb-async-migration branch April 10, 2025 15:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@alisman alisman Awaiting requested review from alisman

Assignees

@gblaih gblaih

Labels

test

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@gblaih @alisman @bemijonathan