foreman_host_reports 0.0.3 → 0.0.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: df2c9864c95d5918c25782bc4fd90c8b19dbf3417bb7f0b00d1350ca55adc399
-  data.tar.gz: 4c7d17ca2b8dbb6ffda51dbf35d92c83ae523dc7b295067076f3d9579dd5da82
+  metadata.gz: 488222c4e90b62231ba01365932c00ccb2619ea7f8225757bf84c423cd56b13a
+  data.tar.gz: 39f11ab9a01e780bff2105f09726f95b564d45169b6c9c2466e9846f0433aff8
 SHA512:
-  metadata.gz: 41eb977481bf368bbcafdc37adad69a0cb85f3b61d052e549af5aadcaa35e33746317663f25ffbdbb34e4cf9583b315d52ccaed031334be946adcd8cd23bd907
-  data.tar.gz: 26ff05b79fbf665b9858c11eebbe5082ed479afc0bbd3810efa0950fb10fde3e51c2e092e431b1bbf52749a57f044da777e3cdf5bf2ed7fac223c9cb703998ed
+  metadata.gz: 9970feb6d0dd2593977eded7cade385a1a17743da6af330f3c5fb1c338dd62a63716b07b19bc5fd7fef1b22bd20147d08d9fc41639f570c6c2b43cd02100a36d
+  data.tar.gz: 76f80c8de18583d372beee125c791f8eef7fd41bbcb501c318951ab7708a311eb6905f3e94b5df841700c9c2f15a1157985cb23d8ea5661aac7f45d738286879

data/app/controllers/api/v2/host_reports_controller.rb

@@ -11,7 +11,7 @@ module Api
       before_action :find_resource, only: %i[destroy]
       before_action :resolve_ids, only: %i[create]
 
-      add_smart_proxy_filters :create, features: proc { HostReport.authorized_smart_proxy_features }
+      add_smart_proxy_filters :create, features: ['Reports']
 
       api :GET, '/host_reports/', N_('List host reports')
       param_group :search_and_pagination, ::Api::V2::BaseController
@@ -34,7 +34,10 @@ module Api
           param :host, String, required: true, desc: N_("Hostname of the report's host origin")
           param :format, HostReport.formats.keys, required: false, desc: N_('Format of the report, e.g. Ansible')
           param :reported_at, String, required: true, desc: N_('UTC time of the report')
-          param :status, Integer, required: true, desc: N_('Bitfield with arbitrary amount of status counters')
+          param :applied, Integer, required: false, desc: N_('Number of applied resources or tasks')
+          param :failed, Integer, required: false, desc: N_('Number of failed resources or tasks')
+          param :pending, Integer, required: false, desc: N_('Number of pending resources or tasks')
+          param :other, Integer, required: false, desc: N_('Number of other resources or tasks')
           param :body, String, required: true, desc: N_('String with JSON formatted body of the report')
           param :proxy, String, required: false, desc: N_('Hostname of the proxy processed the report')
           param :keywords, Array, of: String, required: false, desc: N_('A list of keywords to associate with the report for better searching')

data/app/controllers/concerns/foreman_host_reports/controller/parameters/host_report.rb

@@ -10,7 +10,8 @@ module ForemanHostReports
           def host_report_params_filter
             Foreman::ParameterFilter.new(::HostReport).tap do |filter|
               # body is permitted in controller
-              filter.permit :format, :version, :host, :proxy, :reported_at, :status, :proxy_id, :host_id
+              filter.permit :format, :version, :host, :proxy, :reported_at,
+                :proxy_id, :host_id, :applied, :failed, :pending, :other
               filter.permit :keywords => []
             end
           end

data/app/models/host_report.rb

@@ -10,7 +10,7 @@ class HostReport < ApplicationRecord
   has_one :organization, through: :host
   has_one :location, through: :host
 
-  validates :host_id, :reported_at, :status, presence: true
+  validates :host_id, :reported_at, presence: true
 
   enum format: {
     # plain text report (no processing)
@@ -20,10 +20,6 @@ class HostReport < ApplicationRecord
     ansible: 2,
   }.freeze
 
-  STATUS = {
-    plain: %w[debug normal warning error],
-  }.freeze
-
   scoped_search relation: :host, on: :name, complete_value: true, rename: :host, aliases: %i[host_name]
   scoped_search relation: :proxy, on: :name, complete_value: true, rename: :proxy
   scoped_search relation: :organization, on: :name, complete_value: true, rename: :organization
@@ -45,36 +41,10 @@ class HostReport < ApplicationRecord
 
   default_scope -> { order('reported_at DESC') }
 
-  # TODO: temporary until we decide what will status bitfiled be exactly
-  # rubocop:disable Style/RescueModifier
-  def status
-    @status ||= case format
-                when 'puppet'
-                  JSON.parse(body)['metrics']['resources']['values'] rescue []
-                when 'ansible'
-                  JSON.parse(body)['status'] rescue {}
-                else
-                  super
-                end
-  end
-  # rubocop:enable Style/RescueModifier
-
   def report_keywords
     ReportKeyword.where(id: report_keyword_ids)
   end
 
-  def self.authorized_smart_proxy_features
-    @authorized_smart_proxy_features ||= %w[Puppet Ansible]
-  end
-
-  def self.register_smart_proxy_feature(feature)
-    @authorized_smart_proxy_features = (authorized_smart_proxy_features + [feature]).uniq
-  end
-
-  def self.unregister_smart_proxy_feature(feature)
-    @authorized_smart_proxy_features -= [feature]
-  end
-
   def self.search_by_keyword(_key, operator, value)
     conditions = sanitize_sql_for_conditions(["report_keywords.name #{operator} ?", value_to_sql(operator, value)])
     keyword_ids = ReportKeyword.where(conditions).distinct.pluck(:id)

data/app/models/report_keyword.rb

@@ -8,4 +8,9 @@ class ReportKeyword < ApplicationRecord
   def self.klass
     self
   end
+
+  # Needed for ActiveRecord to simulate ActiveRecord::Associations::CollectionProxy
+  def self.macro
+    :has_many
+  end
 end

data/app/views/api/v2/host_reports/main.json.rabl

@@ -5,7 +5,8 @@ object @host_report
 extends 'api/v2/host_reports/base'
 extends 'api/v2/layouts/permissions'
 
-attributes :format, :host_id, :proxy_id, :reported_at, :status
+attributes :format, :host_id, :proxy_id, :reported_at, :applied, :failed,
+  :pending, :other
 
 node(:host_name) do |report|
   report.host.name

data/db/migrate/20211011141813_change_status_column.rb

@@ -0,0 +1,10 @@
+class ChangeStatusColumn < ActiveRecord::Migration[6.0]
+  def change
+    remove_column :host_reports, :status, :bigint
+
+    add_column :host_reports, :applied, :integer, default: 0
+    add_column :host_reports, :failed, :integer, default: 0
+    add_column :host_reports, :pending, :integer, default: 0
+    add_column :host_reports, :other, :integer, default: 0
+  end
+end

data/db/seeds.d/60-reports_feature.rb

@@ -0,0 +1,2 @@
+f = Feature.where(name: 'Reports').first_or_create
+raise "Unable to create host reports proxy feature: #{format_errors f}" if f.nil? || f.errors.any?

data/lib/foreman_host_reports/version.rb

@@ -1,3 +1,3 @@
 module ForemanHostReports
-  VERSION = '0.0.3'.freeze
+  VERSION = '0.0.4'.freeze
 end

data/package.json

@@ -5,7 +5,7 @@
   "main": "index.js",
   "scripts": {
     "lint": "tfm-lint --plugin -d /webpack",
-    "test": "tfm-test --config jest.config.js",
+    "test": "tfm-test --plugin",
     "test:watch": "tfm-test --plugin --watchAll",
     "test:current": "tfm-test --plugin --watch",
     "publish-coverage": "tfm-publish-coverage",
@@ -21,25 +21,25 @@
     "url": "http://projects.theforeman.org/projects/foreman_host_reports/issues"
   },
   "peerDependencies": {
-    "@theforeman/vendor": ">= 6.0.0"
+    "@theforeman/vendor": "^8.15.0"
   },
   "dependencies": {
-    "react-intl": "^2.8.0",
     "react-json-tree": "^0.11.0"
   },
   "devDependencies": {
     "@babel/core": "^7.7.0",
     "@sheerun/mutationobserver-shim": "^0.3.3",
-    "@theforeman/builder": "^8.4.1",
-    "@theforeman/eslint-plugin-foreman": "8.4.1",
-    "@theforeman/find-foreman": "^8.4.1",
-    "@theforeman/stories": "^8.4.1",
-    "@theforeman/test": "^8.4.1",
-    "@theforeman/vendor-dev": "^8.4.1",
+    "@theforeman/builder": "^8.15.0",
+    "@theforeman/eslint-plugin-foreman": "^8.15.0",
+    "@theforeman/find-foreman": "^8.15.0",
+    "@theforeman/stories": "^8.15.0",
+    "@theforeman/test": "^8.15.0",
+    "@theforeman/vendor-dev": "^8.15.0",
     "babel-eslint": "^10.0.3",
     "eslint": "^6.7.2",
     "prettier": "^1.19.1",
     "stylelint-config-standard": "^18.0.0",
-    "stylelint": "^9.3.0"
+    "stylelint": "^9.3.0",
+    "jed": "^1.1.1"
   }
 }

