foreman_statistics 0.1.1 → 0.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ba423682698624b7d224e67b892a00982dfadb9ba225c3431212fda28a8665bd
-  data.tar.gz: abfbe9e5304f233e205438219fffc9d6ad7b7e8e5bcc9a041c54cb2a344848a2
+  metadata.gz: c2535d7ffe5c9d9e7d594d61a1ed6e5fbed330c3b5739a3792ce78910c365ef2
+  data.tar.gz: 8be508f7afacb9d168013e7d5a101e57dd775107f4d3aa12211b252d5a3bfe94
 SHA512:
-  metadata.gz: 8bbe0bbeaedc311bca4f88dfbd91d7475432c4ed31c4805e34db9c8340c23a537bcbe67c6eba24cda22cf1a84505cee3cfc2509cc2aec6eefab45f701ddec04f
-  data.tar.gz: 22444861f80d94fd85281af4a03420defef52336e006841f6e647a17008dacbdf09d9b24675e3399803c73163348c8f8f8564e496458bd99748da2db521395d0
+  metadata.gz: 1849bc62e6be28e6572727e87afaeba9c9e803643d0de003a0d903a81ba3b5a6a964755a1ebc734054ab1e09cd5116135b34d5392a2eb41e653d956dac1b9973
+  data.tar.gz: 0fa5e7507dcc8d5e75755cdbd8fd7e70195e1bc1fa37c037c0d00740da2771446fec24b49dbbed50dfe4b7d6e7d45a7c6386e5261cd5329e49bffac0808f8946

data/app/controllers/foreman_statistics/api/v2/statistics_controller.rb

@@ -2,13 +2,14 @@ module ForemanStatistics
   module Api
     module V2
       class StatisticsController < ::Api::V2::BaseController
+        before_action :show_deprecation_for_core_routes
+
         resource_description do
           api_version 'v2'
           api_base_url '/foreman_statistics/api'
         end
 
         api :GET, '/statistics/', N_('Get statistics')
-
         def index
           @os_count    = Host.authorized(:view_hosts).count_distribution :operatingsystem
           @arch_count  = Host.authorized(:view_hosts).count_distribution :architecture
@@ -27,6 +28,13 @@ module ForemanStatistics
                             :model_count => @model_count, :mem_size => @mem_size, :mem_free => @mem_free,
                             :swap_free => @swap_free, :mem_totsize => @mem_totsize, :mem_totfree => @mem_totfree }
         end
+
+        private
+
+        def show_deprecation_for_core_routes
+          return if request.path.starts_with?('/foreman_statistics')
+          Foreman::Deprecation.api_deprecation_warning('/api/v2/statistics API endpoint is deprecated, please use /foreman_statistics/api/v2/statistics instead')
+        end
       end
     end
   end

data/app/controllers/foreman_statistics/api/v2/trends_controller.rb

@@ -4,6 +4,7 @@ module ForemanStatistics
       class TrendsController < ::Api::V2::BaseController
         include ForemanStatistics::Parameters::Trend
 
