ICTU/quality-time

Similar blocks of code found in 2 locations. Consider refactoring.

Open

it("does not sort when told not to", () => {
    render(
        <SingleChoiceInput
            value="b"
            options={[

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

it("does sort when told to", () => {
    render(
        <SingleChoiceInput
            value="b"
            options={[

• Read up
  Read up

File utils.js has 349 lines of code (exceeds 250 allowed). Consider refactoring.

Open

import { arrayOf, number, objectOf, oneOf, string } from "prop-types"

import { PERMISSIONS } from "./context/Permissions"
import { defaultDesiredResponseTimes } from "./defaults"
import {

Similar blocks of code found in 2 locations. Consider refactoring.

Open

        {
            menuItem: (
                <Menu.Item key="debt">
                    <Icon name="money" />
                    <FocusableTab>{"Technical debt"}</FocusableTab>

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

        {
            menuItem: (
                <Menu.Item key="configuration">
                    <Icon name="settings" />
                    <FocusableTab>{"Configuration"}</FocusableTab>

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

it("renders the source label and the popup if there is a parse error", async () => {
    renderSourceStatus(metric, { source_uuid: "source_uuid", parse_error: "error" })
    expect(screen.getAllByText(/Source name/).length).toBe(1)
    await userEvent.hover(screen.queryByText(/Source name/))
    await waitFor(() => {

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

it("renders the source label and the popup if there is an connection error", async () => {
    renderSourceStatus(metric, { source_uuid: "source_uuid", connection_error: "error" })
    expect(screen.getAllByText(/Source name/).length).toBe(1)
    await userEvent.hover(screen.queryByText(/Source name/))
    await waitFor(() => {

• Read up
  Read up

Similar blocks of code found in 3 locations. Consider refactoring.

Open

it("does not submit an unchanged value", async () => {
    let setValue = jest.fn()
    render(<IntegerInput value="42" set_value={setValue} />)
    await userEvent.type(screen.getByDisplayValue(/42/), "{Enter}")
    expect(screen.queryAllByDisplayValue(/42/).length).toBe(1)

• Read up
  Read up

Similar blocks of code found in 4 locations. Consider refactoring.

Open

it("does not invoke the callback if the value is unchanged", async () => {
    const mockCallback = jest.fn()
    render(<TextInput value="Hello" set_value={mockCallback} />)
    await userEvent.type(screen.getByText(/Hello/), "{Shift>}{Enter}")
    expect(screen.getByText(/Hello/)).not.toBe(null)

• Read up
  Read up

Similar blocks of code found in 4 locations. Consider refactoring.

Open

it("resets the value on escape", async () => {
    const mockCallback = jest.fn()
    render(<TextInput value="Hello" set_value={mockCallback} />)
    await userEvent.type(screen.getByText(/Hello/), "Revert{Escape}")
    expect(screen.getByText(/Hello/)).not.toBe(null)

• Read up
  Read up

Similar blocks of code found in 4 locations. Consider refactoring.

Open

it("renders the initial value on escape and does not submit", async () => {
    const mockCallback = jest.fn()
    render(<Input value="Hello" set_value={mockCallback} />)
    await userEvent.type(screen.getByDisplayValue(/Hello/), "Bye{Escape}")
    expect(screen.getByDisplayValue(/Hello/)).not.toBe(null)

• Read up
  Read up

Similar blocks of code found in 3 locations. Consider refactoring.

Open

it("undoes the change on escape", async () => {
    let setValue = jest.fn()
    render(<IntegerInput value="42" set_value={setValue} />)
    await userEvent.type(screen.getByDisplayValue(/42/), "24{escape}")
    expect(screen.queryAllByDisplayValue(/42/).length).toBe(1)

• Read up
  Read up

Similar blocks of code found in 3 locations. Consider refactoring.

Open

it("has a default minimum of zero", async () => {
    let setValue = jest.fn()
    render(<IntegerInput value="-1" set_value={setValue} />)
    await userEvent.type(screen.getByDisplayValue(/-1/), "{Enter}")
    expect(screen.queryAllByDisplayValue(/-1/).length).toBe(1)

• Read up
  Read up

Similar blocks of code found in 4 locations. Consider refactoring.

Open

it("does not invoke the callback on enter", async () => {
    const mockCallback = jest.fn()
    render(<TextInput value="Hello" set_value={mockCallback} />)
    await userEvent.type(screen.getByText(/Hello/), "Bye{Enter}")
    expect(screen.getByText(/Bye/)).not.toBe(null)

• Read up
  Read up

File SubjectTableRow.js has 341 lines of code (exceeds 250 allowed). Consider refactoring.

Open

import { array, bool, func, number, object, string } from "prop-types"
import { useContext } from "react"

import { DarkMode } from "../context/DarkMode"
import { DataModel } from "../context/DataModel"

Similar blocks of code found in 2 locations. Consider refactoring.

Open

it("hides the tag cards", async () => {
    history.push("?hidden_cards=tags")
    renderDashboard({ reportToRender: report })
    expect(screen.getAllByText(/Subject title/).length).toBe(1)
    expect(screen.queryAllByText(/tag/).length).toBe(0)

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

it("hides the subject cards", async () => {
    history.push("?hidden_cards=subjects")
    renderDashboard({ reportToRender: report })
    expect(screen.queryAllByText(/Subject title/).length).toBe(0)
    expect(screen.getAllByText(/tag/).length).toBe(1)

• Read up
  Read up

File Button.test.js has 339 lines of code (exceeds 250 allowed). Consider refactoring.

Open

import { act, fireEvent, render, screen, waitFor } from "@testing-library/react"
import userEvent from "@testing-library/user-event"

import {
    AddButton,

Similar blocks of code found in 2 locations. Consider refactoring.

Open

it("displays the sprint in the label if configured, but without sprint end date", async () => {
    history.push("?show_issue_sprint=true")
    const { queryByText } = renderIssueStatus({ sprint: true, sprintEndDate: null })
    expect(queryByText(/Sprint 42/)).not.toBe(null)
    expect(queryByText(/active/)).not.toBe(null)

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

it("renders the target", () => {
    render(
        <DataModel.Provider value={{ metrics: { violations: { direction: "<", unit: "violations" } } }}>
            <MeasurementTarget metric={{ type: "violations" }} />
        </DataModel.Provider>,

• Read up
  Read up