airbnb/caravel

View on GitHub
superset-frontend/src/dashboard/util/updateComponentParentsList.test.js

Summary

Maintainability
B
6 hrs
Test Coverage
/**
 * Licensed to the Apache Software Foundation (ASF) under one
 * or more contributor license agreements.  See the NOTICE file
 * distributed with this work for additional information
 * regarding copyright ownership.  The ASF licenses this file
 * to you under the Apache License, Version 2.0 (the
 * "License"); you may not use this file except in compliance
 * with the License.  You may obtain a copy of the License at
 *
 *   http://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing,
 * software distributed under the License is distributed on an
 * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
 * KIND, either express or implied.  See the License for the
 * specific language governing permissions and limitations
 * under the License.
 */
import updateComponentParentsList from 'src/dashboard/util/updateComponentParentsList';
import { DASHBOARD_ROOT_ID } from 'src/dashboard/util/constants';
import {
  dashboardLayout,
  dashboardLayoutWithTabs,
} from 'spec/fixtures/mockDashboardLayout';

describe('updateComponentParentsList', () => {
  const emptyLayout = {
    DASHBOARD_VERSION_KEY: 'v2',
    GRID_ID: {
      children: [],
      id: 'GRID_ID',
      type: 'GRID',
    },
    ROOT_ID: {
      children: ['GRID_ID'],
      id: 'ROOT_ID',
      type: 'ROOT',
    },
  };
  const gridLayout = {
    ...dashboardLayout.present,
  };
  const tabsLayout = {
    ...dashboardLayoutWithTabs.present,
  };

  it('should handle empty layout', () => {
    const nextState = {
      ...emptyLayout,
    };

    updateComponentParentsList({
      currentComponent: nextState[DASHBOARD_ROOT_ID],
      layout: nextState,
    });

    expect(nextState.GRID_ID.parents).toEqual(['ROOT_ID']);
  });

  it('should handle grid layout', () => {
    const nextState = {
      ...gridLayout,
    };

    updateComponentParentsList({
      currentComponent: nextState[DASHBOARD_ROOT_ID],
      layout: nextState,
    });

    expect(nextState.GRID_ID.parents).toEqual(['ROOT_ID']);
    expect(nextState.CHART_ID.parents).toEqual([
      'ROOT_ID',
      'GRID_ID',
      'ROW_ID',
      'COLUMN_ID',
    ]);
  });

  it('should handle root level tabs', () => {
    const nextState = {
      ...tabsLayout,
    };

    updateComponentParentsList({
      currentComponent: nextState[DASHBOARD_ROOT_ID],
      layout: nextState,
    });

    expect(nextState.GRID_ID.parents).toEqual(['ROOT_ID']);
    expect(nextState.CHART_ID2.parents).toEqual([
      'ROOT_ID',
      'TABS_ID',
      'TAB_ID2',
      'ROW_ID2',
    ]);
  });
});

describe('updateComponentParentsList with bad inputs', () => {
  it('should handle invalid parameters and not throw error', () => {
    updateComponentParentsList({
      currentComponent: undefined,
      layout: undefined,
    });

    expect(() =>
      updateComponentParentsList({
        currentComponent: undefined,
        layout: undefined,
      }),
    ).not.toThrow();

    expect(() =>
      updateComponentParentsList({
        currentComponent: {},
        layout: undefined,
      }),
    ).not.toThrow();

    /**
     * the assignment of layout = {} only works for undefined, not null
     * This was a missed case in the function previously.
     * This test ensure the null check is not removed
     */
    expect(() =>
      updateComponentParentsList({
        currentComponent: {},
        layout: null,
      }),
    ).not.toThrow();

    /**
     * This test catches an edge case that caused runtime error in production system where
     * a simple logic flaw performed a dot notation lookup on an undefined object
     */
    expect(() =>
      updateComponentParentsList({
        currentComponent: { id: 'id3', children: ['id1', 'id2'] },
        layout: { id3: {} },
      }),
    ).not.toThrow();

    /**
     * This test catches an edge case that causes runtime error where
     * a simple logic flaw performed currentComponent.children.forEach without
     * verifying currentComponent.children is an Array with a .forEach function defined
     */
    expect(() =>
      updateComponentParentsList({
        currentComponent: { id: 'id3' },
        layout: { id3: {} },
      }),
    ).not.toThrow();

    expect(() =>
      updateComponentParentsList({
        currentComponent: { id: 'id3' },
        layout: {},
      }),
    ).not.toThrow();
  });
});