test(docs-screenshots): add tenant, scope, permission and role screenshots - #174
Merged
Conversation
…shots Extends the documentation screenshot generator with the four remaining entities that didn't have a 'How To' page yet: Tenant, Scope, Permission and Role. Each gets a list view and a filled-in create-form capture, plus expanded-card captures for scopes/permissions/roles whose list view has meaningful nested detail. Generated PNGs are copied by hand into versola-website/public/img/docs/entities/ per the existing convention documented at the top of this file.
goshacodes
marked this pull request as ready for review
August 15, 2026 15:13
Contributor
🤖 Augment PR SummarySummary: Extends the documentation screenshot generator for the remaining entity guides. Changes:
Technical Notes: Screenshots continue to be written to 🤖 Was this summary useful? React with 👍 or 👎 |
![augmentcode[bot]](https://avatars.githubusercontent.com/in/1027498?s=60&v=4){kind=small}
| await loadAdminApp(page, { path: `/?view=tenants&tenant=${tenant}`, state }); | ||
|
|
||
| await expect(page.getByRole('heading', { name: 'Tenants', exact: true })).toBeVisible(); | ||
| await shot(page, 'tenants-list'); |
Contributor
There was a problem hiding this comment.
content-header is rendered while the tenant list is still loading, so this capture can run before any tenant cards arrive and save skeleton cards rather than the intended documented list. The identical race affects the initial captures at central-ui/tests/docs-screenshots.spec.ts:269 and central-ui/tests/docs-screenshots.spec.ts:297.
Severity: medium
Other Locations
central-ui/tests/docs-screenshots.spec.ts:269central-ui/tests/docs-screenshots.spec.ts:297
🤖 Was this useful? React with 👍 or 👎, or 🚀 if it prevented an incident/outage.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Extends the documentation screenshot generator (
central-ui/tests/docs-screenshots.spec.ts) with the four entities that were missing a "How To" page: Tenant, Scope, Permission and Role.Each gets:
Generated PNGs are copied by hand into
versola-website/public/img/docs/entities/, per the existing convention documented at the top of the spec file. Companion PR on the website side: versolauth/versola-website#.All 15 tests in the file pass (7 pre-existing + 8 new).
Pull Request opened by Augment Code | View session