Skip to content
This repository has been archived by the owner on Dec 7, 2019. It is now read-only.

Add HTML report specification. #19

Merged
merged 4 commits into from
May 19, 2017
Merged

Add HTML report specification. #19

merged 4 commits into from
May 19, 2017

Conversation

arturdryomov
Copy link
Member

No description provided.

@artem-zinnatullin
Copy link
Collaborator

Please rename file to HTML_REPORT.md

spec/REPORT.md Outdated
* test suite success tests count;
* test suite failure tests count;
* test suite ignored tests count.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

test suite duration

* failure tests count;
* ignored tests count;
* duration.
* Search input.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(for now simple text search)

* Files:
* list of file links.
* Screenshots:
* list of screenshot images.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

grid

Copy link
Member Author

@arturdryomov arturdryomov Apr 14, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Welcome to the Grid!

@artem-zinnatullin artem-zinnatullin changed the title Report text description Add HTML report specification. Apr 14, 2017
@artem-zinnatullin
Copy link
Collaborator

Please change commit message to PR title (Report is an action and by looking at old PR title I thought you made some changes to composer behavior…)

@artem-zinnatullin
Copy link
Collaborator

Linking with #11.

@artem-zinnatullin
Copy link
Collaborator

cc @oguseynov please review

@artem-zinnatullin
Copy link
Collaborator

cc @squibms please review as well

spec/REPORT.md Outdated
* Test is a single functionality check.
* Properties:
* name;
* package name;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

* package name;
* class name;
* name;

?

spec/REPORT.md Outdated
* class name;
* duration;
* log;
* device name;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's more abstract thing than device and combines both emulators and real devices

spec/REPORT.md Outdated
* name;
* package name;
* class name;
* duration;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+ completion status: passed | failed | ignored;

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • optional stacktrace

spec/REPORT.md Outdated
* failure tests count;
* ignored tests count;
* log;
* duration.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+ list of devices;

spec/REPORT.md Outdated
* success tests count;
* failure tests count;
* ignored tests count;
* log;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

log per each device

spec/REPORT.md Outdated
* success tests count;
* failure tests count;
* ignored tests count;
* log;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

logcat

  • instrumentation output

spec/REPORT.md Outdated
* package name;
* class name;
* duration;
* log;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

logcat

@artem-zinnatullin artem-zinnatullin merged commit 9e7aad8 into master May 19, 2017
@artem-zinnatullin artem-zinnatullin deleted the ad/report-spec branch May 19, 2017 08:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants