Merge pull request #10 from patternfly/os-9347

merge OS PR 9347
patternfly · Jul 2, 2021 · b0604be · b0604be
2 parents f736d6a + 580e984
commit b0604be
Show file tree

Hide file tree

Showing 7 changed files with 16 additions and 7 deletions.
diff --git a/packages/dev/package.json b/packages/dev/package.json
@@ -9,7 +9,7 @@
     "serve": "serve public"
   },
   "dependencies": {
-    "@patternfly/quickstarts": "1.0.0-rc.27",
+    "@patternfly/quickstarts": "1.0.0-rc.28",
     "@patternfly/react-core": "^4.101.3",
     "asciidoctor": "^2.2.1",
     "react": "^16.14.0",

diff --git a/packages/module/package.json b/packages/module/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@patternfly/quickstarts",
-  "version": "1.0.0-rc.27",
+  "version": "1.0.0-rc.28",
   "description": "PatternFly quick starts",
   "files": [
     "dist"

diff --git a/packages/module/src/QuickStartCatalogPage.tsx b/packages/module/src/QuickStartCatalogPage.tsx
@@ -38,7 +38,7 @@ export const QuickStartCatalogEmptyState = ({ clearFilters }) => {
         {getResource('No results match the filter criteria. Remove filters or clear all filters to show results.')}
       </EmptyStateBody>
       <EmptyStatePrimary>
-        <Button variant="link" onClick={clearFilters}>
+        <Button variant="link" onClick={clearFilters} data-test="clear-filter button">
           {getResource('Clear all filters')}
         </Button>
       </EmptyStatePrimary>
@@ -119,7 +119,7 @@ export const QuickStartCatalogPage: React.FC<QuickStartCatalogPageProps> = ({
     <>
       {showTitle && (
         <div className="ocs-page-layout__header">
-          <Text component="h1" className="ocs-page-layout__title">
+          <Text component="h1" className="ocs-page-layout__title" data-test="page-title">
             {title || getResource('Quick Starts')}
           </Text>
           {hint && <div className="ocs-page-layout__hint">{hint}</div>}

diff --git a/packages/module/src/QuickStartPanelContent.tsx b/packages/module/src/QuickStartPanelContent.tsx
@@ -77,6 +77,7 @@ const QuickStartPanelContent: React.FC<QuickStartPanelContentProps> = ({
       className="co-quick-start-panel-content"
       data-testid={`qs-drawer-${camelize(quickStart.spec.displayName)}`}
       data-qs={`qs-step-${getStep()}`}
+      data-test="quickstart drawer"
       {...props}
     >
       <div className={`co-quick-start-panel-content-head ${headerClasses}`}>
@@ -100,7 +101,7 @@ const QuickStartPanelContent: React.FC<QuickStartPanelContentProps> = ({
           )}
         </DrawerHead>
       </div>
-      <DrawerPanelBody hasNoPadding className="co-quick-start-panel-content__body">
+      <DrawerPanelBody hasNoPadding className="co-quick-start-panel-content__body" data-test="content">
         <QuickStartController
           quickStart={quickStart}
           nextQuickStarts={nextQuickStarts}

diff --git a/packages/module/src/catalog/QuickStartTile.tsx b/packages/module/src/catalog/QuickStartTile.tsx
@@ -76,6 +76,7 @@ const QuickStartTile: React.FC<QuickStartTileProps> = ({
         }
         onClick();
       }}
+      data-test={`tile ${id}`}
       description={
         <QuickStartTileDescription description={description} prerequisites={prerequisites} />
       }

diff --git a/packages/module/src/catalog/QuickStartTileHeader.tsx b/packages/module/src/catalog/QuickStartTileHeader.tsx
@@ -35,7 +35,7 @@ const QuickStartTileHeader: React.FC<QuickStartTileHeaderProps> = ({
 
   return (
     <div className="co-quick-start-tile-header">
-      <Title headingLevel="h3">{name}</Title>
+      <Title headingLevel="h3" data-test="title">{name}</Title>
       <div className="co-quick-start-tile-header__status">
         {type && (
           <Label className="co-quick-start-tile-header--margin" color={type.color}>
@@ -56,6 +56,7 @@ const QuickStartTileHeader: React.FC<QuickStartTileHeaderProps> = ({
             variant="outline"
             color={statusColorMap[status]}
             icon={<StatusIcon status={status} />}
+            data-test="status"
           >
             {statusLocaleMap[status]}
           </Label>

diff --git a/packages/module/src/controller/QuickStartFooter.tsx b/packages/module/src/controller/QuickStartFooter.tsx
@@ -69,7 +69,13 @@ const QuickStartFooter: React.FC<QuickStartFooterProps> = ({
 
   const getPrimaryButton = React.useMemo(
     () => (
-      <Button variant="primary" className="co-quick-start-footer__actionbtn" onClick={onNext} data-testid={`qs-drawer-${camelize(getPrimaryButtonText)}`}>
+      <Button 
+        variant="primary" 
+        className="co-quick-start-footer__actionbtn" 
+        onClick={onNext} 
+        data-testid={`qs-drawer-${camelize(getPrimaryButtonText)}`}
+        data-test={`${getPrimaryButtonText} button`}
+      >
         {getPrimaryButtonText}
       </Button>
     ),