+        before_action :show_deprecation_for_core_routes
         before_action :find_resource, :only => %i[show destroy]
 
         TRENDABLE_TYPES = %w[
@@ -52,6 +53,13 @@ module ForemanStatistics
         def resource_scope(options = {})
           @resource_scope ||= scope_for(ForemanStatistics::Trend.types, options)
         end
+
+        private
+
+        def show_deprecation_for_core_routes
+          return if request.path.starts_with?('/foreman_statistics')
+          Foreman::Deprecation.api_deprecation_warning('/api/v2/trends API endpoints are deprecated, please use /foreman_statistics/api/v2/trends instead')
+        end
       end
     end
   end

data/db/migrate/20200605153005_migrate_core_types.rb

@@ -1,15 +1,19 @@
 class MigrateCoreTypes < ActiveRecord::Migration[6.0]
+  class FakeTrend < ApplicationRecord
+    self.table_name = 'trends'
+  end
+
   def up
     Permission.where(:resource_type => 'Trend').update_all(:resource_type => 'ForemanStatistics::Trend')
     %w[ForemanTrend FactTrend Trend].each do |t|
-      Trend.where(:type => t).update_all(:type => "ForemanStatistics::#{t}")
+      FakeTrend.where(:type => t).update_all(:type => "ForemanStatistics::#{t}")
     end
   end
 
   def down
     Permission.where(:resource_type => 'ForemanStatistics::Trend').update_all(:resource_type => 'Trend')
     %w[ForemanTrend FactTrend Trend].each do |t|
-      Trend.where(:type => "ForemanStatistics::#{t}").update_all(:type => t)
+      FakeTrend.where(:type => "ForemanStatistics::#{t}").update_all(:type => t)
     end
   end
 end

data/lib/foreman_statistics/engine.rb

@@ -8,7 +8,7 @@ module ForemanStatistics
     config.autoload_paths += Dir["#{config.root}/app/models/concerns"]
     config.autoload_paths += Dir["#{config.root}/app/overrides"]
 
-    config.paths['db/migrate'] << 'db/migrate_foreman' if Gem::Dependency.new('', '>= 2.2').match?('', SETTINGS[:version])
+    config.paths['db/migrate'] << 'db/migrate_foreman' unless Gem::Version.new(SETTINGS[:version]).release < Gem::Version.new('2.2')
 
     # Add any db migrations
     initializer 'foreman_statistics.load_app_instance_data' do |app|

data/lib/foreman_statistics/version.rb

@@ -1,3 +1,3 @@
 module ForemanStatistics
-  VERSION = '0.1.1'.freeze
+  VERSION = '0.1.2'.freeze
 end

data/webpack/__mocks__/foremanReact/API.js

@@ -0,0 +1,7 @@
+export default {
+  get: jest.fn(),
+  put: jest.fn(),
+  post: jest.fn(),
+  delete: jest.fn(),
+  patch: jest.fn(),
+};

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

@@ -0,0 +1,24 @@
+import React, { useEffect } from 'react';
+
+export const callOnMount = callback => WrappedComponent => componentProps => {
+  // fires callback onMount, [] means don't listen to any props change
+  useEffect(() => {
+    callback(componentProps);
+  }, [componentProps]);
+
+  return <WrappedComponent {...componentProps} />;
+};
+
+export const withRenderHandler = ({
+  Component,
+  LoadingComponent = () => jest.fn(),
+  ErrorComponent = () => jest.fn(),
+  EmptyComponent = () => jest.fn(),
+}) => componentProps => {
+  const { isLoading, hasData, hasError } = componentProps;
+
+  if (isLoading && !hasData) return <LoadingComponent {...componentProps} />;
+  if (hasError) return <ErrorComponent {...componentProps} />;
+  if (hasData) return <Component {...componentProps} />;
+  return <EmptyComponent {...componentProps} />;
+};

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

@@ -0,0 +1,3 @@
+export const translate = s => s;
+
+export const documentLocale = () => 'en';

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

@@ -0,0 +1 @@
+export const noop = Function.prototype;

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

@@ -0,0 +1 @@
+export const getURIsearch = () => 'a=b';

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

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

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 = () => ({
+  modalOpen,
+  setModalOpen,
+  setModalClosed,
+});
+export default useForemanModal;

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

@@ -0,0 +1,4 @@
+const ForemanModal = () => jest.fn();
+ForemanModal.Header = () => jest.fn();
+ForemanModal.Footer = () => jest.fn();
+export default ForemanModal;

data/webpack/__mocks__/foremanReact/components/Layout/LayoutActions.js

@@ -0,0 +1,2 @@
+export const showLoading = () => null;
+export const hideLoading = () => null;

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

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

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

@@ -0,0 +1,5 @@
+const EmptyState = () => jest.fn();
+
+export const EmptyStatePattern = () => jest.fn();
+
+export default EmptyState;

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

@@ -0,0 +1,4 @@
+import React from 'react';
+
+export const MessageBox = () => <div className="message-box-root" />;
+export default MessageBox;

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

@@ -0,0 +1,5 @@
+import React from 'react';
+
+export const LongDateTime = value => <p>{value}</p>;
+
+export default LongDateTime;

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

@@ -0,0 +1,3 @@
+import React from 'react';
+
+export default date => <p>{`${date} time ago`}</p>;

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

@@ -0,0 +1,5 @@
+import React from 'react';
+
+export const Table = () => <div className="table" />;
+export const createTableReducer = jest.fn(controller => controller);
+export const cellFormatter = cell => cell;

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

@@ -0,0 +1,7 @@
+const createTableActionTypes = tableID => ({
+  REQUEST: `${tableID.toUpperCase()}_REQUEST`,
+  SUCCESS: `${tableID.toUpperCase()}_SUCCESS`,
+  FAILURE: `${tableID.toUpperCase()}_FAILURE`,
+});
+
+export default createTableActionTypes;

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/readme.md

@@ -0,0 +1,11 @@
+For testing components which have imported foreman-core components,
+a mock file is required in this folder.
+
+### Example: Mocking ForemanModal component
+```js
+//  __mocks__/foremanReact/components/ForemanModal/index.js
+const ForemanModal = () => jest.fn();
+ForemanModal.Header = () => jest.fn();
+ForemanModal.Footer = () => jest.fn();
+export default ForemanModal;
+```

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,5 @@
+import React from 'react';
+
+const ExportButton = () => <button>export</button>;
+
+export default ExportButton;

data/webpack/__mocks__/foremanReact/routes/common/reducerHOC/withDataReducer.js

@@ -0,0 +1,35 @@
+import Immutable from 'seamless-immutable';
+
+const initialState = Immutable({
+  isLoading: true,
+  hasError: false,
+  hasData: false,
+  message: { type: 'empty', text: '' },
+});
+
+const withDataReducer = controller => (
+  state = initialState,
+  { type, payload }
+) => {
+  switch (type) {
+    case `${controller}_DATA_RESOLVED`:
+      return state.merge({ ...payload, isLoading: false });
+
+    case `${controller}_DATA_FAILED`:
+      return state.merge({ ...payload, isLoading: false, hasError: true });
+
+    case `${controller}_CLEAR_ERROR`:
+      return state.set('hasError', false);
+
+    case `${controller}_SHOW_LOADING`:
+      return state.set('isLoading', true);
+
+    case `${controller}_HIDE_LOADING`:
+      return state.set('isLoading', false);
+
+    default:
+      return state;
+  }
+};
+
+export default withDataReducer;

data/webpack/fills_index.js

@@ -0,0 +1,15 @@
+// This example for extanding foreman-core's component via slot&fill
+
+/*
+import React from 'react';
+import { addGlobalFill } from 'foremanReact/components/common/Fill/GlobalFill';
+
+addGlobalFill('slotId', 'fillId', <SomeComponent key="some-key" />, 300);
+
+addGlobalFill(
+  'slotId',
+  'fillId',
+  { someProp: 'this is an override prop' },
+  300
+);
+*/

data/webpack/index.js

@@ -0,0 +1,21 @@
+/* eslint import/no-unresolved: [2, { ignore: [foremanReact/*] }] */
+/* eslint-disable import/no-extraneous-dependencies */
+/* eslint-disable import/extensions */
+import componentRegistry from 'foremanReact/components/componentRegistry';
+import { registerReducer } from 'foremanReact/common/MountingService';
+import reducers from './src/reducers';
+import ForemanStatistics from './src/ForemanStatistics';
+import * as trends from './src/trends';
+
+Object.assign(window.tfm, { trends });
+
+// register reducers
+Object.entries(reducers).forEach(([key, reducer]) =>
+  registerReducer(key, reducer)
+);
+
+// register components for erb mounting
+componentRegistry.register({
+  name: 'ForemanStatistics',
+  type: ForemanStatistics,
+});

data/webpack/src/Components/StatisticsChartsList/StatisticsChartsList.fixtures.js

@@ -0,0 +1,19 @@
+export const statisticsData = {
+  operatingsystem: {
+    id: 'operatingsystem',
+    title: 'OS Distribution',
+    url: 'statistics/operatingsystem',
+    search: '/hosts?search=os_title=~VAL~',
+  },
+  architecture: {
+    id: 'architecture',
+    title: 'Architecture Distribution',
+    url: 'statistics/architecture',
+    search: '/hosts?search=facts.architecture=~VAL~',
+  },
+};
+
+export const statisticsMeta = [
+  statisticsData.operatingsystem,
+  statisticsData.architecture,
+];

data/webpack/src/Components/StatisticsChartsList/StatisticsChartsList.test.js

@@ -0,0 +1,18 @@
+import { testComponentSnapshotsWithFixtures } from '@theforeman/test';
+
+import StatisticsChartsList from './';
+import { statisticsData } from './StatisticsChartsList.fixtures';
+
+const fixtures = {
+  'should render no panels for empty data': {
+    data: {},
+  },
+  'should render two panels for fixtures data': {
+    data: statisticsData,
+  },
+};
+
+describe('StatisticsChartsList', () => {
+  describe('rendering', () =>
+    testComponentSnapshotsWithFixtures(StatisticsChartsList, fixtures));
+});

data/webpack/src/Components/StatisticsChartsList/StatisticsChartsListStyles.scss

@@ -0,0 +1,28 @@
+.statistics-charts-list-root {
+  display: flex;
+  flex-wrap: wrap;
+  justify-content: space-around;
+  padding-top: 20px;
+  .chart-box {
+    height: 312px;
+    width: 280px;
+    display: -ms-flexbox;
+    display: flex;
+    -ms-flex-direction: column;
+    flex-direction: column;
+    min-height: 312px;
+
+    .card-pf-heading {
+      -ms-flex: 0 0 30px;
+      flex: 0 0 30px;
+    }
+
+    .card-pf-body {
+      display: -ms-flexbox;
+      display: flex;
+      -ms-flex: 1;
+      flex: 1;
+      margin: 0;
+    }
+  }
+}

data/webpack/src/Components/StatisticsChartsList/__snapshots__/StatisticsChartsList.test.js.snap

@@ -0,0 +1,42 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`StatisticsChartsList rendering should render no panels for empty data 1`] = `
+<div
+  className="statistics-charts-list-root cards-pf"
+/>
+`;
+
+exports[`StatisticsChartsList rendering should render two panels for fixtures data 1`] = `
+<div
+  className="statistics-charts-list-root cards-pf"
+>
+  <ConnectedChartBox
+    chart={
+      Object {
+        "id": "operatingsystem",
+        "search": "/hosts?search=os_title=~VAL~",
+        "title": "OS Distribution",
+        "url": "statistics/operatingsystem",
+      }
+    }
+    key="operatingsystem"
+    noDataMsg="No data available"
+    tip="Expand the chart"
+    type="donut"
+  />
+  <ConnectedChartBox
+    chart={
+      Object {
+        "id": "architecture",
+        "search": "/hosts?search=facts.architecture=~VAL~",
+        "title": "Architecture Distribution",
+        "url": "statistics/architecture",
+      }
+    }
+    key="architecture"
+    noDataMsg="No data available"
+    tip="Expand the chart"
+    type="donut"
+  />
+</div>
+`;