data/test/controllers/api/v2/host_reports_controller_test.rb

@@ -1,6 +1,15 @@
 require 'test_plugin_helper'
 
 class Api::V2::HostReportsControllerTest < ActionController::TestCase
+  setup do
+    @proxy = FactoryBot.create(:smart_proxy,
+      url: "http://reports.foreman.example.com",
+      features: [FactoryBot.create(:feature, name: 'Reports')])
+    Resolv.any_instance.stubs(:getnames).returns([URI.parse(@proxy.url).host])
+    SmartProxy.any_instance.stubs(:with_features).returns([@proxy])
+    ProxyAPI::V2::Features.any_instance.stubs(:features).returns({ "reports" => { "state" => "running" } })
+  end
+
   context 'when user does not have permission to view hosts' do
     let :host_report do
       as_admin { FactoryBot.create(:host_report) }
@@ -38,7 +47,7 @@ class Api::V2::HostReportsControllerTest < ActionController::TestCase
       post :create, params: {
         host_report: {
           host: host.name, body: report_body, reported_at: Time.current,
-          status: 0
+          applied: 5, failed: 1, pending: 1, other: 0
         },
       }, session: set_session_user
       assert_response :success
@@ -50,12 +59,30 @@ class Api::V2::HostReportsControllerTest < ActionController::TestCase
       assert_response :unprocessable_entity
     end
 
+    test 'store statuses' do
+      User.current = nil
+      post :create, params: {
+        host_report: {
+          host: host.name, body: report_body, reported_at: Time.current,
+          keywords: %w[HasError HasFailedResource],
+          applied: 5, failed: 1, pending: 1, other: 6
+        },
+      }, session: set_session_user
+      report = ActiveSupport::JSON.decode(@response.body)
+      assert_response :created
+      assert_equal 5, report['applied']
+      assert_equal 1, report['failed']
+      assert_equal 1, report['pending']
+      assert_equal 6, report['other']
+    end
+
     test 'assign keywords' do
       User.current = nil
       post :create, params: {
         host_report: {
           host: host.name, body: report_body, reported_at: Time.current,
-          status: 0, keywords: %w[HasError HasFailedResource]
+          keywords: %w[HasError HasFailedResource],
+          applied: 5, failed: 1, pending: 1, other: 0
         },
       }, session: set_session_user
       report = ActiveSupport::JSON.decode(@response.body)
@@ -69,14 +96,16 @@ class Api::V2::HostReportsControllerTest < ActionController::TestCase
         post :create, params: {
           host_report: {
             host: host.name, body: report_body, reported_at: Time.current,
-            status: 0, keywords: %w[HasError HasFailedResource]
+            keywords: %w[HasError HasFailedResource],
+            applied: 5, failed: 1, pending: 1, other: 0
           },
         }, session: set_session_user
 
         post :create, params: {
           host_report: {
             host: host.name, body: report_body, reported_at: Time.current,
-            status: 0, keywords: %w[HasError HasFailedResource]
+            keywords: %w[HasError HasFailedResource],
+            applied: 5, failed: 1, pending: 1, other: 0
           },
         }, session: set_session_user
       end
@@ -90,7 +119,7 @@ class Api::V2::HostReportsControllerTest < ActionController::TestCase
       post :create, params: {
         host_report: {
           host: host.name, body: report_body, reported_at: Time.current,
-          status: 0
+          applied: 5, failed: 1, pending: 1, other: 0
         },
       }
       assert_nil @controller.detected_proxy
@@ -101,18 +130,13 @@ class Api::V2::HostReportsControllerTest < ActionController::TestCase
       Setting[:restrict_registered_smart_proxies] = true
       Setting[:require_ssl_smart_proxies] = false
 
