mistercrunch commented on PR #34346: URL: https://github.com/apache/superset/pull/34346#issuecomment-3143162238
This is harder than it seemed at first ... Cypress tests have hard-coded ids, somehow this works even with auto-numbers ... Loading order must be deterministic enough to support hundreds of test-runs per day. Loading yaml the ids don't match and we don't have uuid-support on our endpoints, at least not yet. Solution to move forward would be to add uuid-support for a bunch of endpoints, or moving away from Cypress. Both are worthwhile but beyond the scope of what I'm willing to do here. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@superset.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: notifications-unsubscr...@superset.apache.org For additional commands, e-mail: notifications-h...@superset.apache.org