data/webpack/src/Components/StatisticsChartsList/index.js

@@ -0,0 +1,32 @@
+import React from 'react';
+import PropTypes from 'prop-types';
+
+import { translate as __ } from 'foremanReact/common/I18n';
+import ConnectedChartBox from 'foremanReact/components/ChartBox';
+import './StatisticsChartsListStyles.scss';
+
+const StatisticsChartsList = ({ data }) => {
+  const chartBoxes = Object.values(data).map(chart => (
+    <ConnectedChartBox
+      key={chart.id}
+      type="donut"
+      chart={chart}
+      noDataMsg={__('No data available')}
+      tip={__('Expand the chart')}
+    />
+  ));
+
+  return (
+    <div className="statistics-charts-list-root cards-pf">{chartBoxes}</div>
+  );
+};
+
+StatisticsChartsList.propTypes = {
+  data: PropTypes.oneOfType([PropTypes.array, PropTypes.object]),
+};
+
+StatisticsChartsList.defaultProps = {
+  data: [],
+};
+
+export default StatisticsChartsList;

data/webpack/src/ForemanStatistics.js

@@ -0,0 +1,11 @@
+import React from 'react';
+import { BrowserRouter } from 'react-router-dom';
+import ForemanStatisticsRoute from './Router';
+
+const ForemanStatistics = () => (
+  <BrowserRouter>
+    <ForemanStatisticsRoute />
+  </BrowserRouter>
+);
+
+export default ForemanStatistics;

data/webpack/src/Router/StatisticsPage/Statistics/Statistics.js

@@ -0,0 +1,27 @@
+import React from 'react';
+import PropTypes from 'prop-types';
+import { withRenderHandler } from 'foremanReact/common/HOC';
+import { translate as __ } from 'foremanReact/common/I18n';
+import { EmptyStatePattern } from 'foremanReact/components/common/EmptyState';
+import StatisticsChartsList from '../../../Components/StatisticsChartsList';
+
+const Statistics = ({ statisticsMeta }) => {
+  if (!statisticsMeta) {
+    return (
+      <EmptyStatePattern
+        icon="info"
+        header={__('No Charts To Load')}
+        description=""
+      />
+    );
+  }
+  return <StatisticsChartsList data={statisticsMeta} />;
+};
+
+Statistics.propTypes = {
+  statisticsMeta: PropTypes.array.isRequired,
+};
+
+export default withRenderHandler({
+  Component: Statistics,
+});

data/webpack/src/Router/StatisticsPage/StatisticsPage.fixtures.js

@@ -0,0 +1,11 @@
+import { noop } from 'foremanReact/common/helpers';
+import { statisticsMeta } from '../../Components/StatisticsChartsList/StatisticsChartsList.fixtures';
+
+export const statisticsProps = {
+  statisticsMeta,
+  isLoading: false,
+  hasData: true,
+  hasError: false,
+  message: {},
+  getStatisticsMeta: noop,
+};

data/webpack/src/Router/StatisticsPage/StatisticsPage.js

@@ -0,0 +1,21 @@
+import React from 'react';
+import PropTypes from 'prop-types';
+import { translate as __ } from 'foremanReact/common/I18n';
+import PageLayout from 'foremanReact/routes/common/PageLayout/PageLayout';
+import Statistics from './Statistics/Statistics';
+
+const StatisticsPage = ({ statisticsMeta, ...props }) => (
+  <PageLayout header={__('Statistics')} searchable={false}>
+    <Statistics statisticsMeta={statisticsMeta} {...props} />
+  </PageLayout>
+);
+
+StatisticsPage.propTypes = {
+  statisticsMeta: PropTypes.array,
+};
+
+StatisticsPage.defaultProps = {
+  statisticsMeta: [],
+};
+
+export default StatisticsPage;

data/webpack/src/Router/StatisticsPage/StatisticsPageActions.js