-      stub_smart_proxy_v2_features
-      proxy = smart_proxies(:puppetmaster)
-      as_admin { proxy.update_attribute(:url, 'http://configreports.foreman') }
-      proxy_host = URI.parse(proxy.url).host
-      Resolv.any_instance.stubs(:getnames).returns([proxy_host])
       post :create, params: {
         host_report: {
           host: host.name, body: report_body, reported_at: Time.current,
-          status: 0
+          applied: 5, failed: 1, pending: 1, other: 0
         },
       }
-      assert_equal proxy, @controller.detected_proxy
+      assert_equal @proxy, @controller.detected_proxy
       assert_response :created
     end
 
@@ -124,7 +148,7 @@ class Api::V2::HostReportsControllerTest < ActionController::TestCase
       post :create, params: {
         host_report: {
           host: host.name, body: report_body, reported_at: Time.current,
-          status: 0
+          applied: 5, failed: 1, pending: 1, other: 0
         },
       }
       assert_response :forbidden
@@ -135,12 +159,12 @@ class Api::V2::HostReportsControllerTest < ActionController::TestCase
       Setting[:require_ssl_smart_proxies] = true
 
       @request.env['HTTPS'] = 'on'
-      @request.env['SSL_CLIENT_S_DN'] = 'CN=else.where'
+      @request.env['SSL_CLIENT_S_DN'] = 'CN=reports.foreman.example.com'
       @request.env['SSL_CLIENT_VERIFY'] = 'SUCCESS'
       post :create, params: {
         host_report: {
           host: host.name, body: report_body, reported_at: Time.current,
-          status: 0
+          applied: 5, failed: 1, pending: 1, other: 0
         },
       }
       assert_response :created
@@ -156,7 +180,7 @@ class Api::V2::HostReportsControllerTest < ActionController::TestCase
       post :create, params: {
         host_report: {
           host: host.name, body: report_body, reported_at: Time.current,
-          status: 0
+          applied: 5, failed: 1, pending: 1, other: 0
         },
       }
       assert_response :forbidden
@@ -172,7 +196,7 @@ class Api::V2::HostReportsControllerTest < ActionController::TestCase
       post :create, params: {
         host_report: {
           host: host.name, body: report_body, reported_at: Time.current,
-          status: 0
+          applied: 5, failed: 1, pending: 1, other: 0
         },
       }
       assert_response :forbidden
@@ -187,7 +211,7 @@ class Api::V2::HostReportsControllerTest < ActionController::TestCase
       post :create, params: {
         host_report: {
           host: host.name, body: report_body, reported_at: Time.current,
-          status: 0
+          applied: 5, failed: 1, pending: 1, other: 0
         },
       }
       assert_response :forbidden
@@ -199,11 +223,10 @@ class Api::V2::HostReportsControllerTest < ActionController::TestCase
       Setting[:require_ssl_smart_proxies] = true
       SETTINGS[:require_ssl] = false
 
-      Resolv.any_instance.stubs(:getnames).returns(['else.where'])
       post :create, params: {
         host_report: {
           host: host.name, body: report_body, reported_at: Time.current,
-          status: 0
+          applied: 5, failed: 1, pending: 1, other: 0
         },
       }
       assert_response :created

data/test/factories/foreman_host_reports_factories.rb

@@ -2,7 +2,10 @@ FactoryBot.define do
   factory :host_report do
     host
     reported_at { Time.now.utc }
-    status { 0 }
+    applied { 0 }
+    failed { 0 }
+    pending { 0 }
+    other { 0 }
     body { 'report data' }
   end
 

data/webpack/__mocks__/foremanReact/common/HOC.js

@@ -0,0 +1,30 @@
+import React, { useEffect, useRef } from 'react';
+
+export const withRenderHandler = ({ Component }) => componentProps => (
+  <Component {...componentProps} />
+);
+
+export const callOnMount = callback => WrappedComponent => componentProps => {
+  callback(componentProps);
+
+  return <WrappedComponent {...componentProps} />;
+};
+
+export const callOnPopState = callback => WrappedComponent => componentProps => {
+  const didMount = useRef(false);
+  const {
+    history: {
+      action,
+      location: { search },
+    },
+  } = componentProps;
+  useEffect(() => {
+    if (action === 'POP' && didMount.current) {
+      callback(componentProps);
+    } else {
+      didMount.current = true;
+    }
+  }, [search, action, componentProps]);
+
+  return <WrappedComponent {...componentProps} />;
+};

data/webpack/__mocks__/foremanReact/common/I18n.js

@@ -0,0 +1,7 @@
+export { sprintf } from 'jed';
+
+export const translate = s => s;
+
+export const ngettext = s => s;
+
+export const documentLocale = () => 'en';

