Merge pull request #24 from openwisp/issue#18

[component] Add login component and write tests #18
openwisp · Jul 31, 2019 · ef803ed · ef803ed
2 parents b1349da + 1058dbd
commit ef803ed
Show file tree

Hide file tree

Showing 28 changed files with 1,609 additions and 421 deletions.
diff --git a/.eslintrc.json b/.eslintrc.json
@@ -1,34 +1,35 @@
 {
-	"extends": [
-		"airbnb",
-		"prettier",
-		"prettier/react",
-		"jest-enzyme",
-		"plugin:jest/recommended"
-	],
-	"parser": "babel-eslint",
-	"plugins": ["prettier", "jest"],
-	"parserOptions": {
-		"ecmaVersion": 2016,
-		"sourceType": "module",
-		"ecmaFeatures": {
-			"jsx": true
-		}
-	},
-	"env": {
-		"es6": true,
-		"browser": true,
-		"node": true
-	},
-	"rules": {
-		"array-bracket-spacing": [2, "never"],
-		"semi": [2, "always"],
-		"react/prefer-stateless-function": [0],
-		"react/jsx-filename-extension": [
-			1,
-			{
-				"extensions": [".js", ".jsx"]
-			}
-		]
-	}
+  "extends": [
+    "airbnb",
+    "prettier",
+    "prettier/react",
+    "jest-enzyme",
+    "plugin:jest/recommended"
+  ],
+  "parser": "babel-eslint",
+  "plugins": ["prettier", "jest"],
+  "parserOptions": {
+    "ecmaVersion": 2016,
+    "sourceType": "module",
+    "ecmaFeatures": {
+      "jsx": true
+    }
+  },
+  "env": {
+    "es6": true,
+    "browser": true,
+    "node": true
+  },
+  "rules": {
+    "array-bracket-spacing": [2, "never"],
+    "semi": [2, "always"],
+    "jsx-a11y/label-has-for": 0,
+    "react/prefer-stateless-function": [0],
+    "react/jsx-filename-extension": [
+      1,
+      {
+        "extensions": [".js", ".jsx"]
+      }
+    ]
+  }
 }
diff --git a/client/assets/default/index.css b/client/assets/default/index.css
@@ -0,0 +1 @@
+/*  */
diff --git a/client/components/footer/__snapshots__/footer.test.js.snap b/client/components/footer/__snapshots__/footer.test.js.snap
@@ -13,20 +13,20 @@ exports[`<Footer /> rendering should render correctly 1`] = `
       <a
         className="owisp-footer-link
                   owisp-footer-link-1"
-        href="www.testurl.com"
+        href="https://link-1.com"
         rel="noreferrer noopener"
         target="_blank"
       >
-        link one
+        link-1
       </a>
       <a
         className="owisp-footer-link
                   owisp-footer-link-2"
-        href="www.testurl2.com"
+        href="https://link-2.com"
         rel="noreferrer noopener"
         target="_blank"
       >
-        link two
+        link-2
       </a>
     </div>
   </div>
@@ -35,9 +35,7 @@ exports[`<Footer /> rendering should render correctly 1`] = `
   >
     <div
       className="owisp-footer-row-2-inner"
-    >
-      this is secondary text
-    </div>
+    />
   </div>
 </div>
 `;
diff --git a/client/components/footer/footer.test.js b/client/components/footer/footer.test.js
@@ -3,26 +3,14 @@ import {shallow} from "enzyme";
 import React from "react";
 import renderer from "react-test-renderer";
 
+import getConfig from "../../utils/get-config";
 import Footer from "./footer";
 
+const defaultConfig = getConfig("default");
 const createTestProps = props => {
   return {
     language: "en",
-    footer: {
-      links: [
-        {
-          text: {en: "link one"},
-          url: "www.testurl.com",
-        },
-        {
-          text: {en: "link two"},
-          url: "www.testurl2.com",
-        },
-      ],
-      secondary_text: {
-        en: "this is secondary text",
-      },
-    },
+    footer: defaultConfig.components.footer,
     ...props,
   };
 };
@@ -48,9 +36,14 @@ describe("<Footer /> rendering", () => {
     expect(wrapper.find(".owisp-footer-link")).toHaveLength(0);
   });
   it("should render secondary text", () => {
-    const {secondary_text} = props.footer;
-    expect(wrapper.find(".owisp-footer-row-2-inner").text()).toBe(
-      secondary_text.en,
-    );
+    wrapper.setProps({
+      footer: {...props.footer, secondary_text: {en: "secondary text"}},
+    });
+    expect(
+      wrapper
+        .update()
+        .find(".owisp-footer-row-2-inner")
+        .text(),
+    ).toBe("secondary text");
   });
 });
diff --git a/client/components/header/__snapshots__/header.test.js.snap b/client/components/header/__snapshots__/header.test.js.snap
@@ -46,20 +46,20 @@ exports[`<Header /> rendering should render with links 1`] = `
       <a
         className="owisp-header-link
                     owisp-header-link-1"
-        href="https://testlink1.com"
+        href="https://link-1.com"
         rel="noreferrer noopener"
         target="_blank"
       >
-        test link 1
+        link-1
       </a>
       <a
         className="owisp-header-link
                     owisp-header-link-2"
-        href="https://testlink2.com"
+        href="https://link-2.com"
         rel="noreferrer noopener"
         target="_blank"
       >
-        test link 2
+        link-2
       </a>
     </div>
   </div>

diff --git a/client/components/header/header.js b/client/components/header/header.js
@@ -2,6 +2,7 @@ import "./index.css";
 
 import PropTypes from "prop-types";
 import React from "react";
+import {Link} from "react-router-dom";
 
 import getAssetPath from "../../utils/get-asset-path";
 import getText from "../../utils/get-text";
@@ -18,11 +19,13 @@ export default class Header extends React.Component {
               <div className="owisp-header-left">
                 <div className="owisp-header-logo-div">
                   {logo.url ? (
-                    <img
-                      src={getAssetPath(orgSlug, logo.url)}
-                      alt={logo.alternate_text}
-                      className="owisp-header-logo-image"
-                    />
+                    <Link to={`/${orgSlug}`}>
+                      <img
+                        src={getAssetPath(orgSlug, logo.url)}
+                        alt={logo.alternate_text}
+                        className="owisp-header-logo-image"
+                      />
+                    </Link>
                   ) : null}
                 </div>
               </div>

diff --git a/client/components/header/header.test.js b/client/components/header/header.test.js
@@ -2,34 +2,17 @@ import {shallow} from "enzyme";
 import React from "react";
 import renderer from "react-test-renderer";
 
+import getConfig from "../../utils/get-config";
 import Header from "./header";
 
+const defaultConfig = getConfig("default");
 const createTestProps = props => {
   return {
     setLanguage: jest.fn(),
     orgSlug: "default",
     language: "en",
     languages: [{slug: "en", text: "english"}, {slug: "it", text: "italian"}],
-    header: {
-      logo: {
-        alternate_text: null,
-        url: null,
-      },
-      links: [
-        {
-          text: {
-            en: "test link 1",
-          },
-          url: "https://testlink1.com",
-        },
-        {
-          text: {
-            en: "test link 2",
-          },
-          url: "https://testlink2.com",
-        },
-      ],
-    },
+    header: defaultConfig.components.header,
     ...props,
   };
 };