Skip to content

Commit 6bace16

Browse files
authored
SAT-47603 - Add missing ouiaId labels to components (#1238)
1 parent 73d36a0 commit 6bace16

7 files changed

Lines changed: 9 additions & 3 deletions

File tree

.eslintrc

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,11 @@
11
{
2-
"plugins": ["@theforeman/foreman", "spellcheck"],
2+
"plugins": ["@theforeman/foreman", "@theforeman/rules", "spellcheck"],
33
"extends": [
44
"plugin:@theforeman/foreman/core",
55
"plugin:@theforeman/foreman/plugins"
66
],
77
"rules": {
8+
"@theforeman/rules/require-ouiaid": "error",
89
"spellcheck/spell-checker": [
910
"error",
1011
{

webpack/ForemanInventoryUpload/Components/InventorySettings/MinimalInventoryDropdown.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -79,6 +79,7 @@ const MinimalInventoryDropdown = ({ setChosenValue }) => {
7979
isFullWidth
8080
onClick={onToggleClick}
8181
isExpanded={isOpen}
82+
ouiaId="inventory-dropdown-toggle"
8283
>
8384
{dropdownValues[currentDropdownValue].title}
8485
</MenuToggle>

webpack/ForemanInventoryUpload/Components/PageHeader/PageTitle.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -59,6 +59,7 @@ const PageTitle = () => {
5959
variant="plain"
6060
onClick={() => setIsDropdownOpen(prev => !prev)}
6161
isExpanded={isDropdownOpen}
62+
ouiaId="title-dropdown-toggle"
6263
>
6364
<EllipsisVIcon />
6465
</MenuToggle>

webpack/InsightsCloudSync/Components/InsightsTable/InsightsTableConstants.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ export const actionsFormatter = rowData => {
3131
recommendationUrl &&
3232
!isLocalAdvisorEngine &&
3333
dropdownItems.push(
34-
<DropdownItem key="recommendation-url">
34+
<DropdownItem key="recommendation-url" ouiaId="insights-dropdown-item-recommendation-url">
3535
<a href={recommendationUrl} target="_blank" rel="noopener noreferrer">
3636
{__('View in Red Hat Insights')} <ExternalLinkAltIcon />
3737
</a>
@@ -40,7 +40,7 @@ export const actionsFormatter = rowData => {
4040

4141
accessRHUrl &&
4242
dropdownItems.push(
43-
<DropdownItem key="access-url">
43+
<DropdownItem key="access-url" ouiaId="insights-dropdown-item-knowledgebase">
4444
<a href={accessRHUrl} target="_blank" rel="noopener noreferrer">
4545
{__('Knowledgebase article')} <ExternalLinkAltIcon />
4646
</a>

webpack/InsightsCloudSync/Components/InsightsTable/Pagination.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -42,6 +42,7 @@ const Pagination = ({ variant, ...props }) => {
4242

4343
return (
4444
<PfPagination
45+
ouiaId={`recommendations-pagination-${variant}`}
4546
itemCount={itemCount}
4647
widgetId={`recommendation-pagination-${variant}`}
4748
perPage={perPage}

webpack/InsightsCloudSync/Components/ToolbarDropdown.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -53,6 +53,7 @@ const ToolbarDropdown = ({ onRecommendationSync }) => {
5353
aria-label={__('Recommendations actions')}
5454
onClick={() => setIsDropdownOpen(prev => !prev)}
5555
isExpanded={isDropdownOpen}
56+
ouiaId="recommendations-dropdown-toggle"
5657
>
5758
<EllipsisVIcon />
5859
</MenuToggle>

webpack/common/DropdownToggle.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,7 @@ const DropdownToggle = ({ items, ...props }) => {
1818
aria-label={__('Table actions')}
1919
onClick={() => setOpen(prev => !prev)}
2020
isExpanded={isOpen}
21+
ouiaId="toggle-dropdown-toggle"
2122
>
2223
<EllipsisVIcon />
2324
</MenuToggle>

0 commit comments

Comments
 (0)