-
Notifications
You must be signed in to change notification settings - Fork 199
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
[#2436] feat(IT): Add catalogs page to web e2e test #2528
Conversation
9090ba1
to
c107166
Compare
@ch3yne Please attach Web page snapshot image in PR, It can help to better review PR. |
...on-test/src/test/java/com/datastrato/gravitino/integration/test/web/ui/CatalogsPageTest.java
Outdated
Show resolved
Hide resolved
afed77a
to
391e4f3
Compare
baf66b5
to
7a1bc1c
Compare
7a1bc1c
to
82b2478
Compare
|
||
Assertions.assertTrue(catalogsPage.verifyCreateHiveCatalog(catalogName)); | ||
} | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can add a test case testCreateMultipleCatalogs
and check the render table data is correct values when switch different catalog tree nodes
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Because other providers of catalogs require different environments, I will add these tests in the next pull request.
LGTM, Thanks @ch3yne |
…rato#2528) ### What changes were proposed in this pull request? Add catalogs page to web e2e test. - show parent details - create a catalog - view existed catalog details - edit existed catalog - delete existed catalog - link to schemas - link to tables - back to homepage ### Why are the changes needed? Fix: datastrato#2436 ### Does this PR introduce _any_ user-facing change? N/A ### How was this patch tested? N/A
What changes were proposed in this pull request?
Add catalogs page to web e2e test.
Why are the changes needed?
Fix: #2436
Does this PR introduce any user-facing change?
N/A
How was this patch tested?
N/A