data/webpack/__mocks__/foremanReact/common/helpers.js

@@ -0,0 +1,7 @@
+export const getURIQuery = jest.fn(() => ({}));
+
+export const deepPropsToCamelCase = jest.fn(props => props);
+
+export const foremanUrl = jest.fn(() => '');
+
+export const noop = Function.prototype;

data/webpack/__mocks__/foremanReact/common/urlHelpers.js

@@ -0,0 +1,8 @@
+export const getURIsearch = () => 'a=b';
+
+export const getParams = () => ({
+  page: 1,
+  perPage: 20,
+  searchQuery: '',
+  sort: {},
+});

data/webpack/__mocks__/foremanReact/components/ForemanModal/ForemanModalActions.js

@@ -0,0 +1,2 @@
+const ForemanModalActions = () => jest.fn();
+export default ForemanModalActions;

data/webpack/__mocks__/foremanReact/components/ForemanModal/ForemanModalHooks.js

@@ -0,0 +1,10 @@
+const modalOpen = true;
+const setModalOpen = jest.fn();
+const setModalClosed = jest.fn();
+
+export const useForemanModal = jest.fn(() => ({
+  modalOpen,
+  setModalOpen,
+  setModalClosed,
+}));
+export default useForemanModal;

data/webpack/__mocks__/foremanReact/components/ForemanModal/index.js

@@ -0,0 +1,23 @@
+import React from 'react';
+import PropTypes from 'prop-types';
+
+const ForemanModal = ({ children }) => <div className="modal">{children}</div>;
+ForemanModal.Header = ({ children }) => (
+  <div className="modal-header">{children}</div>
+);
+ForemanModal.Footer = ({ children }) => (
+  <div className="modal-footer">{children}</div>
+);
+
+ForemanModal.propTypes = {
+  children: PropTypes.node,
+};
+
+ForemanModal.defaultProps = {
+  children: [],
+};
+
+ForemanModal.Header.propTypes = ForemanModal.propTypes;
+ForemanModal.Footer.propTypes = ForemanModal.propTypes;
+
+export default ForemanModal;

data/webpack/__mocks__/foremanReact/components/Pagination/PaginationWrapper.js

@@ -0,0 +1,4 @@
+import React from 'react';
+
+const PaginationWrapper = () => <></>;
+export default PaginationWrapper;

data/webpack/__mocks__/foremanReact/components/common/EmptyState.js

@@ -0,0 +1,5 @@
+import React from 'react';
+
+export const DefaultEmptyState = () => (
+  <div className="empty-state-description" />
+);

data/webpack/__mocks__/foremanReact/components/common/dates/RelativeDateTime.js

@@ -0,0 +1,7 @@
+import React from 'react';
+
+const RelativeDateTime = (date, defaultValue, context) => (
+  <span>{defaultValue || date}</span>
+);
+
+export default RelativeDateTime;

data/webpack/__mocks__/foremanReact/components/common/forms/ForemanForm.js

@@ -0,0 +1,9 @@
+import React from 'react';
+
+const ForemanForm = () => (
+  <React.Fragment>
+    <form />
+  </React.Fragment>
+);
+
+export default ForemanForm;

data/webpack/__mocks__/foremanReact/components/common/forms/FormField.js

@@ -0,0 +1,3 @@
+import React from 'react';
+
+export const ForemanField = props => <div />;

data/webpack/__mocks__/foremanReact/components/common/table.js

@@ -0,0 +1,26 @@
+import React from 'react';
+
+export const Table = () => <div className="table" />;
+export const createTableReducer = jest.fn(controller => controller);
+export const cellFormatter = cell => cell;
+export const deleteActionCellFormatter = cell => cell;
+export const sortableColumn = jest.fn();
+export const column = (
+  property,
+  label,
+  headFormat,
+  cellFormat,
+  headProps = {},
+  cellProps = {}
+) => ({
+  property,
+  header: {
+    label,
+    props: headProps,
+    formatters: headFormat,
+  },
+  cell: {
+    props: cellProps,
+    formatters: cellFormat,
+  },
+});

data/webpack/__mocks__/foremanReact/constants.js

@@ -0,0 +1,24 @@
+export const STATUS = {
+  PENDING: 'PENDING',
+  RESOLVED: 'RESOLVED',
+  ERROR: 'ERROR',
+};
+
+export const getControllerSearchProps = (
+  controller,
+  id = 'searchBar',
+  canCreateBookmarks = true
+) => ({
+  controller,
+  autocomplete: {
+    id,
+    searchQuery: '',
+    url: `${controller}/auto_complete_search`,
+    useKeyShortcuts: true,
+  },
+  bookmarks: {
+    url: '/api/bookmarks',
+    canCreateBookmarks,
+    documentationUrl: `4.1.5Searching`,
+  },
+});