@@ -0,0 +1,43 @@
+import API from 'foremanReact/API';
+
+import {
+  STATISTICS_PAGE_DATA_RESOLVED,
+  STATISTICS_PAGE_DATA_FAILED,
+  STATISTICS_PAGE_HIDE_LOADING,
+  STATISTICS_PAGE_URL,
+} from './constants';
+
+export const getStatisticsMeta = (
+  url = STATISTICS_PAGE_URL
+) => async dispatch => {
+  const onFetchSuccess = ({ data }) => {
+    dispatch(hideLoading());
+    dispatch({
+      type: STATISTICS_PAGE_DATA_RESOLVED,
+      payload: { metadata: data.charts, hasData: data.charts.length > 0 },
+    });
+  };
+
+  const onFetchError = ({ message }) => {
+    dispatch(hideLoading());
+    dispatch({
+      type: STATISTICS_PAGE_DATA_FAILED,
+      payload: {
+        message: {
+          type: 'error',
+          text: message,
+        },
+      },
+    });
+  };
+  try {
+    const response = await API.get(url);
+    return onFetchSuccess(response);
+  } catch (error) {
+    return onFetchError(error);
+  }
+};
+
+const hideLoading = () => ({
+  type: STATISTICS_PAGE_HIDE_LOADING,
+});

data/webpack/src/Router/StatisticsPage/StatisticsPageSelectors.js

@@ -0,0 +1,12 @@
+export const selectStatisticsPage = state => state.statisticsPage;
+
+export const selectStatisticsMetadata = state =>
+  selectStatisticsPage(state).metadata;
+export const selectStatisticsIsLoading = state =>
+  selectStatisticsPage(state).isLoading;
+export const selectStatisticsMessage = state =>
+  selectStatisticsPage(state).message;
+export const selectStatisticsHasError = state =>
+  selectStatisticsPage(state).hasError;
+export const selectStatisticsHasMetadata = state =>
+  selectStatisticsPage(state).hasData;

data/webpack/src/Router/StatisticsPage/__tests__/StatisticsPage.test.js

@@ -0,0 +1,12 @@
+import { testComponentSnapshotsWithFixtures } from '@theforeman/test';
+import { statisticsProps } from '../StatisticsPage.fixtures';
+import StatisticsPage from '../StatisticsPage';
+
+const fixtures = {
+  'render with props': statisticsProps,
+};
+
+describe('StatisticsPage', () => {
+  describe('rendering', () =>
+    testComponentSnapshotsWithFixtures(StatisticsPage, fixtures));
+});

data/webpack/src/Router/StatisticsPage/__tests__/StatisticsPageActions.test.js

@@ -0,0 +1,27 @@
+import API from 'foremanReact/API';
+
+import { testActionSnapshotWithFixtures } from '@theforeman/test';
+import { getStatisticsMeta } from '../StatisticsPageActions';
+import { statisticsProps } from '../StatisticsPage.fixtures';
+
+jest.mock('foremanReact/API');
+
+const runStatisticsAction = (callback, props, serverMock) => {
+  API.get.mockImplementation(serverMock);
+
+  return callback(props);
+};
+
+const fixtures = {
+  'should fetch statisticsMeta': () =>
+    runStatisticsAction(getStatisticsMeta, {}, async () => ({
+      data: { charts: statisticsProps.statisticsMeta },
+    })),
+  'should fetch statisticsMeta and fail': () =>
+    runStatisticsAction(getStatisticsMeta, {}, async () => {
+      throw new Error('some-error');
+    }),
+};
+
+describe('StatisticsPage actions', () =>
+  testActionSnapshotWithFixtures(fixtures));

data/webpack/src/Router/StatisticsPage/__tests__/StatisticsPageSelectors.test.js

@@ -0,0 +1,32 @@
+import { testSelectorsSnapshotWithFixtures } from '@theforeman/test';
+import {
+  selectStatisticsPage,
+  selectStatisticsMetadata,
+  selectStatisticsHasMetadata,
+  selectStatisticsIsLoading,
+  selectStatisticsMessage,
+  selectStatisticsHasError,
+} from '../StatisticsPageSelectors';
+import { statisticsProps } from '../StatisticsPage.fixtures';
+
+const state = {
+  statisticsPage: {
+    ...statisticsProps,
+  },
+};
+
+const fixtures = {
+  'should return StatisticsPage': () => selectStatisticsPage(state),
+  'should return StatisticsHasMetadata': () =>
+    selectStatisticsHasMetadata(state),
+  'should return StatisticsPage statisticsMeta': () =>
+    selectStatisticsMetadata(state),
+  'should return StatisticsPage isLoading': () =>
+    selectStatisticsIsLoading(state),
+  'should return StatisticsPage Message': () => selectStatisticsMessage(state),
+  'should return StatisticsPage hasError': () =>
+    selectStatisticsHasError(state),
+};
+
+describe('StatisticsPage selectors', () =>
+  testSelectorsSnapshotWithFixtures(fixtures));

data/webpack/src/Router/StatisticsPage/__tests__/__snapshots__/StatisticsPage.test.js.snap

