Dev

.eslintrc.json

@@ -0,0 +1,33 @@
+{
+    "env": {
+        "browser": true,
+        "commonjs": true,
+        "es6": true
+    },
+    "globals": {
+        "$": false
+    },
+    "extends": "eslint:recommended",
+    "parserOptions": {
+        "ecmaVersion": 2015,
+        "sourceType": "module"
+    },
+    "rules": {
+        "indent": [
+            "error",
+            2
+        ],
+        "linebreak-style": [
+            "error",
+            "windows"
+        ],
+        "quotes": [
+            "error",
+            "single"
+        ],
+        "semi": [
+            "error",
+            "always"
+        ]
+    }
+}

.hermione.conf.js

@@ -0,0 +1,25 @@
+module.exports = {
+    browsers: {
+        // Для тестирования на MacOS -- поменять закомменченные и раскомменченные браузеры
+        // safari: {
+        //     desiredCapabilities: {
+        //         browserName: 'safari'
+        //     }
+        // },
+        chrome: {
+            desiredCapabilities: {
+                browserName: 'chrome'
+            }
+        },
+        firefox: {
+            desiredCapabilities: {
+                browserName: 'firefox'
+            }
+        }
+    },
+    plugins: {
+        'html-reporter/hermione': {
+            path: 'hermione-html-report'
+        }
+    }
+};

README.md

@@ -28,6 +28,51 @@ npm start
 
 ## Модульные тесты
 
-- нужно добавить в README список логических блоков системы и их сценариев
-- для каждого блока нужно написать модульные тесты
-- если необходимо, выполните рефакторинг, чтобы реорганизовать логические блоки или добавить точки расширения
+### Cписок логических блоков системы и их сценариев
+
+#### Контроллеры
+- Контроллер контента
+- Контроллер файлов
+- Контроллер индекса
+
+### utils
+- git
+_приватные_
+-- Парсинг истории
+-- Парсинг файлового дерева
+_публичные_
+-- Сборка истории
+-- Сборка файлового дерева
+-- Сборка контента
+
+- навигация
+-- Сборка URL папки
+-- Сборка URL файла
+-- Сборка хлебных крошек
+
+# Сборка тестов
+
+## Модульные тесты
+Для запуска модульных тестов выполните последовательность команд
+```
+npm i
+npm run test
+```
+
+## Интеграционные тесты
+Для запуска интеграционных тестов выполните последовательность команд
+```
+npm i -g selenium-standalone
+selenium-standalone install
+npm i -D hermione
+
+# Вкладка консоли 1
+npm run selen
+# Вкладка консоли 2
+npm start
+# Вкладка консоли 3
+npm run hermi
+```
+
+*Note:* для работы гермионы на Windows требуется установка Python 2 и MS VisualStudio.
+*Note 2:* наблюдаются баги в работе assertView на OS Windows, оптимально тестирование на Unix платформах

controllers/contentController.js