data/webpack/__mocks__/foremanReact/redux/API/APISelectors.js

@@ -0,0 +1,6 @@
+export const selectAPIResponse = (state, key) =>
+  selectAPIByKey(state, key).response;
+
+export const selectAPIStatus = (state, key) => 'PENDING';
+export const selectAPIByKey = (state, key) => state.API[key];
+export const selectAPIError = (state, key) => ({ error: `${key} ERRROR` });

data/webpack/__mocks__/foremanReact/redux/API/index.js

@@ -0,0 +1,10 @@
+export const API = {
+  get: jest.fn(),
+  put: jest.fn(),
+  post: jest.fn(),
+  delete: jest.fn(),
+  patch: jest.fn(),
+};
+
+export const get = data => ({ type: 'get-some-type', ...data });
+export const post = data => ({ type: 'post-some-type', ...data });

data/webpack/__mocks__/foremanReact/redux/actions/common/forms.js

@@ -0,0 +1 @@
+export const submitForm = jest.fn(() => '');

data/webpack/__mocks__/foremanReact/redux/actions/toasts.js

@@ -0,0 +1,8 @@
+export const addToast = toast => ({
+  type: 'TOASTS_ADD',
+  payload: {
+    message: toast,
+  },
+});
+
+export default addToast;

data/webpack/__mocks__/foremanReact/routes/common/PageLayout/PageLayout.js

@@ -0,0 +1,10 @@
+import React from 'react';
+import PropTypes from 'prop-types';
+
+const PageLayout = ({ children }) => <div>{children}</div>;
+
+PageLayout.propTypes = {
+  children: PropTypes.node.isRequired,
+};
+
+export default PageLayout;

data/webpack/__mocks__/foremanReact/routes/common/PageLayout/components/ExportButton/ExportButton.js

@@ -0,0 +1,10 @@
+import React from 'react';
+import { Button } from 'patternfly-react';
+
+const ExportButton = props => (
+  <Button className="export-csv" href="#" title="Export">
+    Export
+  </Button>
+);
+
+export default ExportButton;

data/webpack/src/Router/HostReports/IndexPage/Components/HostReportsTable/Components/Formatters/statusFormatter.js

@@ -2,8 +2,14 @@ import React from 'react';
 
 import StatusCell from '../StatusCell';
 
-const statusFormatter = () => (value, { rowData: { format, status } }) => (
-  <StatusCell format={format} value={value} />
-);
+const statusFormatter = () => (_, { rowData }) => {
+  const statuses = {
+    applied: rowData.applied,
+    failed: rowData.failed,
+    pending: rowData.pending,
+    other: rowData.other,
+  };
+  return <StatusCell statuses={statuses} />;
+};
 
 export default statusFormatter;

data/webpack/src/Router/HostReports/IndexPage/Components/HostReportsTable/Components/StatusCell.js

@@ -4,74 +4,32 @@ import { capitalize } from 'lodash';
 
 import './StatusCell.scss';
 
-const StatusCell = ({ format, value }) => {
-  switch (format) {
-    case 'puppet':
-      return (
-        <td>
-          <ul className="status-list">
-            {value.map(status => {
-              let style = '';
-              switch (status[0]) {
-                case 'failed':
-                  style = 'label-danger';
-                  break;
-                case 'failed_to_restart':
-                  style = 'label-warning';
-                  break;
-                default:
-                  style = 'label-info';
-              }
-              if (!status[2]) style = 'label-default';
-              return (
-                <li key={status[0]}>
-                  {`${status[1]}: `}
-                  <span className={`label ${style}`}>{status[2]}</span>
-                </li>
-              );
-            })}
-          </ul>
-        </td>
-      );
-    case 'ansible':
-      return (
-        <td>
-          <ul className="status-list">
-            {Object.keys(value).map(status => {
-              let style = '';
-              switch (status) {
-                case 'failed':
-                  style = 'label-danger';
-                  break;
-                case 'skipped':
-                  style = 'label-warning';
-                  break;
-                default:
-                  style = 'label-info';
-              }
-              if (!value[status]) style = 'label-default';
-              return (
-                <li key={status}>
-                  {`${capitalize(status)}: `}
-                  <span className={`label ${style}`}>{value[status]}</span>
-                </li>
-              );
-            })}
-          </ul>
-        </td>
-      );
-    default:
-      return <td>N/A</td>;
-  }
-};
+const StatusCell = ({ statuses }) => (
+  <td>
+    <ul className="status-list">
+      {Object.keys(statuses).map(status => {
+        let style = '';
+        switch (status) {
+          case 'failed':
+            style = 'label-danger';
+            break;
+          default:
+            style = 'label-info';
+        }
+        if (!statuses[status]) style = 'label-default';
+        return (
+          <li key={status}>
+            {`${capitalize(status)}: `}
+            <span className={`label ${style}`}>{statuses[status]}</span>
+          </li>
+        );
+      })}
+    </ul>
+  </td>
+);
 
 StatusCell.propTypes = {
-  format: PropTypes.string,
-  value: PropTypes.any.isRequired,
-};
-
-StatusCell.defaultProps = {
-  format: 'plain',
+  statuses: PropTypes.object.isRequired,
 };
 
 export default StatusCell;