@@ -0,0 +1,32 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`StatisticsPage rendering render with props 1`] = `
+<PageLayout
+  header="Statistics"
+  searchable={false}
+>
+  <Component
+    getStatisticsMeta={[Function]}
+    hasData={true}
+    hasError={false}
+    isLoading={false}
+    message={Object {}}
+    statisticsMeta={
+      Array [
+        Object {
+          "id": "operatingsystem",
+          "search": "/hosts?search=os_title=~VAL~",
+          "title": "OS Distribution",
+          "url": "statistics/operatingsystem",
+        },
+        Object {
+          "id": "architecture",
+          "search": "/hosts?search=facts.architecture=~VAL~",
+          "title": "Architecture Distribution",
+          "url": "statistics/architecture",
+        },
+      ]
+    }
+  />
+</PageLayout>
+`;

data/webpack/src/Router/StatisticsPage/__tests__/__snapshots__/StatisticsPageActions.test.js.snap

@@ -0,0 +1,54 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`StatisticsPage actions should fetch statisticsMeta 1`] = `
+Array [
+  Array [
+    Object {
+      "type": "STATISTICS_PAGE_HIDE_LOADING",
+    },
+  ],
+  Array [
+    Object {
+      "payload": Object {
+        "hasData": true,
+        "metadata": Array [
+          Object {
+            "id": "operatingsystem",
+            "search": "/hosts?search=os_title=~VAL~",
+            "title": "OS Distribution",
+            "url": "statistics/operatingsystem",
+          },
+          Object {
+            "id": "architecture",
+            "search": "/hosts?search=facts.architecture=~VAL~",
+            "title": "Architecture Distribution",
+            "url": "statistics/architecture",
+          },
+        ],
+      },
+      "type": "STATISTICS_PAGE_DATA_RESOLVED",
+    },
+  ],
+]
+`;
+
+exports[`StatisticsPage actions should fetch statisticsMeta and fail 1`] = `
+Array [
+  Array [
+    Object {
+      "type": "STATISTICS_PAGE_HIDE_LOADING",
+    },
+  ],
+  Array [
+    Object {
+      "payload": Object {
+        "message": Object {
+          "text": "some-error",
+          "type": "error",
+        },
+      },
+      "type": "STATISTICS_PAGE_FETCH_FAILED",
+    },
+  ],
+]
+`;

data/webpack/src/Router/StatisticsPage/__tests__/__snapshots__/StatisticsPageSelectors.test.js.snap

@@ -0,0 +1,35 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`StatisticsPage selectors should return StatisticsHasMetadata 1`] = `true`;
+
+exports[`StatisticsPage selectors should return StatisticsPage 1`] = `
+Object {
+  "getStatisticsMeta": [Function],
+  "hasData": true,
+  "hasError": false,
+  "isLoading": false,
+  "message": Object {},
+  "statisticsMeta": Array [
+    Object {
+      "id": "operatingsystem",
+      "search": "/hosts?search=os_title=~VAL~",
+      "title": "OS Distribution",
+      "url": "statistics/operatingsystem",
+    },
+    Object {
+      "id": "architecture",
+      "search": "/hosts?search=facts.architecture=~VAL~",
+      "title": "Architecture Distribution",
+      "url": "statistics/architecture",
+    },
+  ],
+}
+`;
+
+exports[`StatisticsPage selectors should return StatisticsPage Message 1`] = `Object {}`;
+
+exports[`StatisticsPage selectors should return StatisticsPage hasError 1`] = `false`;
+
+exports[`StatisticsPage selectors should return StatisticsPage isLoading 1`] = `false`;
+
+exports[`StatisticsPage selectors should return StatisticsPage statisticsMeta 1`] = `undefined`;

data/webpack/src/Router/StatisticsPage/constants.js

@@ -0,0 +1,4 @@
+export const STATISTICS_PAGE_DATA_RESOLVED = 'STATISTICS_PAGE_DATA_RESOLVED';
+export const STATISTICS_PAGE_DATA_FAILED = 'STATISTICS_PAGE_FETCH_FAILED';
+export const STATISTICS_PAGE_HIDE_LOADING = 'STATISTICS_PAGE_HIDE_LOADING';
+export const STATISTICS_PAGE_URL = '/foreman_statistics/statistics';

data/webpack/src/Router/StatisticsPage/index.js

