fix(wren-ui/e2e): Update E2E test for navigation change

[ongdisheng]

Description

This issue was identified as a navigation change introduced in PR #843, which redirects to the Modeling page after onboarding. Fix #922.

Screenshots

• PostgreSQL
  

• Trino
  

[ongdisheng]

Hi @fredalai, could you review my PR when you have a chance? Thanks!

[fredalai]

Hi @ongdisheng,

Thank you for your contribution. 😄

Would you like to fix the connectSampleECommerce.spec.ts and connectSampleHR.spec.ts for the navigation change issue?
Also, no worry, I know PR #855 breaks the E2E tests for the Sample dataset, just ignore this PR and we'll fix subsequent tests in a new PR.

[ongdisheng]

Sure, no problem 😄.

[ongdisheng]

Hi @ongdisheng,

Thank you for your contribution. 😄

Would you like to fix the connectSampleECommerce.spec.ts and connectSampleHR.spec.ts for the navigation change issue? Also, no worry, I know PR #855 breaks the E2E tests for the Sample dataset, just ignore this PR and we'll fix subsequent tests in a new PR.

Hi @fredalai, I’ve updated connectSampleHR.spec.ts. Could you take a look if it’s okay when you’re free before I proceed with connectSampleECommerce.spec.ts.

Thanks a lot.

[fredalai]

Hi @ongdisheng,
Thank you for your contribution. 😄
Would you like to fix the connectSampleECommerce.spec.ts and connectSampleHR.spec.ts for the navigation change issue? Also, no worry, I know PR #855 breaks the E2E tests for the Sample dataset, just ignore this PR and we'll fix subsequent tests in a new PR.

Hi @fredalai, I’ve updated connectSampleHR.spec.ts. Could you take a look if it’s okay when you’re free before I proceed with connectSampleECommerce.spec.ts.

Thanks a lot.

Hi @ongdisheng,

Thanks~ LGTM 😄

[ongdisheng]

Hi @fredalai, I’ve also updated connectSampleECommerce.spec.ts. Let me know if further changes are needed.
Thanks!

[fredalai]

LGTM. :D