data/webpack/src/Router/HostReports/IndexPage/__tests__/HostReportsIndexPage.test.js

@@ -0,0 +1,40 @@
+import { testComponentSnapshotsWithFixtures } from '@theforeman/test';
+import HostReportsIndexPage from '../IndexPage';
+
+const fixtures = {
+  'render with minimal props': {
+    search: '',
+    history: { location: { search: '' } },
+    hostId: '1',
+    fetchAndPush: jest.fn(),
+    reloadWithSearch: jest.fn(),
+    isLoading: false,
+    hasError: false,
+    hasData: true,
+    itemCount: 1,
+    canCreate: true,
+    sort: {},
+    page: 1,
+    perPage: 20,
+    reports: [
+      {
+        id: '1',
+        hostId: '1',
+        hostName: 'foreman.example.com',
+        proxyId: '1',
+        proxyName: 'foreman.example.com',
+        format: 'plain',
+        reportedAt: '2021-01-19T12:34:02.841645028Z',
+        applied: 0,
+        failed: 0,
+        pending: 0,
+        other: 0,
+      },
+    ],
+  },
+};
+
+describe('HostReportsIndexPage', () => {
+  describe('redering', () =>
+    testComponentSnapshotsWithFixtures(HostReportsIndexPage, fixtures));
+});

data/webpack/src/Router/HostReports/IndexPage/__tests__/__snapshots__/HostReportsIndexPage.test.js.snap

@@ -0,0 +1,67 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`HostReportsIndexPage redering render with minimal props 1`] = `
+<PageLayout
+  header="Host Reports"
+  isLoading={false}
+  onBookmarkClick={[MockFunction]}
+  onSearch={[MockFunction]}
+  searchProps={
+    Object {
+      "autocomplete": Object {
+        "id": "searchBar",
+        "searchQuery": "",
+        "url": "host_reports/auto_complete_search",
+        "useKeyShortcuts": true,
+      },
+      "bookmarks": Object {
+        "canCreateBookmarks": true,
+        "documentationUrl": "4.1.5Searching",
+        "url": "/api/bookmarks",
+      },
+      "controller": "host_reports",
+    }
+  }
+  searchQuery=""
+  searchable={true}
+  toolbarButtons={
+    <ExportButton
+      title="Export"
+      url="/api/v2/host_reports/export"
+    />
+  }
+>
+  <WrappedHostReportsTable
+    fetchAndPush={[MockFunction]}
+    hostId="1"
+    itemCount={1}
+    pagination={
+      Object {
+        "page": 1,
+        "perPage": 20,
+      }
+    }
+    reloadWithSearch={[MockFunction]}
+    results={
+      Array [
+        Object {
+          "applied": 0,
+          "failed": 0,
+          "format": "plain",
+          "hostId": "1",
+          "hostName": "foreman.example.com",
+          "id": "1",
+          "other": 0,
+          "pending": 0,
+          "proxyId": "1",
+          "proxyName": "foreman.example.com",
+          "reportedAt": "2021-01-19T12:34:02.841645028Z",
+        },
+      ]
+    }
+    setToDelete={[Function]}
+    sort={Object {}}
+    toDelete={Object {}}
+  />
+</PageLayout>
+`;

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: foreman_host_reports
 version: !ruby/object:Gem::Version
