foreman_webhooks 5.1.0 → 6.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c90b385f92db59657a6f026239c2ccfda1b574c852563755fd4dab6522cbf84c
-  data.tar.gz: 05324c9d1589fed4e35b0bf477d0edc7380b5e1abe391872d27a18457355f6f9
+  metadata.gz: 2504e406a10a1be7d18fe058224497d23d5888a67c73c3a37a79a3c60d14b91d
+  data.tar.gz: f558a5e1d7f670ba489dea28d371c7ddd71a604131ffe0b50206286899794e5c
 SHA512:
-  metadata.gz: d8956d6c344db760a44ee50c9699d701078a5cae604dc561a40d55ecb25717d03129dceed5bb770c337cd9b1bc28455b0fff5aa80431e2b6834a368476f358ad
-  data.tar.gz: 5d47beee070a29ac44960ea8c8bbe81e19179be5d806f062809a804457fc2929ea63dab6c6569b89d8ebf0d22a0af7c8a14dee2e1c53b15f1a3e6889eb485755
+  metadata.gz: 3c31f081144dad2899b5b54f9eb6dd7fc04a8314ba684b82925856771d80449872c859e399015c5daaa542d0a743cbe44741d11251b8d1318f1fbed034140453
+  data.tar.gz: a18a91b96ddf51b0989c85fe74a9285c0e4a341253bcdc88ea152bbef3e8cce500dfc5c00f62d8b3fe9765f3e4e4464d59339b939df183d78d7ca418f355485f

data/db/seeds.d/62_shellhooks_proxy_feature.rb

@@ -3,4 +3,4 @@
 proxy_feature = Feature.where(name: 'Shellhooks').first_or_create
 return unless proxy_feature.nil? || proxy_feature.errors.any?
 
-raise "Unable to create proxy feature: #{format_errors proxy_feature}"
+raise "Unable to create proxy feature: #{SeedHelper.format_errors(proxy_feature)}"

data/lib/foreman_webhooks/engine.rb

@@ -14,7 +14,7 @@ module ForemanWebhooks
     initializer 'foreman_webhooks.register_plugin', before: :finisher_hook do |app|
       app.reloader.to_prepare do
         Foreman::Plugin.register :foreman_webhooks do
-          requires_foreman '>= 3.18'
+          requires_foreman '>= 5.0'
           register_gettext
 
           apipie_documented_controllers ["#{ForemanWebhooks::Engine.root}/app/controllers/api/v2/*.rb"]

data/lib/foreman_webhooks/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module ForemanWebhooks
-  VERSION = "5.1.0"
+  VERSION = '6.0.1'
 end

data/webpack/ForemanWebhooks/Routes/Webhooks/Components/WebhookForm/Components/WebhookFormTabs.js

@@ -2,10 +2,10 @@ import React from 'react';
 import PropTypes from 'prop-types';
 import { Tabs, Tab, TabTitleText } from '@patternfly/react-core';
 
+import FieldConstructor from 'foremanReact/components/common/FieldConstructor/FieldConstructor';
 import { sprintf, translate as __ } from 'foremanReact/common/I18n';
 
 import './WebhookFormTabs.css';
