Модульное тестирование. Иванов Н.С.

.hermione.conf.js

@@ -0,0 +1,17 @@
+module.exports = {
+    baseUrl: 'http://localhost:3000',
+
+    sets: {
+        desktop: {
+            files: 'tests/int'
+        }
+    },
+
+    browsers: {
+        chrome: {
+            desiredCapabilities: {
+                browserName: 'chrome'
+            }
+        }
+    }
+};

README.md

@@ -1,3 +1,61 @@
+# Комментарий
+
+Интеграционное тестирование:
+
+К сожалению, за данное время так и не успел реализовать интеграционное тестирование.
+На удаленных ресурсах тест проходит успешно, на поднятом локально селениумом сервере не видит элементов.
+
+
+Модульное тестирование:
+
+Логические блоки:
+1. gitHistory (получение истории коммитов):
+    * получаем массив с указанием всех аргументов;
+    * с указанием всех аргументов у объектов массива корректные данные;
+    * получаем массив без указания аргументов;
+    * без указания аргументов у объектов массива корректные данные.
+
+1. gitFileTree (получение списка файлов и директорий):
+    * получаем массив передав path;
+    * передав path у объектов массива корректные данные.
+
+1. buildFolderUrl (получение url папки):
+    * получаем строку;
+    * строка является корректной.
+
+1. buildFileUrl (получение url файла):
+    * получаем строку;
+    * строка является корректной.
+
+1. buildBreadcrumbs (получение хлебных крошек):
+    * при формировании с аргументами получаем строку;
+    * при формировании с аргументами получаем корректную строку;
+    * при формировании с одним аргументом получаем строку;
+    * при формировании с одним аргументом получаем корректную строку;
+    * при формировании без аргументов получаем строку;
+    * при формировании без аргументов получаем корректную строку.
+
+Запуск модульных тестов:
+* mocha "npm test";
+
+Запуск интеграционных тестов (tests/int/links.js). Запускаем 3 процесса:
+1. selenium-standalone start
+1. npm start 
+1. hermione
+
+
+
+# Ход работы
+1. Добавил в зависимости mocha и chai. Установил зависимости.
+1. Описал в readme логические блоки и сценарии тестирования.
+1. Переделал Git и Navigation на классы.
+1. Создал 2 файла с тестами, git-test и navigation-test в папке units.
+1. Создал заглушки в папке units/stubs.
+1. Написал модульные тесты.
+1. Написал часть интеграционных тестов.
+
+
+
 # Домашнее задание: автотесты
 
 Вам дано приложение на JavaScript и нужно написать для него автотесты: интеграционные тесты на интерфейс и модульные тесты на серверную часть.

app.js

@@ -25,23 +25,23 @@ app.get('/files/:hash/*?', filesController);
 app.get('/content/:hash/*?', contentController);
 
 // error handlers