-  version: 0.0.3
+  version: 0.0.4
 platform: ruby
 authors:
 - Lukas Zapletal
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-10-18 00:00:00.000000000 Z
+date: 2021-10-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rdoc
@@ -48,6 +48,8 @@ files:
 - config/routes.rb
 - db/migrate/20210112183526_add_host_reports.rb
 - db/migrate/20210616133601_create_report_keywords.rb
+- db/migrate/20211011141813_change_status_column.rb
+- db/seeds.d/60-reports_feature.rb
 - lib/foreman_host_reports.rb
 - lib/foreman_host_reports/engine.rb
 - lib/foreman_host_reports/version.rb
@@ -62,6 +64,26 @@ files:
 - test/snapshots/foreman-web.json
 - test/test_plugin_helper.rb
 - test/unit/foreman_host_reports_test.rb
+- webpack/__mocks__/foremanReact/common/HOC.js
+- webpack/__mocks__/foremanReact/common/I18n.js
+- webpack/__mocks__/foremanReact/common/helpers.js
+- webpack/__mocks__/foremanReact/common/urlHelpers.js
+- webpack/__mocks__/foremanReact/components/ForemanModal/ForemanModalActions.js
+- webpack/__mocks__/foremanReact/components/ForemanModal/ForemanModalHooks.js
+- webpack/__mocks__/foremanReact/components/ForemanModal/index.js
+- webpack/__mocks__/foremanReact/components/Pagination/PaginationWrapper.js
+- webpack/__mocks__/foremanReact/components/common/EmptyState.js
+- webpack/__mocks__/foremanReact/components/common/dates/RelativeDateTime.js
+- webpack/__mocks__/foremanReact/components/common/forms/ForemanForm.js
+- webpack/__mocks__/foremanReact/components/common/forms/FormField.js
+- webpack/__mocks__/foremanReact/components/common/table.js
+- webpack/__mocks__/foremanReact/constants.js
+- webpack/__mocks__/foremanReact/redux/API/APISelectors.js
+- webpack/__mocks__/foremanReact/redux/API/index.js
+- webpack/__mocks__/foremanReact/redux/actions/common/forms.js
+- webpack/__mocks__/foremanReact/redux/actions/toasts.js
+- webpack/__mocks__/foremanReact/routes/common/PageLayout/PageLayout.js
+- webpack/__mocks__/foremanReact/routes/common/PageLayout/components/ExportButton/ExportButton.js
 - webpack/global_index.js
 - webpack/global_test_setup.js
 - webpack/index.js
@@ -87,6 +109,8 @@ files:
 - webpack/src/Router/HostReports/IndexPage/IndexPageActions.js
 - webpack/src/Router/HostReports/IndexPage/IndexPageHelpers.js
 - webpack/src/Router/HostReports/IndexPage/IndexPageSelectors.js
+- webpack/src/Router/HostReports/IndexPage/__tests__/HostReportsIndexPage.test.js
+- webpack/src/Router/HostReports/IndexPage/__tests__/__snapshots__/HostReportsIndexPage.test.js.snap
 - webpack/src/Router/HostReports/IndexPage/constants.js
 - webpack/src/Router/HostReports/IndexPage/index.js
 - webpack/src/Router/HostReports/ShowPage/Components/HostReportMetrics/HostReportMetrics.scss
@@ -101,7 +125,6 @@ files:
 - webpack/src/Router/HostReports/ShowPage/index.js
 - webpack/src/Router/HostReports/constants.js
 - webpack/src/Router/routes.js
-- webpack/test_setup.js
 homepage: https://github.com/theforeman/foreman_host_reports
 licenses:
 - GPL-3.0
@@ -131,3 +154,4 @@ test_files:
 - test/controllers/api/v2/host_reports_controller_test.rb
 - test/snapshots/foreman-web.json
 - test/test_plugin_helper.rb
+- webpack/src/Router/HostReports/IndexPage/__tests__/HostReportsIndexPage.test.js

data/webpack/test_setup.js

@@ -1,17 +0,0 @@
-import 'core-js/shim';
-import 'regenerator-runtime/runtime';
-import MutationObserver from '@sheerun/mutationobserver-shim';
-
-import { configure } from 'enzyme';
-import Adapter from 'enzyme-adapter-react-16';
-
-configure({ adapter: new Adapter() });
-
-// Mocking translation function
-global.__ = text => text; // eslint-disable-line
-
-// Mocking locales to prevent unnecessary fallback messages
-window.locales = { en: { domain: 'app', locale_data: { app: { '': {} } } } };
-
-// see https://github.com/testing-library/dom-testing-library/releases/tag/v7.0.0
-window.MutationObserver = MutationObserver;