From 6cb1740e01e1a6e1e4e226765e097282beea1f42 Mon Sep 17 00:00:00 2001
From: lvillen <l.villenjaldon@gmail.com>
Date: Mon, 11 Nov 2024 09:54:59 +0100
Subject: [PATCH] =?UTF-8?q?=F0=9F=A7=AA=20Add=20tests?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../authentication_providers/new.feature        | 13 ++++++++++++-
 jest.config.js                                  |  7 +++----
 .../AccountAuthenticationProviders.spec.tsx     |  7 ++++---
 .../AuthenticationProvidersEmptyState.spec.tsx  | 17 +++++++++++++++++
 4 files changed, 36 insertions(+), 8 deletions(-)
 create mode 100644 spec/javascripts/AuthenticationProviders/components/AuthenticationProvidersEmptyState.spec.tsx

diff --git a/features/provider/admin/account/authentication_providers/new.feature b/features/provider/admin/account/authentication_providers/new.feature
index c9aed6a9e9..a8879259bb 100644
--- a/features/provider/admin/account/authentication_providers/new.feature
+++ b/features/provider/admin/account/authentication_providers/new.feature
@@ -6,9 +6,20 @@ Feature: Account Settings > Users > SSO Integrations > New
 
   Scenario: Navigation
     Given they go to the users sso integrations page
-    When they follow "Create a new SSO integration"
+    When they follow "Add a SSO integration"
     Then the current page is the new sso integration page
 
+  Scenario: Navigation when there is an integration
+    Given a red hat single sign-on integration
+    And they go to the users sso integrations page
+    Then they should not see "Add a SSO integration"
+    And there should be a link to "Create a new SSO integration"
+
+  Scenario: Empty state
+    Given they go to the users sso integrations page
+    Then they should see "No SSO integrations"
+    And there should be a link to "Add a SSO integration"
+
   Scenario: Create RH SSO new integration
     Given they go to the new sso integration page
     When the form is submitted with:
diff --git a/jest.config.js b/jest.config.js
index 23afd65ec1..0a5e855289 100644
--- a/jest.config.js
+++ b/jest.config.js
@@ -191,10 +191,9 @@ module.exports = {
   },
 
   // An array of regexp pattern strings that are matched against all source file paths, matched files will skip transformation
-  // transformIgnorePatterns: [
-  //   "/node_modules/",
-  //   "\\.pnp\\.[^\\/]+$"
-  // ],
+  transformIgnorePatterns: [
+    'node_modules/(?!(?:@patternfly/react-icons)/)', // Transform @patternfly/react-icons
+  ],
 
   // An array of regexp pattern strings that are matched against all modules before the module loader will automatically return a mock for them
   // unmockedModulePathPatterns: undefined,
diff --git a/spec/javascripts/AuthenticationProviders/components/AccountAuthenticationProviders.spec.tsx b/spec/javascripts/AuthenticationProviders/components/AccountAuthenticationProviders.spec.tsx
index 61bea0f4c5..d02ad50f7b 100644
--- a/spec/javascripts/AuthenticationProviders/components/AccountAuthenticationProviders.spec.tsx
+++ b/spec/javascripts/AuthenticationProviders/components/AccountAuthenticationProviders.spec.tsx
@@ -8,6 +8,7 @@ import { AccountAuthenticationProviders } from 'AuthenticationProviders/componen
 import { mockLocation, waitForPromises } from 'utilities/test-utils'
 import { EnforceSSOSwitch } from 'AuthenticationProviders/components/EnforceSSOSwitch'
 import { AuthenticationProvidersTable } from 'AuthenticationProviders/components/AuthenticationProvidersTable'
+import { AuthenticationProvidersEmptyState } from 'AuthenticationProviders/components/AuthenticationProvidersEmptyState'
 
 import type { Props } from 'AuthenticationProviders/components/AccountAuthenticationProviders'
 
@@ -43,15 +44,15 @@ afterAll(() => {
 describe('when SSO toggle is hidden', () => {
   const props = { showToggle: false }
 
-  it('should only render a table', () => {
+  it('should render empty state', () => {
     const wrapper = shallowWrapper(props)
     expect(wrapper.exists(EnforceSSOSwitch)).toEqual(false)
-    expect(wrapper.exists(AuthenticationProvidersTable)).toEqual(true)
+    expect(wrapper.exists(AuthenticationProvidersEmptyState)).toEqual(true)
   })
 })
 
 describe('when SSO toggle is visible', () => {
-  const props = { showToggle: true }
+  const props = { showToggle: true, table: { ...defaultProps.table, count: 1 } }
 
   it('should render both a switch and a table', () => {
     const wrapper = shallowWrapper(props)
diff --git a/spec/javascripts/AuthenticationProviders/components/AuthenticationProvidersEmptyState.spec.tsx b/spec/javascripts/AuthenticationProviders/components/AuthenticationProvidersEmptyState.spec.tsx
new file mode 100644
index 0000000000..3bbce05da1
--- /dev/null
+++ b/spec/javascripts/AuthenticationProviders/components/AuthenticationProvidersEmptyState.spec.tsx
@@ -0,0 +1,17 @@
+import { mount } from 'enzyme'
+
+import { AuthenticationProvidersEmptyState } from 'AuthenticationProviders/components/AuthenticationProvidersEmptyState'
+
+import type { Props } from 'AuthenticationProviders/components/AuthenticationProvidersEmptyState'
+
+const defaultProps = {
+  newHref: '/new'
+}
+
+const mountWrapper = (props: Partial<Props> = {}) => mount(<AuthenticationProvidersEmptyState {...{ ...defaultProps, ...props }} />)
+
+it('should render itself', () => {
+  const wrapper = mountWrapper()
+
+  expect(wrapper.exists()).toEqual(true)
+})