-app.use(function(req, res, next) {
-  var err = new Error('Not Found');
-  err.status = 404;
-  next(err);
+app.use(function (req, res, next) {
+    var err = new Error('Not Found');
+    err.status = 404;
+    next(err);
 });
 
 // error handler
-app.use(function(err, req, res, next) {
-  const { status = 500, message } = err;
+app.use(function (err, req, res, next) {
+    const { status = 500, message } = err;
 
-  // render the error page
-  res.status(status);
-  res.render('error', { title: 'error', status, message });
+    // render the error page
+    res.status(status);
+    res.render('error', { title: 'error', status, message });
 });
 
 app.listen(PORT, HOST, () => {
-  console.log(`App listening at http://localhost:${PORT}`);
+    console.log(`App listening at http://localhost:${PORT}`);
 });
 
 module.exports = app;

controllers/contentController.js

@@ -1,28 +1,31 @@
-const { gitFileContent, gitFileTree } = require('../utils/git');
-const { buildFolderUrl, buildBreadcrumbs } = require('../utils/navigation');
+const Git = require('../utils/git');
+const Navigation = require('../utils/navigation');
 
-module.exports = function(req, res, next) {
-  const { hash } = req.params;
-  const path = req.params[0].split('/').filter(Boolean);
+const git = new Git();
+const navigation = new Navigation();
 
-  gitFileTree(hash, path.join('/'))
-    .then(function([file]) {
-      if (file && file.type === 'blob') {
-        return gitFileContent(file.hash);
-      }
-    })
-    .then(
-      content => {
-        if (content) {
-          res.render('content', {
-            title: 'content',
-            breadcrumbs: buildBreadcrumbs(hash, path.join('/')),
-            content
-          });
-        } else {
-          next();
-        }
-      },
-      err => next(err)
-    );
+module.exports = function (req, res, next) {
+    const { hash } = req.params;
+    const path = req.params[0].split('/').filter(Boolean);
+
+    git.gitFileTree(hash, path.join('/'))
+        .then(function ([file]) {
+            if (file && file.type === 'blob') {
+                return git.gitFileContent(file.hash);
+            }
+        })
+        .then(
+            content => {
+                if (content) {
+                    res.render('content', {
+                        title: 'content',
+                        breadcrumbs: navigation.buildBreadcrumbs(hash, path.join('/')),
+                        content
+                    });
+                } else {
+                    next();
+                }
+            },
+            err => next(err)
+        );
 };

controllers/filesController.js

@@ -1,41 +1,40 @@
-const { gitFileTree } = require('../utils/git');
-const {
-  buildFolderUrl,
-  buildFileUrl,
-  buildBreadcrumbs
-} = require('../utils/navigation');
+const Git = require('../utils/git');
+const Navigation = require('../utils/navigation');
+
+const git = new Git();
+const navigation = new Navigation();
 
 function buildObjectUrl(parentHash, { path, type }) {
-  switch (type) {
-    case 'tree':
-      return buildFolderUrl(parentHash, path);
-    case 'blob':
-      return buildFileUrl(parentHash, path);
-    default:
-      return '#';
-  }
+    switch (type) {
+        case 'tree':
+            return navigation.buildFolderUrl(parentHash, path);
+        case 'blob':
+            return navigation.buildFileUrl(parentHash, path);
+        default:
+            return '#';
+    }
 }
 
-module.exports = function(req, res, next) {
-  const { hash } = req.params;
-  const pathParam = (req.params[0] || '').split('/').filter(Boolean);
+module.exports = function (req, res, next) {
+    const { hash } = req.params;
+    const pathParam = (req.params[0] || '').split('/').filter(Boolean);
 
-  const path = pathParam.length ? pathParam.join('/') + '/' : '';
+    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()
-      }));
+    return git.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
-      });
-    },
-    err => next(err)
-  );
+            res.render('files', {
+                title: 'files',
+                breadcrumbs: navigation.buildBreadcrumbs(hash, pathParam.join('/')),
+                files
+            });
+        },
+        err => next(err)
+    );
 };

controllers/indexController.js

@@ -1,20 +1,23 @@
-const { gitHistory } = require('../utils/git');
-const { buildFolderUrl, buildBreadcrumbs } = require('../utils/navigation');
+const Git = require('../utils/git');
+const Navigation = require('../utils/navigation');
 
-module.exports = function(req, res) {
-  gitHistory(1, 20).then(
-    history => {
-      const list = history.map(item => ({
-        ...item,
-        href: buildFolderUrl(item.hash, '')
-      }));
+const git = new Git();
+const navigation = new Navigation();
 
-      res.render('index', {
-        title: 'history',
-        breadcrumbs: buildBreadcrumbs(),
-        list
-      });
-    },
-    err => next(err)
-  );
+module.exports = function (req, res) {
+    git.gitHistory(1, 20).then(
+        history => {
+            const list = history.map(item => ({
+                ...item,
+                href: navigation.buildFolderUrl(item.hash, '')
+            }));
+
+            res.render('index', {
+                title: 'history',
+                breadcrumbs: navigation.buildBreadcrumbs(),
+                list
+            });
+        },
+        err => next(err)
+    );
 };