[ZEPPELIN-6365] Add E2E tests about Notebook Repositories (/notebook-repos)

[kmularise]

What is this PR for?

Addition of Notebook Repositories E2E tests for New UI

---

PAGES.WORKSPACE.NOTEBOOK_REPOS

→ src/app/pages/workspace/notebook-repos/notebook-repos.component

PAGES.WORKSPACE.NOTEBOOK_REPOS_ITEM

→ src/app/pages/workspace/notebook-repos/item/item.component

What type of PR is it?

Improvement

Todos

• - add Notebook Repositories E2E test

What is the Jira issue?

ZEPPELIN-6365

How should this be tested?

by E2E test CLI

Screenshots (if appropriate)

Questions:

• Does the license files need to update? No
• Is there breaking changes for older versions? No
• Does this needs documentation? No

[dididy]

https://github.com/apache/zeppelin/actions/runs/18502535395/job/52722968526?pr=5105

dididy@cfdc2dd

Currently, the user logging in during the auth test is not an admin, so access to the notebook-repos is restricted. Therefore, it seems necessary to modify this part as shown above commit.

dididy@19ff367

Also, it seems you need to ensure that the page is ready before calling performLoginIfRequired. You can refer to above commit for that part. Thanks for contributing to the E2E tests!

---

https://github.com/kmularise/zeppelin/blob/ZEPPELIN-6365/conf/shiro.ini.template#L125

[kmularise]

@dididy Thanks for the detailed review and the code examples in your branch! Your suggestions effectively address the test user permissions and page loading issues.

I've studied your code changes and will apply them to my branch.

[dididy]

https://github.com/apache/zeppelin/actions/runs/18749572769/job/53485262597?pr=5105

5971435

It seems that the above changes only added waitForZeppelinReady-related code to notebook-repo-item-display.spec.ts. The same should probably be added to the other four notebook-repo-item-* tests as well as notebook-repos-page-structure.spec.ts. In fact, in the above test, the parts where waitForZeppelinReady was added passed successfully, but the parts where it wasn’t added failed.

---

Please have a look at the following commit.

dididy@19ff367

[dididy]

e2e/models/published-paragraph-page.ts:
isPublishedParagraphVisible, getErrorModalTitle, isDynamicFormsVisible, isResultVisible

e2e/models/published-paragraph-page.util.ts:
testConfirmationModalForNoResultParagraph

It doesn't seem to be used, so it can probably be removed.

[dididy]

It looks like all the e2e tests are passing. Thanks!

[tbonelee]

Thanks, merged into master and branch.