@@ -0,0 +1,36 @@
+import { compose, bindActionCreators } from 'redux';
+import { connect } from 'react-redux';
+import { callOnMount } from 'foremanReact/common/HOC';
+import withDataReducer from 'foremanReact/routes/common/reducerHOC/withDataReducer';
+
+import * as actions from './StatisticsPageActions';
+import {
+  selectStatisticsMetadata,
+  selectStatisticsMessage,
+  selectStatisticsIsLoading,
+  selectStatisticsHasMetadata,
+  selectStatisticsHasError,
+} from './StatisticsPageSelectors';
+
+import StatisticsPage from './StatisticsPage';
+
+// map state to props
+const mapStateToProps = state => ({
+  statisticsMeta: selectStatisticsMetadata(state),
+  isLoading: selectStatisticsIsLoading(state),
+  message: selectStatisticsMessage(state),
+  hasData: selectStatisticsHasMetadata(state),
+  hasError: selectStatisticsHasError(state),
+});
+
+// map action dispatchers to props
+const mapDispatchToProps = dispatch => bindActionCreators(actions, dispatch);
+
+// export reducers
+export const reducers = { statisticsPage: withDataReducer('STATISTICS_PAGE') };
+
+// export connected component
+export default compose(
+  connect(mapStateToProps, mapDispatchToProps),
+  callOnMount(({ getStatisticsMeta }) => getStatisticsMeta())
+)(StatisticsPage);

data/webpack/src/Router/__snapshots__/routes.test.js.snap

@@ -0,0 +1,47 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`ForemanPluginTemplateRoutes should create routes 1`] = `
+Object {
+  "statistics": Object {
+    "component": Object {
+      "$$typeof": Symbol(react.memo),
+      "WrappedComponent": [Function],
+      "compare": null,
+      "displayName": "Connect(Component)",
+      "type": [Function],
+    },
+    "exact": true,
+    "path": "/foreman_statistics/statistics",
+    "renderResult": <ContextProvider
+      value={
+        Object {
+          "store": Object {
+            "dispatch": [MockFunction],
+            "getState": [Function],
+            "subscribe": [MockFunction],
+          },
+          "subscription": Subscription {
+            "handleChangeWrapper": [Function],
+            "listeners": Object {
+              "notify": [Function],
+            },
+            "onStateChange": [Function],
+            "parentSub": undefined,
+            "store": Object {
+              "dispatch": [MockFunction],
+              "getState": [Function],
+              "subscribe": [MockFunction],
+            },
+            "unsubscribe": null,
+          },
+        }
+      }
+    >
+      <Connect(Component)
+        history={Object {}}
+        some="props"
+      />
+    </ContextProvider>,
+  },
+}
+`;

data/webpack/src/Router/index.js

@@ -0,0 +1,14 @@
+import React from 'react';
+import { Switch, Route } from 'react-router-dom';
+
+import routes from './routes';
+
+const Router = () => (
+  <Switch>
+    {Object.entries(routes).map(([key, props]) => (
+      <Route key={key} {...props} />
+    ))}
+  </Switch>
+);
+
+export default Router;

data/webpack/src/Router/routes.js

@@ -0,0 +1,11 @@
+import StatisticsPage from './StatisticsPage';
+
+const routes = {
+  statistics: {
+    path: '/foreman_statistics/statistics',
+    exact: true,
+    component: StatisticsPage,
+  },
+};
+
+export default routes;

data/webpack/src/Router/routes.test.js

@@ -0,0 +1,27 @@
+import React from 'react';
+import { Provider } from 'react-redux';
+import { shallow } from '@theforeman/test';
+
+import { statisticsProps } from './StatisticsPage/StatisticsPage.fixtures';
+import Routes from './routes';
+
+describe('ForemanPluginTemplateRoutes', () => {
+  it('should create routes', () => {
+    Object.entries(Routes).forEach(([key, Route]) => {
+      const store = {
+        subscribe: jest.fn(),
+        dispatch: jest.fn(),
+        getState: () => statisticsProps,
+      };
+      const RouteComponent = Route.component;
+      const component = shallow(
+        <Provider store={store}>
+          <RouteComponent history={{}} some="props" />
+        </Provider>
+      );
+      Route.renderResult = component;
+    });
+
+    expect(Routes).toMatchSnapshot();
+  });
+});

data/webpack/src/index.js

@@ -0,0 +1 @@
+export { default } from './ForemanStatistics';

data/webpack/src/reducers.js

@@ -0,0 +1,7 @@
+// import { combineReducers } from 'redux';
+
+const reducers = {
+  // foremanStatistics: combineReducers({}),
+};
+
+export default reducers;

data/webpack/src/trends.js

@@ -0,0 +1,7 @@
+export function trendTypeSelected({ value }) {
+  ['trend_trendable_id', 'trend_name'].forEach(id => {
+    const element = document.getElementById(id);
+    element.disabled = value !== 'FactName';
+    element.value = '';
+  });
+}

data/webpack/src/trends.test.js