-import FieldConstructor from './FieldConstructor';
 
 const WebhookFormTabs = ({
   inputValues,
@@ -48,7 +48,7 @@ const WebhookFormTabs = ({
             type="select"
             label={__('Subscribe to')}
             required
-            allowClear={false}
+            allowClear
             options={availableEvents}
             isLoading={isEventsLoading}
             placeholder={__('Start typing to search...')}
@@ -93,7 +93,7 @@ const WebhookFormTabs = ({
             type="select"
             label={__('Template')}
             required
-            allowClear={false}
+            allowClear
             options={webhookTemplates}
             isLoading={isTemplatesLoading}
             placeholder={__('Start typing to search...')}
@@ -105,7 +105,7 @@ const WebhookFormTabs = ({
             type="select"
             label={__('HTTP Method')}
             required
-            allowClear={false}
+            allowClear
             options={httpMethods}
             placeholder={__('Start typing to search...')}
           />
@@ -173,7 +173,6 @@ const WebhookFormTabs = ({
             placeholder={__(
               "Optional CAs in PEM format concatenated to verify the receiver's SSL certificate"
             )}
-            inputSizeClass="col-md-8"
             rows={8}
             setValue={updateFieldValue}
           />
@@ -200,7 +199,6 @@ const WebhookFormTabs = ({
             label={__('HTTP Headers')}
             labelHelp={__('Optional. Must be a JSON object (ERB allowed)')}
             placeholder='{&#13;&#10;"X-Shellhook-Arg-1": "value",&#13;&#10;"X-Shellhook-Arg-2": "<%= @object.id %>"&#13;&#10;}'
-            inputSizeClass="col-md-8"
             rows={8}
             setValue={updateFieldValue}
           />

data/webpack/ForemanWebhooks/Routes/Webhooks/WebhooksIndexPage/Components/WebhookTestModal.js

@@ -12,6 +12,7 @@ import {
   ModalVariant,
 } from '@patternfly/react-core';
 
+import FieldConstructor from 'foremanReact/components/common/FieldConstructor/FieldConstructor';
 import { APIActions } from 'foremanReact/redux/API';
 import {
   WEBHOOK_TEST_MODAL_ID,
@@ -19,7 +20,6 @@ import {
 } from '../../constants';
 
 import './WebhookModal.scss';
-import FieldConstructor from '../../Components/WebhookForm/Components/FieldConstructor';
 
 const WebhookTestModal = ({ toTest, modalState }) => {
   const dispatch = useDispatch();
@@ -80,7 +80,6 @@ const WebhookTestModal = ({ toTest, modalState }) => {
           label={__('Payload')}
           labelHelp={__('Will be sent as is')}
           placeholder="{
id: 1,
name: test
}"
-          inputSizeClass="col-md-8"
           rows={8}
           value={value.payload}
           setValue={(key, val) => {

data/webpack/ForemanWebhooks/Routes/Webhooks/WebhooksIndexPage/__tests__/WebhooksIndexPage.fixtures.js

@@ -38,9 +38,10 @@ export const queryParams = {
   ...querySort,
 };
 
-export const stateFactory = state => ({
+export const stateFactory = (state = {}, { status } = {}) => ({
   API: {
     [WEBHOOKS_API_REQUEST_KEY]: {
+      ...(status !== undefined && { status }),
       response: {
         ...stateParams,
         ...state,

data/webpack/ForemanWebhooks/Routes/Webhooks/__tests__/WebhooksPageSelectors.test.js

@@ -1,4 +1,4 @@
-import { testSelectorsSnapshotWithFixtures } from '@theforeman/test';
+import { STATUS } from 'foremanReact/constants';
 
 import {
   selectWebhooks,
@@ -11,6 +11,7 @@ import {
   selectIsLoading,
   selectSubtotal,
   selectMessage,
+  selectCanCreate,
 } from '../WebhooksPageSelectors';
 
 import {
@@ -18,28 +19,132 @@ import {
   webhooks,
 } from '../WebhooksIndexPage/__tests__/WebhooksIndexPage.fixtures';
 
-const state = stateFactory({
+const populatedState = stateFactory({
   results: webhooks,
   sort: { by: 'name', order: 'DESC' },
   page: 1,
   perPage: 1,
   search: 'name ~ foo',
   subtotal: 42,
-  message: { type: 'error', text: 'This is error' },
+  canCreate: true,
+  message: { type: 'info', text: 'Saved successfully' },
 });
 
-const fixtures = {
-  'should return webhooks': () => selectWebhooks(state),
-  'should return page': () => selectPage(state),
-  'should return perPage': () => selectPerPage(state),
-  'should return search': () => selectSearch(state),
-  'should return sort': () => selectSort(state),
-  'should return hasData': () => selectHasData(state),
-  'should return hasError': () => selectHasError(state),
-  'should return isLoading': () => selectIsLoading(state),
-  'should return subtotal': () => selectSubtotal(state),
-  'should return message': () => selectMessage(state),
+const resolvedState = stateFactory(
+  { results: webhooks, subtotal: webhooks.length },
+  { status: STATUS.RESOLVED }
+);
+
+const pendingState = stateFactory({ results: [] }, { status: STATUS.PENDING });
+
+const errorState = {
+  API: {
+    webhooks: {
+      status: STATUS.ERROR,
+      response: {
+        message: 'Request failed with status code 500',
+        response: {
+          data: { error: { message: 'Unable to load webhooks' } },
+        },
+      },
+    },
+  },
 };
 
-describe('WebhooksPage selectors', () =>
-  testSelectorsSnapshotWithFixtures(fixtures));
+describe('WebhooksPage selectors', () => {
+  it('returns webhooks from the API response', () => {
+    expect(selectWebhooks(populatedState)).toEqual(webhooks);
+  });
+
+  it('returns page from the API response', () => {
+    expect(selectPage(populatedState)).toBe(1);
+  });
+
+  it('returns perPage from the API response', () => {
+    expect(selectPerPage(populatedState)).toBe(1);
+  });
+
+  it('returns search from the API response', () => {
+    expect(selectSearch(populatedState)).toBe('name ~ foo');
+  });
+
+  it('returns sort with camelCased column name', () => {
+    expect(selectSort(populatedState)).toEqual({ by: 'name', order: 'DESC' });
+  });
+
+  it('camelCases snake_case sort column names', () => {
+    const state = stateFactory({
+      sort: { by: 'target_url', order: 'ASC' },
+    });
+
+    expect(selectSort(state)).toEqual({ by: 'targetUrl', order: 'ASC' });
+  });
+
+  it('returns hasData as false when the API request is not resolved', () => {
+    expect(selectHasData(populatedState)).toBe(false);
+  });
+
+  it('returns hasData as true when the API request is resolved with results', () => {
+    expect(selectHasData(resolvedState)).toBe(true);
+  });
+
+  it('returns hasError as false for a successful API response', () => {
+    expect(selectHasError(populatedState)).toBe(false);
+  });
+
+  it('returns hasError as true when the API request failed', () => {
+    expect(selectHasError(errorState)).toBe(true);
+  });
+
+  it('returns isLoading as true when the API status is missing or pending', () => {
+    expect(selectIsLoading(populatedState)).toBe(true);
+    expect(selectIsLoading(pendingState)).toBe(true);
+  });
+
+  it('returns isLoading as false when the API request is resolved', () => {
+    expect(selectIsLoading(resolvedState)).toBe(false);
+  });
+
+  it('returns subtotal from the API response', () => {
+    expect(selectSubtotal(populatedState)).toBe(42);
+  });
+
+  it('returns message from the API response when there is no error', () => {
+    expect(selectMessage(populatedState)).toEqual({
+      type: 'info',
+      text: 'Saved successfully',
+    });
+  });
+
+  it('returns an error message from the API when the request failed', () => {
+    expect(selectMessage(errorState)).toEqual({
+      type: 'error',
+      text: 'Unable to load webhooks',
+    });
+  });
+
+  it('returns an empty webhook list when the API request failed', () => {
+    expect(selectWebhooks(errorState)).toEqual([]);
+  });
+
+  it('returns canCreate from the API response', () => {
+    expect(selectCanCreate(populatedState)).toBe(true);
+  });
+
+  it('returns default pagination values when the API response is empty', () => {
+    const emptyState = {
+      API: {
+        webhooks: {
+          status: STATUS.RESOLVED,
+          response: {},
+        },
+      },
+    };
+
+    expect(selectPage(emptyState)).toBe(1);
+    expect(selectPerPage(emptyState)).toBe(20);
+    expect(selectWebhooks(emptyState)).toEqual([]);
+    expect(selectSubtotal(emptyState)).toBe(0);
+    expect(selectCanCreate(emptyState)).toBe(false);
+  });
+});

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: foreman_webhooks
 version: !ruby/object:Gem::Version
-  version: 5.1.0
+  version: 6.0.1
 platform: ruby
 authors:
 - Timo Goebel
@@ -138,10 +138,8 @@ files:
 - test/test_plugin_helper.rb
 - test/unit/foreman_webhooks/webhook_service_test.rb
 - webpack/ForemanWebhooks/Routes/ForemanWebhooksRoutes.js
-- webpack/ForemanWebhooks/Routes/Webhooks/Components/WebhookForm/Components/FieldConstructor.js
 - webpack/ForemanWebhooks/Routes/Webhooks/Components/WebhookForm/Components/WebhookFormTabs.css
 - webpack/ForemanWebhooks/Routes/Webhooks/Components/WebhookForm/Components/WebhookFormTabs.js
-- webpack/ForemanWebhooks/Routes/Webhooks/Components/WebhookForm/Components/__tests__/FieldConstructor.test.js
 - webpack/ForemanWebhooks/Routes/Webhooks/Components/WebhookForm/WebhookForm.js
 - webpack/ForemanWebhooks/Routes/Webhooks/Components/WebhookForm/WebhookFormSelectors.js
 - webpack/ForemanWebhooks/Routes/Webhooks/Components/WebhookForm/__tests__/WebhookForm.test.js
@@ -170,7 +168,6 @@ files:
 - webpack/ForemanWebhooks/Routes/Webhooks/WebhooksPageSelectors.js
 - webpack/ForemanWebhooks/Routes/Webhooks/__tests__/WebhooksPageHelpers.test.js
 - webpack/ForemanWebhooks/Routes/Webhooks/__tests__/WebhooksPageSelectors.test.js
-- webpack/ForemanWebhooks/Routes/Webhooks/__tests__/__snapshots__/WebhooksPageSelectors.test.js.snap
 - webpack/ForemanWebhooks/Routes/Webhooks/constants.js
 - webpack/index.js
 - webpack/routes_index.js
@@ -197,7 +194,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 4.0.10
+rubygems_version: 4.0.20
 specification_version: 4
 summary: Configure webhooks for Foreman.
 test_files:
@@ -214,7 +211,6 @@ test_files:
 - test/models/webhook_test.rb
 - test/test_plugin_helper.rb
 - test/unit/foreman_webhooks/webhook_service_test.rb
-- webpack/ForemanWebhooks/Routes/Webhooks/Components/WebhookForm/Components/__tests__/FieldConstructor.test.js
 - webpack/ForemanWebhooks/Routes/Webhooks/Components/WebhookForm/__tests__/WebhookForm.test.js
 - webpack/ForemanWebhooks/Routes/Webhooks/Components/WebhookTable/Components/Formatters/__tests__/actionCellFormatter.test.js
 - webpack/ForemanWebhooks/Routes/Webhooks/Components/WebhookTable/Components/Formatters/__tests__/enabledCellFormatter.test.js

data/webpack/ForemanWebhooks/Routes/Webhooks/Components/WebhookForm/Components/FieldConstructor.js

@@ -1,312 +0,0 @@
-/* eslint-disable max-lines */
-import React, { useEffect, useState } from 'react';
-import PropTypes from 'prop-types';
-import AutocompleteInput from 'foremanReact/components/common/AutocompleteInput/AutocompleteInput';
-import { translate as __ } from 'foremanReact/common/I18n';
-import {
-  ExclamationCircleIcon,
-  HelpIcon,
-  PencilAltIcon,
-} from '@patternfly/react-icons';
-import {
-  TextInput,
-  Checkbox,
-  Button,
-  FormHelperText,
-  HelperText,
-  HelperTextItem,
-  Icon,
-  TextArea,
-  FormGroup,
-  Popover,
-  Grid,
-  GridItem,
-} from '@patternfly/react-core';
-
-const FormField = ({
-  name,
-  type,
-  required,
-  options,
-  validated,
-  value,
-  disabled,
-  setValue,
-  placeholder,
-  errMsg,
-  fieldId,
-  setIsPasswordDisabled,
-  ...props
-}) => {
-  const [fieldValidated, setFieldValidated] = useState('default');
-  const [firstLoad, setFirstLoad] = useState(true);
-
-  const requiredValidate = () => {
-    if (firstLoad || !required) return;
-    if (!value || value === '' || validated === 'error')
-      setFieldValidated('error');
-    else setFieldValidated('success');
-  };
-
-  const localHandler = (_event, newValue) => {
-    setValue(name, newValue);
-  };
-
-  useEffect(() => {
-    setFirstLoad(false);
-  }, []);
-
-  useEffect(() => {
-    requiredValidate();
-    // eslint-disable-next-line react-hooks/exhaustive-deps
-  }, [value]);
-
-  if (type === 'checkbox') {
-    return (
-      <>
-        <Checkbox
-          ouiaId={name}
-          id={fieldId ?? `id-${name}`}
-          isChecked={value || false}
-          onChange={(_, newValue) => {
-            setValue(name, newValue);
-          }}
-          isDisabled={disabled}
-          isRequired={required}
-          type={type}
-          validated={fieldValidated}
-        />
-        <ValidationComponent localValidated={fieldValidated} errMsg={errMsg} />
-      </>
-    );
-  } else if (type === 'textarea') {
-    return (
-      <>
-        <TextArea
-          rows={6}
-          name={name}
-          id={fieldId ?? `id-${name}`}
-          value={value ?? ''}
-          onChange={(_, newValue) => setValue(name, newValue)}
-          isRequired={required}
-          isDisabled={disabled}
-          type={type}
-          validated={fieldValidated}
-          placeholder={placeholder}
-        />
-        <ValidationComponent localValidated={fieldValidated} errMsg={errMsg} />
-      </>
-    );
-  } else if (options.length !== 0) {
-    return (
-      <>
-        <AutocompleteInput
-          options={options}
-          selected={value ?? ''}
-          onSelect={newValue => setValue(name, newValue)}
-          onChange={() => requiredValidate()}
-          name={name}
-          placeholder={placeholder}
-          validationStatus={fieldValidated}
-          validationMsg={errMsg}
-          fieldId={fieldId}
-        />
-      </>
-    );
-  }
-  return (
-    <>
-      {name === 'password' && disabled && setIsPasswordDisabled ? (
-        <Grid hasGutter={false}>
-          <GridItem span={11}>
-            <TextInput
-              name={name}
-              value={value ?? ''}
-              id={fieldId ?? `id-${name}`}
-              onChange={localHandler}
-              isDisabled={disabled}
-              isRequired={required}
-              type={type}
-              validated={fieldValidated}
-              placeholder={placeholder}
-              onBlur={requiredValidate}
-              autoComplete="off"
-            />
-          </GridItem>
-          <GridItem span={1}>
-            <Button
-              ouiaId={`reset-${name}`}
-              onClick={() => setIsPasswordDisabled(!disabled)}
-              variant="control"
-              icon={<PencilAltIcon />}
-            />
-          </GridItem>
-        </Grid>
-      ) : (
-        <TextInput
-          name={name}
-          value={value ?? ''}
-          id={fieldId ?? `id-${name}`}
-          onChange={localHandler}
-          isRequired={required}
-          type={type}
-          validated={fieldValidated}
-          onBlur={requiredValidate}
-          autoComplete={type === 'password' ? 'new-password' : null}
-        />
-      )}
-
-      <ValidationComponent localValidated={fieldValidated} errMsg={errMsg} />
-    </>
-  );
-};
-
-const ValidationComponent = ({ localValidated, errMsg }) => (
-  <>
-    {localValidated === 'error' && (
-      <FormHelperText>
-        <HelperText>
-          <HelperTextItem
-            icon={
-              <Icon>
-                <ExclamationCircleIcon />
-              </Icon>
-            }
-            variant={localValidated}
-          >
-            {errMsg ?? __('Field is required')}
-          </HelperTextItem>
-        </HelperText>
-      </FormHelperText>
-    )}
-  </>
-);
-
-const FieldConstructor = ({
-  label,
-  value,
-  required,
-  labelHelp,
-  fieldId,
-  name,
-  ...props
-}) => (
-  <FormGroup
-    isInline
-    fieldId={fieldId ?? `id-${name}`}
-    label={label}
-    isRequired={required}
-    labelIcon={
-      labelHelp ? (
-        <Popover
-          bodyContent={
-            typeof labelHelp === 'string' ? (
-              <div dangerouslySetInnerHTML={{ __html: labelHelp }} />
-            ) : (
-              labelHelp
-            )
-          }
-        >
-          <Button
-            ouiaId={`label-${label}`}
-            type="button"
-            variant="plain"
-            onClick={e => e.preventDefault()}
-          >
-            <Icon>
-              <HelpIcon />
-            </Icon>
-          </Button>
-        </Popover>
-      ) : (
-        ''
-      )
-    }
-  >
-    <FormField
-      value={value}
-      required={required}
-      name={name}
-      {...props}
-      fieldId={fieldId}
-    />
-  </FormGroup>
-);
-
-FieldConstructor.propTypes = {
-  label: PropTypes.string.isRequired,
-  required: PropTypes.bool,
-  labelHelp: PropTypes.oneOfType([
-    PropTypes.string,
-    PropTypes.object,
-    PropTypes.node,
-  ]),
-  setValue: PropTypes.func.isRequired,
-  name: PropTypes.string.isRequired,
-  disabled: PropTypes.bool,
-  value: PropTypes.oneOfType([
-    PropTypes.string,
-    PropTypes.object,
-    PropTypes.bool,
-    PropTypes.number,
-  ]),
-  fieldId: PropTypes.oneOfType([PropTypes.string, PropTypes.number]),
-};
-
-FieldConstructor.defaultProps = {
-  labelHelp: undefined,
-  required: false,
-  disabled: false,
-  value: '',
-  fieldId: undefined,
-};
-
-FormField.propTypes = {
-  disabled: PropTypes.bool,
-  setValue: PropTypes.func.isRequired,
-  name: PropTypes.string.isRequired,
-  type: PropTypes.string.isRequired,
-  label: PropTypes.string,
-  required: PropTypes.bool,
-  allowClear: PropTypes.bool,
-  options: PropTypes.arrayOf(
-    PropTypes.shape({
-      value: PropTypes.oneOfType([PropTypes.string, PropTypes.number])
-        .isRequired,
-      label: PropTypes.string.isRequired,
-    })
-  ),
-  validated: PropTypes.string,
-  placeholder: PropTypes.string,
-  errMsg: PropTypes.string,
-  value: PropTypes.oneOfType([
-    PropTypes.bool,
-    PropTypes.string,
-    PropTypes.number,
-  ]),
-  fieldId: PropTypes.string,
-};
-
-FormField.defaultProps = {
-  label: '',
-  errMsg: null,
-  placeholder: '',
-  validated: 'default',
-  required: false,
-  allowClear: false,
-  options: [],
-  disabled: false,
-  value: '',
-  fieldId: undefined,
-};
-
-ValidationComponent.propTypes = {
-  localValidated: PropTypes.string.isRequired,
-  errMsg: PropTypes.string,
-};
-
-ValidationComponent.defaultProps = {
-  errMsg: null,
-};
-
-export default FieldConstructor;

data/webpack/ForemanWebhooks/Routes/Webhooks/Components/WebhookForm/Components/__tests__/FieldConstructor.test.js

@@ -1,216 +0,0 @@
-import React from 'react';
-import { render, screen, fireEvent } from '@testing-library/react';
-import userEvent from '@testing-library/user-event';
-import '@testing-library/jest-dom';
-import FieldConstructor from '../FieldConstructor';
-
-const defaultProps = {
-  name: 'test-field',
-  setValue: jest.fn(),
-  label: 'Test Field',
-  value: '',
-  type: 'text',
-};
-
-describe('FieldConstructor RTL Tests', () => {
-  beforeEach(() => {
-    jest.clearAllMocks();
-  });
-
-  describe('Text Input Fields', () => {
-    test('renders text input field', () => {
-      render(<FieldConstructor {...defaultProps} type="text" />);
-
-      expect(screen.getByText('Test Field')).toBeInTheDocument();
-      expect(document.getElementById('id-test-field')).toHaveAttribute(
-        'type',
-        'text'
-      );
-    });
-
-    test('renders password input field', () => {
-      render(<FieldConstructor {...defaultProps} type="password" />);
-
-      expect(screen.getByText('Test Field')).toBeInTheDocument();
-      expect(document.getElementById('id-test-field')).toHaveAttribute(
-        'type',
-        'password'
-      );
-    });
-
-    test('calls setValue when text input changes', () => {
-      render(<FieldConstructor {...defaultProps} type="text" />);
-
-      const input = document.getElementById('id-test-field');
-      fireEvent.change(input, { target: { value: 'test value' } });
-
-      expect(defaultProps.setValue).toHaveBeenCalledWith(
-        'test-field',
-        'test value'
-      );
-    });
-
-    test('renders with initial value', () => {
-      render(
-        <FieldConstructor {...defaultProps} type="text" value="initial value" />
-      );
-
-      expect(document.getElementById('id-test-field')).toHaveValue(
-        'initial value'
-      );
-    });
-
-    test('renders as enabled when loading', () => {
-      render(<FieldConstructor {...defaultProps} type="text" isLoading />);
-
-      expect(document.getElementById('id-test-field')).toBeEnabled();
-    });
-  });
-
-  describe('Textarea Fields', () => {
-    test('renders textarea field', () => {
-      render(<FieldConstructor {...defaultProps} type="textarea" />);
-
-      expect(screen.getByText('Test Field')).toBeInTheDocument();
-      expect(document.getElementById('id-test-field').tagName).toBe('TEXTAREA');
-    });
-
-    test('calls setValue when textarea changes', () => {
-      render(<FieldConstructor {...defaultProps} type="textarea" />);
-
-      const textarea = document.getElementById('id-test-field');
-      fireEvent.change(textarea, { target: { value: 'textarea content' } });
-
-      expect(defaultProps.setValue).toHaveBeenCalledWith(
-        'test-field',
-        'textarea content'
-      );
-    });
-
-    test('renders with specified rows', () => {
-      render(<FieldConstructor {...defaultProps} type="textarea" rows={5} />);
-
-      expect(document.getElementsByTagName('textarea')).toHaveLength(1);
-    });
-  });
-
-  describe('Checkbox Fields', () => {
-    test('renders checkbox field', () => {
-      render(<FieldConstructor {...defaultProps} type="checkbox" />);
-
-      expect(screen.getByText('Test Field')).toBeInTheDocument();
-      expect(document.getElementById('id-test-field')).toHaveAttribute(
-        'type',
-        'checkbox'
-      );
-    });
-
-    test('renders unchecked checkbox', () => {
-      render(
-        <FieldConstructor {...defaultProps} type="checkbox" value={false} />
-      );
-
-      expect(document.getElementById('id-test-field')).not.toBeChecked();
-    });
-  });
-
-  describe('Autocomplete render  test', () => {
-    test('renders autocomplete', () => {
-      const props = {
-        ...defaultProps,
-        onChange: defaultProps.setValue,
-        options: [
-          { value: 'host_created.event.foreman', label: 'Host Created' },
-          { value: 'host_updated.event.foreman', label: 'Host Updated' },
-        ],
-      };
-      render(<FieldConstructor {...props} type="select" />);
-
-      expect(screen.getByRole('combobox')).toBeInTheDocument();
-    });
-  });
-
-  describe('Label Help', () => {
-    test('renders help icon when labelHelp is provided', () => {
-      render(
-        <FieldConstructor {...defaultProps} labelHelp="This is help text" />
-      );
-
-      expect(screen.getByRole('button')).toBeInTheDocument();
-    });
-
-    test('does not render help icon when labelHelp is not provided', () => {
-      render(<FieldConstructor {...defaultProps} />);
-
-      expect(screen.queryByRole('button')).not.toBeInTheDocument();
-    });
-
-    test('renders help icon with JSX content', async () => {
-      const helpContent = <div>JSX help content</div>;
-      render(<FieldConstructor {...defaultProps} labelHelp={helpContent} />);
-
-      const button = screen.getByRole('button');
-      expect(button).toBeInTheDocument();
-
-      userEvent.click(button);
-
-      expect(await screen.findByText('JSX help content')).toBeInTheDocument();
-    });
-  });
-
-  describe('Validation', () => {
-    test('renders error message when validated is error', async () => {
-      render(
-        <>
-          <FieldConstructor {...defaultProps} required type="text" />
-          <button>Dummy</button>
-        </>
-      );
-
-      const input = document.getElementById('id-test-field');
-      await userEvent.click(input);
-      expect(input).toHaveFocus();
-
-      await userEvent.tab();
-      expect(input).not.toHaveFocus();
-
-      expect(screen.getByText('Field is required')).toBeInTheDocument();
-    });
-
-    test('does not render error message when validated is not error', () => {
-      render(
-        <FieldConstructor {...defaultProps} type="text" validated="success" />
-      );
-
-      expect(screen.queryByText('Field is required')).not.toBeInTheDocument();
-    });
-  });
-
-  describe('Edge Cases', () => {
-    test('handles undefined value', () => {
-      render(
-        <FieldConstructor {...defaultProps} type="text" value={undefined} />
-      );
-
-      expect(document.getElementById('id-test-field')).toHaveValue('');
-    });
-
-    test('handles null value', () => {
-      render(<FieldConstructor {...defaultProps} type="text" value={null} />);
-
-      expect(document.getElementById('id-test-field')).toHaveValue('');
-    });
-
-    test('handles numeric value', () => {
-      render(<FieldConstructor {...defaultProps} type="text" value={123} />);
-
-      expect(document.getElementById('id-test-field')).toHaveValue('123');
-    });
-
-    test('handles boolean value for checkbox', () => {
-      render(<FieldConstructor {...defaultProps} type="checkbox" value />);
-
-      expect(document.getElementById('id-test-field')).toBeChecked();
-    });
-  });
-});

data/webpack/ForemanWebhooks/Routes/Webhooks/__tests__/__snapshots__/WebhooksPageSelectors.test.js.snap

@@ -1,50 +0,0 @@
-// Jest Snapshot v1, https://goo.gl/fbAQLP
-
-exports[`WebhooksPage selectors should return hasData 1`] = `false`;
-
-exports[`WebhooksPage selectors should return hasError 1`] = `false`;
-
-exports[`WebhooksPage selectors should return isLoading 1`] = `true`;
-
-exports[`WebhooksPage selectors should return message 1`] = `
-Object {
-  "text": "This is error",
-  "type": "error",
-}
-`;
-
-exports[`WebhooksPage selectors should return page 1`] = `1`;
-
-exports[`WebhooksPage selectors should return perPage 1`] = `1`;
-
-exports[`WebhooksPage selectors should return search 1`] = `"name ~ foo"`;
-
-exports[`WebhooksPage selectors should return sort 1`] = `
-Object {
-  "by": "name",
-  "order": "DESC",
-}
-`;
-
-exports[`WebhooksPage selectors should return subtotal 1`] = `42`;
-
-exports[`WebhooksPage selectors should return webhooks 1`] = `
-Array [
-  Object {
-    "canDelete": true,
-    "canEdit": true,
-    "enabled": true,
-    "id": 1,
-    "name": "my-webhook",
-    "targetUrl": "https://my-machine.example.com",
-  },
-  Object {
-    "canDelete": false,
-    "canEdit": false,
-    "enabled": false,
-    "id": 2,
-    "name": "your-webhook",
-    "targetUrl": "https://your-machine.example.com",
-  },
-]
-`;