@@ -2,27 +2,43 @@ const { gitFileContent, gitFileTree } = require('../utils/git');
 const { buildFolderUrl, buildBreadcrumbs } = require('../utils/navigation');
 
 module.exports = function(req, res, next) {
-  const { hash } = req.params;
-  const path = req.params[0].split('/').filter(Boolean);
+  const { hash } = req ? req.params : '';
+  const path = req ? req.params[0].split('/').filter(Boolean) : '';
 
-  gitFileTree(hash, path.join('/'))
-    .then(function([file]) {
-      if (file && file.type === 'blob') {
-        return gitFileContent(file.hash);
-      }
-    })
-    .then(
-      content => {
-        if (content) {
-          res.render('content', {
+  this.gitFileTree = gitFileTree;
+  this.gitFileContent = gitFileContent;
+  this.buildBreadcrumbs = buildBreadcrumbs;
+
+  this.buildRenderData = (hash, path) => {
+    path = path ? path.join('/') : '';
+
+    return new Promise ((res, rej) => {
+      this.gitFileTree(hash, path)
+        .then(([file]) => {
+          if (file && file.type === 'blob') {
+            return this.gitFileContent(file.hash);
+          }
+        })
+        .then(content => {
+          res({
             title: 'content',
-            breadcrumbs: buildBreadcrumbs(hash, path.join('/')),
+            breadcrumbs: this.buildBreadcrumbs(hash, path),
             content
           });
-        } else {
-          next();
         }
-      },
-      err => next(err)
-    );
+        );
+    });
+  };
+
+  if (req && res) {
+    this.buildRenderData(hash, path).then(data => {
+      if (data && res) {
+        res.render('content', data);
+      } else {
+        next();
+      }
+    },
+    err => next(err));
+  }
+
 };

controllers/filesController.js

@@ -5,37 +5,59 @@ const {
   buildBreadcrumbs
 } = require('../utils/navigation');
 
-function buildObjectUrl(parentHash, { path, type }) {
-  switch (type) {
-    case 'tree':
-      return buildFolderUrl(parentHash, path);
-    case 'blob':
-      return buildFileUrl(parentHash, path);
-    default:
-      return '#';
+module.exports = function(req, res, next) {
+  const { hash } = req ? req.params : '';
+  const pathParam = req ? (req.params[0] || '').split('/').filter(Boolean) : '';
+
+  const path = pathParam && pathParam.length ? pathParam.join('/') + '/' : '';
+
+  this.gitFileTree = gitFileTree;
+  this.buildFolderUrl = buildFolderUrl;
+  this.buildFileUrl = buildFileUrl;
+  this.buildBreadcrumb = buildBreadcrumbs;
+
+  this.buildObjectUrl = (parentHash, { path, type }) => {
+    switch (type) {
+      case 'tree':
+        return this.buildFolderUrl(parentHash, path);
+      case 'blob':
+        return this.buildFileUrl(parentHash, path);
+      default:
+        return '#';
+    }
   }
-}
 
-module.exports = function(req, res, next) {
-  const { hash } = req.params;
-  const pathParam = (req.params[0] || '').split('/').filter(Boolean);
-
-  const path = pathParam.length ? pathParam.join('/') + '/' : '';
-
-  return gitFileTree(hash, path).then(
-    list => {
-      const files = list.map(item => ({
-        ...item,
-        href: buildObjectUrl(hash, item),
-        name: item.path.split('/').pop()
-      }));
-
-      res.render('files', {
-        title: 'files',
-        breadcrumbs: buildBreadcrumbs(hash, pathParam.join('/')),
-        files
-      });
+  this.buildRenderData = () => {
+    return new Promise((res) => {
+      this.gitFileTree(hash, path).then(
+        list => {
+          const files = list.map(item => ({
+            ...item,
+            href: this.buildObjectUrl(hash, item),
+            name: item.path.split('/').pop()
+          }));
+
+          thisPathParam = pathParam && pathParam.length ? pathParam.join('/') : '';
+
+          res({
+            title: 'files',
+            breadcrumbs: this.buildBreadcrumb(hash, thisPathParam),
+            files
+          });
+        },
+        err => next(err)
+      );
+    });
+  }
+
+  if (req && res) {
+    this.buildRenderData(hash, path).then(data => {
+      if (data && res) {
+        res.render('files', data);
+      } else {
+        next();
+      }
     },
-    err => next(err)
-  );
+    err => next(err));
+  }
 };

controllers/indexController.js

@@ -1,20 +1,40 @@
 const { gitHistory } = require('../utils/git');
 const { buildFolderUrl, buildBreadcrumbs } = require('../utils/navigation');
 
-module.exports = function(req, res) {
-  gitHistory(1, 20).then(
-    history => {
-      const list = history.map(item => ({
-        ...item,
-        href: buildFolderUrl(item.hash, '')
-      }));
+module.exports = function (req, res) {
+  this.gitHistory = gitHistory;
+  this.buildFolderUrl = buildFolderUrl;
 
+  this.buildList = () => {
+    const thisGitHistory = this.gitHistory;
+    const thisBuildFolderUrl = this.buildFolderUrl;
+
+    return thisGitHistory(1, 20).then(
+      history => {
+        return history.map(item => ({
+          ...item,
+          href: thisBuildFolderUrl(item.hash, '')
+        }));
+      },
+      /**
+       * в рамках данного задания не предполагалось исправления существующего кода,
+       * только реорганизация, да и исходная задумка автора неизвестна.
+       * Потому это неопределённое next оставлю без изменений,
+       * хотя по-хорошему его тоже можно было бы протестировать.
+       */
+      err => next(err)
+    );
+  }
+
+  this.render = list => {
+    if (res) {
       res.render('index', {
         title: 'history',
         breadcrumbs: buildBreadcrumbs(),
         list
       });
-    },
-    err => next(err)
-  );
-};
+    }
+  }
+
+  this.buildList().then(list => this.render(list));
+};

hermione-html-report/index.html

@@ -0,0 +1,10 @@
+<!DOCTYPE html>
+<html>
+<head>
+<title>HTML report</title>
+<meta charset="utf-8">
+<link href="report.min.css" rel="stylesheet"></head>
+<body class="report">
+    <div id="app"></div>
+<script type="text/javascript" src="data.js"></script><script type="text/javascript" src="report.min.js"></script></body>
+</html>