@@ -0,0 +1,44 @@
+/* eslint-disable jquery/no-val */
+/* eslint-disable jquery/no-is */
+
+import $ from 'jquery';
+
+jest.unmock('jquery');
+jest.unmock('./trends');
+window.trends = require('./trends');
+
+describe('selecting trend type', () => {
+  it('should disable fields on non-fact trend', () => {
+    document.body.innerHTML = `<select id="trendable_type" onchange="trends.trendTypeSelected(this)">
+      <option value="FactName">Facts</option>
+      <option value="Hostgroup">Host group</option>
+    </select>
+    <select id="trend_trendable_id">
+      <option value=""></option>
+      <option value="27">architecture</option>
+    </select>
+    <input id="trend_name">`;
+    $('#trendable_type')
+      .val('Hostgroup')
+      .change();
+    expect($('#trend_trendable_id').is(':disabled')).toBeTruthy();
+    expect($('#trend_name').is(':disabled')).toBeTruthy();
+  });
+
+  it('should enable fields on non-fact trend', () => {
+    document.body.innerHTML = `<select id="trendable_type" onchange="trends.trendTypeSelected(this)">
+      <option value="Evironment">Environment</option>
+      <option value="FactName">Facts</option>
+    </select>
+    <select id="trend_trendable_id" disabled>
+      <option value=""></option>
+      <option value="27">architecture</option>
+    </select>
+    <input id="trend_name" disabled>`;
+    $('#trendable_type')
+      .val('FactName')
+      .change();
+    expect($('#trend_trendable_id').is(':disabled')).toBeFalsy();
+    expect($('#trend_name').is(':disabled')).toBeFalsy();
+  });
+});

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: foreman_statistics
 version: !ruby/object:Gem::Version
-  version: 0.1.1
+  version: 0.1.2
 platform: ruby
 authors:
 - Ondrej Ezr
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-06-18 00:00:00.000000000 Z
+date: 2020-07-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rdoc
@@ -28,16 +28,16 @@ dependencies:
   name: rubocop
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '0.83'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '0.83'
 - !ruby/object:Gem::Dependency
   name: rubocop-minitest
   requirement: !ruby/object:Gem::Requirement
@@ -159,6 +159,58 @@ files:
 - test/unit/foreman_statistics/statistics_test.rb
 - test/unit/foreman_statistics_test.rb
 - test/unit/tasks/foreman_statistics_tasks_test.rb
+- webpack/__mocks__/foremanReact/API.js
+- 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/ChartBox/index.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/Layout/LayoutActions.js
+- webpack/__mocks__/foremanReact/components/Pagination/PaginationWrapper.js
+- webpack/__mocks__/foremanReact/components/common/EmptyState.js
+- webpack/__mocks__/foremanReact/components/common/MessageBox.js
+- webpack/__mocks__/foremanReact/components/common/dates/LongDateTime.js
+- webpack/__mocks__/foremanReact/components/common/dates/RelativeDateTime.js
+- webpack/__mocks__/foremanReact/components/common/table.js
+- webpack/__mocks__/foremanReact/components/common/table/actionsHelpers/actionTypeCreator.js
+- webpack/__mocks__/foremanReact/constants.js
+- webpack/__mocks__/foremanReact/readme.md
+- 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/__mocks__/foremanReact/routes/common/reducerHOC/withDataReducer.js
+- webpack/fills_index.js
+- webpack/index.js
+- webpack/src/Components/StatisticsChartsList/StatisticsChartsList.fixtures.js
+- webpack/src/Components/StatisticsChartsList/StatisticsChartsList.test.js
+- webpack/src/Components/StatisticsChartsList/StatisticsChartsListStyles.scss
+- webpack/src/Components/StatisticsChartsList/__snapshots__/StatisticsChartsList.test.js.snap
+- webpack/src/Components/StatisticsChartsList/index.js
+- webpack/src/ForemanStatistics.js
+- webpack/src/Router/StatisticsPage/Statistics/Statistics.js
+- webpack/src/Router/StatisticsPage/StatisticsPage.fixtures.js
+- webpack/src/Router/StatisticsPage/StatisticsPage.js
+- webpack/src/Router/StatisticsPage/StatisticsPageActions.js
+- webpack/src/Router/StatisticsPage/StatisticsPageSelectors.js
+- webpack/src/Router/StatisticsPage/__tests__/StatisticsPage.test.js
+- webpack/src/Router/StatisticsPage/__tests__/StatisticsPageActions.test.js
+- webpack/src/Router/StatisticsPage/__tests__/StatisticsPageSelectors.test.js
+- webpack/src/Router/StatisticsPage/__tests__/__snapshots__/StatisticsPage.test.js.snap
+- webpack/src/Router/StatisticsPage/__tests__/__snapshots__/StatisticsPageActions.test.js.snap
+- webpack/src/Router/StatisticsPage/__tests__/__snapshots__/StatisticsPageSelectors.test.js.snap
+- webpack/src/Router/StatisticsPage/constants.js
+- webpack/src/Router/StatisticsPage/index.js
+- webpack/src/Router/__snapshots__/routes.test.js.snap
+- webpack/src/Router/index.js
+- webpack/src/Router/routes.js
+- webpack/src/Router/routes.test.js
+- webpack/src/index.js
+- webpack/src/reducers.js
+- webpack/src/trends.js
+- webpack/src/trends.test.js
 homepage: https://theforeman.org
 licenses:
 - GPL-3.0