Merge pull request #134 from pi-hole/tweak/tests-directory

Move tests into __tests__ folders

src/components/common/__tests__/Alert.test.js

@@ -10,7 +10,7 @@
 
 import React from "react";
 import { shallow } from "enzyme";
-import Alert from "./Alert";
+import Alert from "../Alert";
 
 it("renders an alert correctly", () => {
   const message = "test message";

src/components/common/__tests__/BoxedLayoutButton.test.js

@@ -10,7 +10,7 @@
 
 import React from "react";
 import { shallow } from "enzyme";
-import BoxedLayoutButton from "./BoxedLayoutButton";
+import BoxedLayoutButton from "../BoxedLayoutButton";
 
 it("renders without crashing", () => {
   shallow(<BoxedLayoutButton />);

src/components/common/__tests__/Footer.test.js

@@ -10,7 +10,7 @@
 
 import React from "react";
 import { shallow } from "enzyme";
-import Footer from "./Footer";
+import Footer from "../Footer";
 
 it("renders without crashing", () => {
   shallow(<Footer />);

src/components/common/__tests__/Sidebar.test.js

@@ -10,8 +10,8 @@
 
 import React from "react";
 import { shallow } from "enzyme";
-import { isDropdownOpen, navDropdown, navItem, navList } from "./Sidebar";
-import api from "../../util/api";
+import { isDropdownOpen, navDropdown, navItem, navList } from "../Sidebar";
+import api from "../../../util/api";
 
 it("expands active drop down items", () => {
   const isOpen = isDropdownOpen("/testRoute", {

src/components/common/__tests__/StatusBadge.test.js

@@ -10,7 +10,7 @@
 
 import React from "react";
 import { shallow } from "enzyme";
-import { TranslatedStatusBadge } from "./StatusBadge";
+import { TranslatedStatusBadge } from "../StatusBadge";
 
 it("shows green enabled message if API returns enabled", async () => {
   const wrapper = shallow(<TranslatedStatusBadge status="enabled" />);

src/components/dashboard/__tests__/ClientsGraph.test.js

@@ -14,7 +14,7 @@ import {
   TranslatedClientsGraph,
   transformData,
   loadingProps
-} from "./ClientsGraph";
+} from "../ClientsGraph";
 
 const fakeData = {
   over_time: [

src/components/dashboard/__tests__/GenericDoughnutChart.test.js

@@ -14,7 +14,7 @@ import {
   GenericDoughnutChart,
   transformData,
   loadingProps
-} from "./GenericDoughnutChart";
+} from "../GenericDoughnutChart";
 
 const fakeData = [
   { name: "roberta.net", ip: "8.239.48.32", percent: 0.38411761010240625 },

src/components/dashboard/__tests__/SummaryStats.test.js

@@ -14,7 +14,7 @@ import {
   TranslatedSummaryStats,
   transformData,
   errorProps
-} from "./SummaryStats";
+} from "../SummaryStats";
 
 const fakeData = {
   active_clients: 2,

src/components/dashboard/__tests__/TopBlocked.test.js

@@ -9,7 +9,7 @@
  * Please see LICENSE file for your rights under this license. */
 
 import React from "react";
-import { transformData, generateRows } from "./TopBlocked";
+import { transformData, generateRows } from "../TopBlocked";
 
 const fakeData = {
   top_domains: [

src/components/dashboard/__tests__/TopClients.test.js

@@ -9,7 +9,7 @@
  * Please see LICENSE file for your rights under this license. */
 
 import React from "react";
-import { transformData, generateRows } from "./TopClients";
+import { transformData, generateRows } from "../TopClients";
 
 const fakeData = {
   top_clients: [

src/components/dashboard/__tests__/TopDomains.test.js

@@ -9,7 +9,7 @@
  * Please see LICENSE file for your rights under this license. */
 
 import React from "react";
-import { generateRows, transformData } from "./TopDomains";
+import { generateRows, transformData } from "../TopDomains";
 
 const fakeData = {
   top_domains: [

src/components/dashboard/__tests__/TopTable.test.js

@@ -10,7 +10,7 @@
 
 import React from "react";
 import { shallow } from "enzyme";
-import { TopTable } from "./TopTable";
+import { TopTable } from "../TopTable";
 
 it("shows loading indicator correctly", () => {
   const wrapper = shallow(<TopTable loading={true} />);

src/components/list/__tests__/DomainInput.test.js

@@ -10,9 +10,9 @@
 
 import React from "react";
 import { shallow } from "enzyme";
-import DomainInput from "./DomainInput";
-import api from "../../util/api";
-import { isValidDomain } from "../../validate";
+import DomainInput from "../DomainInput";
+import api from "../../../util/api";
+import { isValidDomain } from "../../../util/validate";
 
 it("has a placeholder", () => {
   const placeholder = "placeholder";

src/components/list/__tests__/DomainList.test.js

@@ -10,8 +10,8 @@
 
 import React from "react";
 import { shallow } from "enzyme";
-import DomainList from "./DomainList";
-import api from "../../util/api";
+import DomainList from "../DomainList";
+import api from "../../../util/api";
 
 const domains = ["domain1.com", "domain2.com", "domain3.com"];
 

src/components/list/__tests__/ListPage.test.js

@@ -10,7 +10,7 @@
 
 import React from "react";
 import { shallow } from "enzyme";
-import ListPage from "./ListPage";
+import ListPage from "../ListPage";
 
 it("shows the title", () => {
   const title = "My Title";

src/components/login/__tests__/ForgotPassword.test.js

@@ -10,7 +10,7 @@
 
 import React from "react";
 import { shallow } from "enzyme";
-import ForgotPassword from "./ForgotPassword";
+import ForgotPassword from "../ForgotPassword";
 
 it("collapses and displays normal if there is no error", () => {
   const wrapper = shallow(<ForgotPassword error={false} />);

src/components/settings/DHCPInfo.js

@@ -22,7 +22,11 @@ import {
   InputGroupAddon,
   Label
 } from "reactstrap";
-import { isPositiveNumber, isValidHostname, isValidIpv4 } from "../../validate";
+import {
+  isPositiveNumber,
+  isValidHostname,
+  isValidIpv4
+} from "../../util/validate";
 import Alert from "../common/Alert";
 
 class DHCPInfo extends Component {

src/components/settings/DNSInfo.js

@@ -17,7 +17,7 @@ import { Button, Col, Form, FormGroup } from "reactstrap";
 import ConditionalForwardingSettings from "./ConditionalForwardingSettings";
 import DnsOptionSettings from "./DnsOptionSettings";
 import Alert from "../common/Alert";
-import { isValidHostname, isValidIpv4 } from "../../validate";
+import { isValidHostname, isValidIpv4 } from "../../util/validate";
 
 class DNSInfo extends Component {
   state = {

src/components/settings/DnsList.js

@@ -13,7 +13,7 @@ import PropTypes from "prop-types";
 import { ListGroup } from "reactstrap";
 import DnsListItem from "./DnsListItem";
 import DnsListNewItem from "./DnsListNewItem";
-import { isValidIpv4OptionalPort } from "../../validate";
+import { isValidIpv4OptionalPort } from "../../util/validate";
 
 /**
  * Check if an upstream address is unique and valid

src/components/settings/__tests__/DHCPInfo.test.js

@@ -10,7 +10,7 @@
 
 import React from "react";
 import { shallow } from "enzyme";
-import DHCPInfo from "./DHCPInfo";
+import DHCPInfo from "../DHCPInfo";
 import fetchMock from "fetch-mock";
 
 const endpoint = "/admin/api/settings/dhcp";

src/index.js

@@ -18,7 +18,7 @@ import "react-bootstrap-typeahead/css/Typeahead.css";
 import "./scss/style.css";
 import Full from "./containers/Full";
 import api from "./util/api";
-import { setupI18n } from "./i18n";
+import { setupI18n } from "./util/i18n";
 
 // Before rendering anything, check if there is a session cookie.
 // Note: the user could have an old session, so the first API call

src/util/__tests__/api.test.js

@@ -8,7 +8,7 @@
  * This file is copyright under the latest version of the EUPL.
  * Please see LICENSE file for your rights under this license. */
 
-import api from "./api";
+import api from "../api";
 
 // This is a dumb test used to set up the next test,
 // which checks that the logged in state is reset before each test

src/util/__tests__/validate.test.js

@@ -14,7 +14,7 @@ import {
   isValidIpv4,
   isValidIpv4OptionalPort,
   isValidRegex
-} from "./validate";
+} from "../validate";
 
 describe("Testing the validation functions", () => {
   describe("isValidDomain", () => {

src/util/i18n.js

@@ -12,7 +12,7 @@ import i18n from "i18next";
 import XHR from "i18next-xhr-backend";
 import LanguageDetector from "i18next-browser-languagedetector";
 import { reactI18nextModule } from "react-i18next";
-import config from "./config";
+import config from "../config";
 
 export function setupI18n() {
   i18n

src/views/Blacklist.js

@@ -12,7 +12,7 @@ import React from "react";
 import { withNamespaces } from "react-i18next";
 import api from "../util/api";
 import ListPage from "../components/list/ListPage";
-import { isValidDomain } from "../validate";
+import { isValidDomain } from "../util/validate";
 
 const Blacklist = props => {
   const { t } = props;

src/views/Regexlist.js

@@ -12,7 +12,7 @@ import React from "react";
 import { withNamespaces } from "react-i18next";
 import ListPage from "../components/list/ListPage";
 import api from "../util/api";
-import { isValidRegex } from "../validate";
+import { isValidRegex } from "../util/validate";
 
 const Regexlist = props => {
   const { t } = props;

src/views/Whitelist.js

@@ -12,7 +12,7 @@ import React from "react";
 import { withNamespaces } from "react-i18next";
 import ListPage from "../components/list/ListPage";
 import api from "../util/api";
-import { isValidDomain } from "../validate";
+import { isValidDomain } from "../util/validate";
 
 const Whitelist = props => {
   const { t } = props;