fix: add steps support

Author: KuznetsovRoman

lib/constants/database.js

@@ -7,6 +7,7 @@ const SUITES_TABLE_COLUMNS = [
     {name: 'name', type: DB_TYPES.text},
     {name: 'suiteUrl', type: DB_TYPES.text},
     {name: 'metaInfo', type: DB_TYPES.text},
+    {name: 'history', type: DB_TYPES.text},
     {name: 'description', type: DB_TYPES.text},
     {name: 'error', type: DB_TYPES.text},
     {name: 'skipReason', type: DB_TYPES.text},

lib/history-utils.js

@@ -1,32 +1,50 @@
 'use strict';
 
-const {last, isUndefined} = require('lodash');
+const {isEmpty} = require('lodash');
 
 const formatDuration = (d) => `<- ${d}ms`;
 
 const wrapArg = (arg) => `"${arg}"`;
 
 const getCommand = ({
     n: name,
-    a: args = [],
-    d: duration}
-) => `${name}(${args.map(wrapArg).join(', ')}) ${formatDuration(duration)}`;
+    a: args = []
+}) => `${name}(${args.map(wrapArg).join(', ')})`;
+
+const traverseNodes = (nodes, traverseCb, depth = 0) => {
+    nodes.forEach(node => {
+        const shouldTraverseChildren = traverseCb(node, depth);
+
+        if (shouldTraverseChildren) {
+            traverseNodes(node.c, traverseCb, depth + 1);
+        }
+    });
+};
 
 const getCommandsHistory = (history) => {
-    if (isUndefined(history)) {
+    if (isEmpty(history)) {
         return;
     }
 
     try {
-        const formatedCommands = history
-            .map(getCommand)
-            .map((s) => `\t${s}\n`);
-        const lastCommandChildren = last(history).c;
-        const formatedChildren = lastCommandChildren
-            .map((cmd) => `${getCommand(cmd)}`)
-            .map((s) => `\t\t${s}\n`);
-
-        return [...formatedCommands, ...formatedChildren];
+        const formatedHistory = [];
+
+        const traverseCb = (node, depth) => {
+            const offset = '\t'.repeat(depth);
+            const isStep = node.n === 'runStep';
+            const duration = node.d;
+            const isFailed = !!node.f;
+            const title = isStep ? node.a[0] : getCommand(node);
+            const formatedDuration = formatDuration(duration);
+
+            formatedHistory.push(`${offset}${title} ${formatedDuration}\n`);
+
+            return isFailed;
+        };
+
+        traverseNodes(history, traverseCb);
+
+        return formatedHistory;
     } catch (e) {
         return `failed to get command history: ${e.message}`;
     }

lib/report-builder/static.js

@@ -38,7 +38,7 @@ module.exports = class StaticReportBuilder {
 
         return result instanceof TestAdapter
             ? result
-            : TestAdapter.create(result, this._hermione, this._pluginConfig, status);
+            : TestAdapter.create(result, this._hermione, status);
     }
 
     async saveStaticFiles() {
@@ -111,16 +111,17 @@ module.exports = class StaticReportBuilder {
 
     _createTestResult(result, props) {
         const {
-            browserId, suite, sessionId, description, imagesInfo, screenshot, multipleTabs, errorDetails
+            browserId, suite, sessionId, description, history,
+            imagesInfo, screenshot, multipleTabs, errorDetails
         } = result;
 
         const {baseHost, saveErrorDetails} = this._pluginConfig;
         const suiteUrl = suite.getUrl({browserId, baseHost});
         const metaInfo = _.merge(_.cloneDeep(result.meta), {url: suite.fullUrl, file: suite.file, sessionId});
 
         const testResult = Object.assign({
-            suiteUrl, name: browserId, metaInfo, description, imagesInfo,
-            screenshot: Boolean(screenshot), multipleTabs
+            suiteUrl, name: browserId, metaInfo, description, history,
+            imagesInfo, screenshot: Boolean(screenshot), multipleTabs
         }, props);
 
         if (saveErrorDetails && errorDetails) {

lib/sqlite-adapter.js

@@ -66,6 +66,7 @@ module.exports = class SqliteAdapter {
             name,
             suiteUrl,
             metaInfo,
+            history,
             description,
             error,
             skipReason,
@@ -82,6 +83,7 @@ module.exports = class SqliteAdapter {
             name,
             suiteUrl,
             metaInfo,
+            history,
             description,
             error,
             skipReason,

lib/static/components/section/body/history/index.js

@@ -0,0 +1,29 @@
+import React from 'react';
+import {connect} from 'react-redux';
+import PropTypes from 'prop-types';
+import {isEmpty} from 'lodash';
+import Details from '../../../details';
+
+import './index.styl';
+
+const History = ({history}) => (
+    isEmpty(history)
+        ? null
+        : <Details
+            title='History'
+            content={history}
+            extendClassNames='details_type_text history'
+        />
+);
+
+History.propTypes = {
+    resultId: PropTypes.string.isRequired,
+    // from store
+    history: PropTypes.arrayOf(PropTypes.string).isRequired
+};
+
+export default connect(({tree}, {resultId}) => {
+    const {history = []} = tree.results.byId[resultId];
+
+    return {history};
+})(History);

lib/static/components/section/body/history/index.styl

@@ -0,0 +1,4 @@
+.history {
+    white-space: pre-wrap;
+    word-wrap: break-word;
+}

lib/static/components/section/body/result.js

@@ -2,6 +2,7 @@ import React, {Component, Fragment} from 'react';
 import {connect} from 'react-redux';
 import PropTypes from 'prop-types';
 import MetaInfo from './meta-info';
+import History from './history';
 import Description from './description';
 import Tabs from './tabs';
 import ExtensionPoint from '../../extension-point';
@@ -26,6 +27,7 @@ class Result extends Component {
             <Fragment>
                 <ExtensionPoint name={RESULT_META} result={result} testName={testName}>
                     <MetaInfo resultId={resultId} />
+                    <History resultId={resultId} />
                 </ExtensionPoint>
                 {result.description && <Description content={result.description} />}
                 <Tabs result={result} />

lib/test-adapter.js

@@ -27,14 +27,13 @@ const globalCacheDiffImages = new Map();
 const testsAttempts = new Map();
 
 module.exports = class TestAdapter {
-    static create(testResult = {}, hermione, pluginConfig, status) {
-        return new this(testResult, hermione, pluginConfig, status);
+    static create(testResult = {}, hermione, status) {
+        return new this(testResult, hermione, status);
     }
 
-    constructor(testResult, hermione, pluginConfig, status) {
+    constructor(testResult, hermione, status) {
         this._testResult = testResult;
         this._hermione = hermione;
-        this._pluginConfig = pluginConfig;
         this._errors = this._hermione.errors;
         this._suite = SuiteAdapter.create(this._testResult);
         this._imagesSaver = this._hermione.htmlReporter.imagesSaver;
@@ -220,16 +219,12 @@ module.exports = class TestAdapter {
         return this.imagesInfo;
     }
 
-    get error() {
-        const err = _.pick(this._testResult.err, ['message', 'stack', 'stateName']);
-        const {history} = this._testResult;
-        const {commandsWithShortHistory} = this._pluginConfig;
-
-        if (!_.isEmpty(history)) {
-            err.history = getCommandsHistory(history, commandsWithShortHistory);
-        }
+    get history() {
+        return getCommandsHistory(this._testResult.history);
+    }
 
-        return err;
+    get error() {
+        return _.pick(this._testResult.err, ['message', 'stack', 'stateName']);
     }
 
     get imageDir() {

lib/tests-tree-builder/static.js

@@ -154,6 +154,7 @@ function mkTestResult(row, data = {}) {
         description: row[DB_COLUMN_INDEXES.description],
         imagesInfo: JSON.parse(row[DB_COLUMN_INDEXES.imagesInfo]),
         metaInfo: JSON.parse(row[DB_COLUMN_INDEXES.metaInfo]),
+        history: JSON.parse(row[DB_COLUMN_INDEXES.history]),
         multipleTabs: Boolean(row[DB_COLUMN_INDEXES.multipleTabs]),
         name: row[DB_COLUMN_INDEXES.name],
         screenshot: Boolean(row[DB_COLUMN_INDEXES.screenshot]),