From 383d5ea42991ac538475def839682a34025b6fb2 Mon Sep 17 00:00:00 2001 From: Kevin Cormier Date: Thu, 5 Dec 2024 14:56:43 -0500 Subject: [PATCH] Revert "Temporarily skip CSV export tests" This reverts commit 288315d5b4cc009364e29e587d72bcc4ef8a0677. Signed-off-by: Kevin Cormier --- frontend/src/routes/Applications/Overview.test.tsx | 2 +- frontend/src/routes/Credentials/CredentialsPage.test.tsx | 2 +- .../Governance/overview/ClusterPolicySummarySidebar.test.tsx | 2 +- .../policies/policy-details/PolicyDetailsHistory.test.tsx | 2 +- .../policies/policy-details/PolicyDetailsResults.test.tsx | 2 +- .../policy-sets/components/PolicySetDetailSidebar.test.tsx | 2 +- .../Infrastructure/Automations/AnsibleAutomations.test.tsx | 2 +- .../Infrastructure/Clusters/ClusterPools/ClusterPools.test.tsx | 2 +- .../Infrastructure/Clusters/ClusterSets/ClusterSets.test.tsx | 2 +- .../Clusters/ManagedClusters/ManagedClusters.test.tsx | 2 +- .../InfraEnvironments/InfraEnvironmentsPage.test.tsx | 2 +- frontend/src/ui-components/AcmTable/AcmTable.test.tsx | 2 +- 12 files changed, 12 insertions(+), 12 deletions(-) diff --git a/frontend/src/routes/Applications/Overview.test.tsx b/frontend/src/routes/Applications/Overview.test.tsx index 52a1f7dfb..89eec87f7 100644 --- a/frontend/src/routes/Applications/Overview.test.tsx +++ b/frontend/src/routes/Applications/Overview.test.tsx @@ -235,7 +235,7 @@ describe('Applications Page', () => { userEvent.click(screen.getByRole('button', { name: /close openshift/i })) }) - test.skip('export button should produce a file for download', async () => { + test('export button should produce a file for download', async () => { nockAggegateRequest('applications', fetchAggregate.req, applicationAggregate.res) await waitForText('feng-remote-argo8') diff --git a/frontend/src/routes/Credentials/CredentialsPage.test.tsx b/frontend/src/routes/Credentials/CredentialsPage.test.tsx index 6142a3092..cb6e3ea0c 100644 --- a/frontend/src/routes/Credentials/CredentialsPage.test.tsx +++ b/frontend/src/routes/Credentials/CredentialsPage.test.tsx @@ -267,7 +267,7 @@ describe('provider connections page RBAC', () => { }) describe('Export from clusterpool table', () => { - test.skip('export button should produce a file for download', async () => { + test('export button should produce a file for download', async () => { nockGet(getSecrets1.req, getSecrets1.res) // get 'secrets' in 'provider-connection-namespace' namespace render( diff --git a/frontend/src/routes/Governance/overview/ClusterPolicySummarySidebar.test.tsx b/frontend/src/routes/Governance/overview/ClusterPolicySummarySidebar.test.tsx index d37648b0a..5a683da5f 100644 --- a/frontend/src/routes/Governance/overview/ClusterPolicySummarySidebar.test.tsx +++ b/frontend/src/routes/Governance/overview/ClusterPolicySummarySidebar.test.tsx @@ -113,7 +113,7 @@ describe('Policies Page', () => { }) describe('Export from ClusterPolicySummarySidebar table', () => { - test.skip('export button should produce a file for download', async () => { + test('export button should produce a file for download', async () => { render( { diff --git a/frontend/src/routes/Governance/policies/policy-details/PolicyDetailsHistory.test.tsx b/frontend/src/routes/Governance/policies/policy-details/PolicyDetailsHistory.test.tsx index 3e429789a..482bfff7c 100644 --- a/frontend/src/routes/Governance/policies/policy-details/PolicyDetailsHistory.test.tsx +++ b/frontend/src/routes/Governance/policies/policy-details/PolicyDetailsHistory.test.tsx @@ -156,7 +156,7 @@ describe('Policy Details History content', () => { }) describe('Export from policy details history table', () => { - test.skip('export button should produce a file for download', async () => { + test('export button should produce a file for download', async () => { render( { diff --git a/frontend/src/routes/Governance/policies/policy-details/PolicyDetailsResults.test.tsx b/frontend/src/routes/Governance/policies/policy-details/PolicyDetailsResults.test.tsx index 90afe4d54..6d376e55f 100644 --- a/frontend/src/routes/Governance/policies/policy-details/PolicyDetailsResults.test.tsx +++ b/frontend/src/routes/Governance/policies/policy-details/PolicyDetailsResults.test.tsx @@ -198,7 +198,7 @@ describe('Export from policy details results table', () => { nockIgnoreRBAC() nockIgnoreApiPaths() }) - test.skip('export button should produce a file for download', async () => { + test('export button should produce a file for download', async () => { const context: PolicyDetailsContext = { policy: mockPolicy[0] } render( { }) }) describe('Export from policy details results table', () => { - test.skip('export button should produce a file for download', async () => { + test('export button should produce a file for download', async () => { const policySet: PolicySet = { apiVersion: 'policy.open-cluster-management.io/v1beta1', kind: 'PolicySet', diff --git a/frontend/src/routes/Infrastructure/Automations/AnsibleAutomations.test.tsx b/frontend/src/routes/Infrastructure/Automations/AnsibleAutomations.test.tsx index 672d06907..09245f5ba 100644 --- a/frontend/src/routes/Infrastructure/Automations/AnsibleAutomations.test.tsx +++ b/frontend/src/routes/Infrastructure/Automations/AnsibleAutomations.test.tsx @@ -281,7 +281,7 @@ describe('Automations page - operator checking', () => { }) describe('Export from automation table', () => { - test.skip('export button should produce a file for download', async () => { + test('export button should produce a file for download', async () => { nockIgnoreOperatorCheck() render( { }) describe('Export from clusterpool table', () => { - test.skip('export button should produce a file for download', async () => { + test('export button should produce a file for download', async () => { nockIgnoreRBAC() nockIgnoreApiPaths() render( diff --git a/frontend/src/routes/Infrastructure/Clusters/ClusterSets/ClusterSets.test.tsx b/frontend/src/routes/Infrastructure/Clusters/ClusterSets/ClusterSets.test.tsx index bc3982884..565189ab0 100644 --- a/frontend/src/routes/Infrastructure/Clusters/ClusterSets/ClusterSets.test.tsx +++ b/frontend/src/routes/Infrastructure/Clusters/ClusterSets/ClusterSets.test.tsx @@ -116,7 +116,7 @@ describe('ClusterSets page with csv export', () => { ) }) - test.skip('export button should produce a file for download', async () => { + test('export button should produce a file for download', async () => { window.URL.createObjectURL = jest.fn() window.URL.revokeObjectURL = jest.fn() const documentBody = document.body.appendChild diff --git a/frontend/src/routes/Infrastructure/Clusters/ManagedClusters/ManagedClusters.test.tsx b/frontend/src/routes/Infrastructure/Clusters/ManagedClusters/ManagedClusters.test.tsx index 37388e6a0..5141f500f 100644 --- a/frontend/src/routes/Infrastructure/Clusters/ManagedClusters/ManagedClusters.test.tsx +++ b/frontend/src/routes/Infrastructure/Clusters/ManagedClusters/ManagedClusters.test.tsx @@ -291,7 +291,7 @@ describe('Clusters Page export', () => { await waitForNock(metricNock) await waitForText(mockManagedCluster0.metadata.name!, true) }) - test.skip('export button should produce a file for download', async () => { + test('export button should produce a file for download', async () => { window.URL.createObjectURL = jest.fn() window.URL.revokeObjectURL = jest.fn() const documentBody = document.body.appendChild diff --git a/frontend/src/routes/Infrastructure/InfraEnvironments/InfraEnvironmentsPage.test.tsx b/frontend/src/routes/Infrastructure/InfraEnvironments/InfraEnvironmentsPage.test.tsx index 9fbcb090f..c4feb3337 100644 --- a/frontend/src/routes/Infrastructure/InfraEnvironments/InfraEnvironmentsPage.test.tsx +++ b/frontend/src/routes/Infrastructure/InfraEnvironments/InfraEnvironmentsPage.test.tsx @@ -128,7 +128,7 @@ describe('Infrastructure Environments page utility functions', () => { }) describe('Export from host inventory table', () => { - test.skip('export button should produce a file for download', async () => { + test('export button should produce a file for download', async () => { const { getByLabelText, getByText } = render() window.URL.createObjectURL = jest.fn() window.URL.revokeObjectURL = jest.fn() diff --git a/frontend/src/ui-components/AcmTable/AcmTable.test.tsx b/frontend/src/ui-components/AcmTable/AcmTable.test.tsx index 7a61385aa..681ccd51f 100644 --- a/frontend/src/ui-components/AcmTable/AcmTable.test.tsx +++ b/frontend/src/ui-components/AcmTable/AcmTable.test.tsx @@ -1017,7 +1017,7 @@ describe('AcmTable', () => { expect(exportContent).toEqual("'Ready':'true','Hibernating':'false'") }) - test.skip('export button should produce a file for download', () => { + test('export button should produce a file for download', () => { const addSubRowsCallback = () => { return [ { -- 2.39.1