Skip to content

Commit e067a98

Browse files
VladislavKostyukevichVladislavKostyukevich
authored
Merge pull request #89 from aquality-automation/feature/last_results_exclude_debug
Last results exclude debug
2 parents 25fbe5d + 7d3d57e commit e067a98

File tree

21 files changed

+223
-94
lines changed

21 files changed

+223
-94
lines changed

‎CHANGELOG.md‎

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,13 @@
11
# CHANGELOG
22

3+
## 0.3.7 (unreleased)
4+
5+
Features:
6+
- Improve Test Run and Test List page performance -> [View Issue](https://github.com/aquality-automation/aquality-tracking/issues/71)
7+
- Exclude Debug results from last results column -> [View Issue](https://github.com/aquality-automation/aquality-tracking/issues/70)
8+
9+
Bugfixes:
10+
311
## 0.3.6 (2020-02-24)
412

513
Features:

‎e2e/api/editor.api.ts‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@ import { Milestone } from '../../src/app/shared/models/milestone';
66
import { sendPost, sendGet, sendDelete } from '../utils/aqualityTrackingAPI.util';
77
import { TestResult } from '../../src/app/shared/models/test-result';
88
import { Project } from '../../src/app/shared/models/project';
9+
import { ResultResolution } from '../../src/app/shared/models/result_resolution';
910

1011
enum Endpoints {
1112
suite = '/suite',
@@ -53,6 +54,7 @@ export class EditorAPI {
5354
}
5455

5556
public async createResult(testResult: TestResult): Promise<TestResult> {
57+
testResult.project_id = this.project.id;
5658
return sendPost(Endpoints.testresult, undefined, testResult, this.token, this.project.id);
5759
}
5860

‎e2e/data/expected/testRunSpecialSymbols.csv‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
11
Test Name,Fail Reason,Last Results,Result,Resolution,Assignee,Comment
2-
"Test Feature with all results: step failed","step was failed !”#$%&’()*+,-./:;<=>?@[\]^_`{|}~","","Failed","Not Assigned",,
3-
"Test Feature with all results: All passed","","","Passed",,,
4-
"Test Feature with all results: Step skipped","step was skippedstep was skipped","","Pending","Not Assigned",,
2+
"Test Feature with all results: step failed","step was failed !”#$%&’()*+,-./:;<=>?@[\]^_`{|}~","3","Failed","Not Assigned",,
3+
"Test Feature with all results: All passed","","5","Passed",,,
4+
"Test Feature with all results: Step skipped","step was skippedstep was skipped","3","Pending","Not Assigned",,

‎e2e/data/resultsTable/nunitV3ClassName.csv‎

Lines changed: 24 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -1,25 +1,25 @@
11
Test Name,Fail Reason,Last Results,Result,Resolution,Assignee,Comment
2-
"Example.Tests.Class1.AttachmentsAreOptional","","","Passed",,,
3-
"Example.Tests.Class1.Test1","","","Passed",,,
4-
"Example.Tests.Class1.Test2","","","Passed",,,
5-
"Example.Tests.Class1.Test3","Test exceeded Timeout value of 1000ms","","Failed","Not Assigned",,
6-
"Example.Tests.Class1.Test4","Assertion: Expected: True But was: False ","","Failed","Not Assigned",,
7-
"Example.Tests.Class2.IgnoredTest","My ignore reason","","Not Executed","Not Assigned",,
8-
"Example.Tests.Class2.SyncTest","","","Passed",,,
9-
"Example.Tests.Class2.Test1","","","Passed",,,
10-
"Example.Tests.Class2.Test2","Assertion: My warn message","","Failed","Not Assigned",,
11-
"Example.Tests.Class2.Test3","Assertion: test failed.","","Failed","Not Assigned",,
12-
"Example.Tests.Class2.Test3FailedWithoutMessage","Failed without any message. Please see logs on result page.","","Failed","Not Assigned",,
13-
"Example.Tests.Class2.Test4","","","Passed",,,
14-
"Example.Tests.Class2.TestWithTestCaseSource(0)","","","Passed",,,
15-
"Example.Tests.Class2.TestWithTestCaseSource(1)","","","Passed",,,
16-
"Example.Tests.Class2.TestWithTestCaseSource(2)","","","Passed",,,
17-
"Example.Tests.Class3.Test1","OneTimeSetUp: System.Exception : Setup exception.","","Failed","Not Assigned",,
18-
"Example.Tests.Class3.Test2","OneTimeSetUp: System.Exception : Setup exception.","","Failed","Not Assigned",,
19-
"Example.Tests.Class3.Test3","OneTimeSetUp: System.Exception : Setup exception.","","Failed","Not Assigned",,
20-
"Example.Tests.FailedClass.Test1","OneTimeSetUp: System.Exception : Here is OneTimeSetup exception.","","Failed","Not Assigned",,
21-
"Example.Tests.FailedClass.Test2","OneTimeSetUp: System.Exception : Here is OneTimeSetup exception.","","Failed","Not Assigned",,
22-
"Example.Tests.IgnoredClass.Test1","OneTimeSetUp: Ignore reason for suite","","Not Executed","Not Assigned",,
23-
"Example.Tests.IgnoredClass.Test2","OneTimeSetUp: Ignore reason for suite","","Not Executed","Not Assigned",,
24-
"Example.Tests.InnerFolder.Class1.Test1","OneTimeSetUp: System.Exception : Assembly SetUpFixture exception.","","Failed","Not Assigned",,
25-
"Example.Tests.InnerFolder.Class1.Test2","OneTimeSetUp: System.Exception : Assembly SetUpFixture exception.","","Failed","Not Assigned",,
2+
"Example.Tests.Class1.AttachmentsAreOptional","","5","Passed",,,
3+
"Example.Tests.Class1.Test1","","5","Passed",,,
4+
"Example.Tests.Class1.Test2","","5","Passed",,,
5+
"Example.Tests.Class1.Test3","Test exceeded Timeout value of 1000ms","3","Failed","Not Assigned",,
6+
"Example.Tests.Class1.Test4","Assertion: Expected: True But was: False ","3","Failed","Not Assigned",,
7+
"Example.Tests.Class2.IgnoredTest","My ignore reason","3","Not Executed","Not Assigned",,
8+
"Example.Tests.Class2.SyncTest","","5","Passed",,,
9+
"Example.Tests.Class2.Test1","","5","Passed",,,
10+
"Example.Tests.Class2.Test2","Assertion: My warn message","3","Failed","Not Assigned",,
11+
"Example.Tests.Class2.Test3","Assertion: test failed.","3","Failed","Not Assigned",,
12+
"Example.Tests.Class2.Test3FailedWithoutMessage","Failed without any message. Please see logs on result page.","3","Failed","Not Assigned",,
13+
"Example.Tests.Class2.Test4","","5","Passed",,,
14+
"Example.Tests.Class2.TestWithTestCaseSource(0)","","5","Passed",,,
15+
"Example.Tests.Class2.TestWithTestCaseSource(1)","","5","Passed",,,
16+
"Example.Tests.Class2.TestWithTestCaseSource(2)","","5","Passed",,,
17+
"Example.Tests.Class3.Test1","OneTimeSetUp: System.Exception : Setup exception.","3","Failed","Not Assigned",,
18+
"Example.Tests.Class3.Test2","OneTimeSetUp: System.Exception : Setup exception.","3","Failed","Not Assigned",,
19+
"Example.Tests.Class3.Test3","OneTimeSetUp: System.Exception : Setup exception.","3","Failed","Not Assigned",,
20+
"Example.Tests.FailedClass.Test1","OneTimeSetUp: System.Exception : Here is OneTimeSetup exception.","3","Failed","Not Assigned",,
21+
"Example.Tests.FailedClass.Test2","OneTimeSetUp: System.Exception : Here is OneTimeSetup exception.","3","Failed","Not Assigned",,
22+
"Example.Tests.IgnoredClass.Test1","OneTimeSetUp: Ignore reason for suite","3","Not Executed","Not Assigned",,
23+
"Example.Tests.IgnoredClass.Test2","OneTimeSetUp: Ignore reason for suite","3","Not Executed","Not Assigned",,
24+
"Example.Tests.InnerFolder.Class1.Test1","OneTimeSetUp: System.Exception : Assembly SetUpFixture exception.","3","Failed","Not Assigned",,
25+
"Example.Tests.InnerFolder.Class1.Test2","OneTimeSetUp: System.Exception : Assembly SetUpFixture exception.","3","Failed","Not Assigned",,

‎e2e/data/resultsTable/nunitV3FeatureName.csv‎

Lines changed: 24 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -1,25 +1,25 @@
11
Test Name,Fail Reason,Last Results,Result,Resolution,Assignee,Comment
2-
"Class1: AttachmentsAreOptional","","","Passed",,,
3-
"Class1: Test1","","","Passed",,,
4-
"Class1: Test1","OneTimeSetUp: System.Exception : Assembly SetUpFixture exception.","","Failed","Not Assigned",,
5-
"Class1: Test2","","","Passed",,,
6-
"Class1: Test2","OneTimeSetUp: System.Exception : Assembly SetUpFixture exception.","","Failed","Not Assigned",,
7-
"Class1: Test3","Test exceeded Timeout value of 1000ms","","Failed","Not Assigned",,
8-
"Class1: Test4","Assertion: Expected: True But was: False ","","Failed","Not Assigned",,
9-
"Class2: IgnoredTest","My ignore reason","","Not Executed","Not Assigned",,
10-
"Class2: SyncTest","","","Passed",,,
11-
"Class2: Test1","","","Passed",,,
12-
"Class2: Test2","Assertion: My warn message","","Failed","Not Assigned",,
13-
"Class2: Test3","Assertion: test failed.","","Failed","Not Assigned",,
14-
"Class2: Test3FailedWithoutMessage","Failed without any message. Please see logs on result page.","","Failed","Not Assigned",,
15-
"Class2: Test4","","","Passed",,,
16-
"Class2: TestWithTestCaseSource(0)","","","Passed",,,
17-
"Class2: TestWithTestCaseSource(1)","","","Passed",,,
18-
"Class2: TestWithTestCaseSource(2)","","","Passed",,,
19-
"Class3: Test1","OneTimeSetUp: System.Exception : Setup exception.","","Failed","Not Assigned",,
20-
"Class3: Test2","OneTimeSetUp: System.Exception : Setup exception.","","Failed","Not Assigned",,
21-
"Class3: Test3","OneTimeSetUp: System.Exception : Setup exception.","","Failed","Not Assigned",,
22-
"FailedClass: Test1","OneTimeSetUp: System.Exception : Here is OneTimeSetup exception.","","Failed","Not Assigned",,
23-
"FailedClass: Test2","OneTimeSetUp: System.Exception : Here is OneTimeSetup exception.","","Failed","Not Assigned",,
24-
"IgnoredClass: Test1","OneTimeSetUp: Ignore reason for suite","","Not Executed","Not Assigned",,
25-
"IgnoredClass: Test2","OneTimeSetUp: Ignore reason for suite","","Not Executed","Not Assigned",,
2+
"Class1: AttachmentsAreOptional","","5","Passed",,,
3+
"Class1: Test1","","5,3","Passed",,,
4+
"Class1: Test1","OneTimeSetUp: System.Exception : Assembly SetUpFixture exception.","5,3","Failed","Not Assigned",,
5+
"Class1: Test2","","5,3","Passed",,,
6+
"Class1: Test2","OneTimeSetUp: System.Exception : Assembly SetUpFixture exception.","5,3","Failed","Not Assigned",,
7+
"Class1: Test3","Test exceeded Timeout value of 1000ms","3","Failed","Not Assigned",,
8+
"Class1: Test4","Assertion: Expected: True But was: False ","3","Failed","Not Assigned",,
9+
"Class2: IgnoredTest","My ignore reason","3","Not Executed","Not Assigned",,
10+
"Class2: SyncTest","","5","Passed",,,
11+
"Class2: Test1","","5","Passed",,,
12+
"Class2: Test2","Assertion: My warn message","3","Failed","Not Assigned",,
13+
"Class2: Test3","Assertion: test failed.","3","Failed","Not Assigned",,
14+
"Class2: Test3FailedWithoutMessage","Failed without any message. Please see logs on result page.","3","Failed","Not Assigned",,
15+
"Class2: Test4","","5","Passed",,,
16+
"Class2: TestWithTestCaseSource(0)","","5","Passed",,,
17+
"Class2: TestWithTestCaseSource(1)","","5","Passed",,,
18+
"Class2: TestWithTestCaseSource(2)","","5","Passed",,,
19+
"Class3: Test1","OneTimeSetUp: System.Exception : Setup exception.","3","Failed","Not Assigned",,
20+
"Class3: Test2","OneTimeSetUp: System.Exception : Setup exception.","3","Failed","Not Assigned",,
21+
"Class3: Test3","OneTimeSetUp: System.Exception : Setup exception.","3","Failed","Not Assigned",,
22+
"FailedClass: Test1","OneTimeSetUp: System.Exception : Here is OneTimeSetup exception.","3","Failed","Not Assigned",,
23+
"FailedClass: Test2","OneTimeSetUp: System.Exception : Here is OneTimeSetup exception.","3","Failed","Not Assigned",,
24+
"IgnoredClass: Test1","OneTimeSetUp: Ignore reason for suite","3","Not Executed","Not Assigned",,
25+
"IgnoredClass: Test2","OneTimeSetUp: Ignore reason for suite","3","Not Executed","Not Assigned",,

‎e2e/elements/base.element.ts‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ export class BaseElement {
1616
}
1717

1818
async isVisible(): Promise<boolean> {
19-
return waiter.forTrue(() => this.element.isPresent(), 2, 500);
19+
return waiter.forTrue(() => this.element.isPresent(), 5, 500);
2020
}
2121

2222
async isPresent(): Promise<boolean> {

‎e2e/elements/dots.element.ts‎

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,13 @@
1+
import { ElementFinder, Locator, Key, by } from 'protractor';
2+
import { BaseElement } from './base.element';
3+
4+
export class Dots extends BaseElement {
5+
6+
constructor(locatorOrElement: Locator | ElementFinder) {
7+
super(locatorOrElement);
8+
}
9+
10+
public async getDotsCount() {
11+
return (await this.element.all(by.tagName('span'))).length;
12+
}
13+
}

‎e2e/elements/smartTable.element/index.ts‎

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@ import { Paginator } from './paginator.element';
88
import { Row, CellElements } from './row.element';
99
import { ManageColumns } from './manageCollumns.element';
1010
import { compareCSVStrings } from '../../utils/csv.util';
11+
import { Dots } from '../dots.element';
1112

1213
const EC = protractor.ExpectedConditions;
1314

@@ -250,6 +251,13 @@ export class SmartTable extends BaseElement {
250251
return rowElements.lookup();
251252
}
252253

254+
public async getCellDots(column: string, searchValue: string, searchColumn: string): Promise<Dots> {
255+
const columnIndex = await this.getColumnIndex(column);
256+
const row = await this.getRow(searchValue, searchColumn);
257+
const rowElements = await row.getRowElements(columnIndex);
258+
return rowElements.dots();
259+
}
260+
253261
public async isRowEditableByValue(searchValue: string, searchColumn: string) {
254262
const row = await this.getRow(searchValue, searchColumn);
255263
return row.isRowEditable();

‎e2e/elements/smartTable.element/row.element.ts‎

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@ import { InlineEditor } from '../inlineEditor.element';
88
import { logger } from '../../utils/log.util';
99
import { InlineAttach } from '../inlineAttach.element';
1010
import { Multiselect } from '../multiselect.element';
11+
import { Dots } from '../dots.element';
1112

1213
export class Row extends BaseElement {
1314
constructor(locator: ElementFinder | Locator) {
@@ -58,7 +59,8 @@ export class Row extends BaseElement {
5859
inlineEditor: () => new InlineEditor(cell.element(by.tagName('inline-editor'))),
5960
lookup: () => new Lookup(cell.element(by.xpath('.//lookup-colored'))),
6061
inlineAttachment: () => new InlineAttach(cell.element(by.xpath('.//attachment-inline'))),
61-
multiselect: () => new Multiselect(cell.element(by.xpath('.//lookup-autocomplete-multiselect')))
62+
multiselect: () => new Multiselect(cell.element(by.xpath('.//lookup-autocomplete-multiselect'))),
63+
dots: () => new Dots(cell.element(by.xpath('.//app-color-dots')))
6264
};
6365
}
6466

@@ -196,4 +198,5 @@ export class CellElements {
196198
lookup: () => Lookup;
197199
inlineAttachment: () => InlineAttach;
198200
multiselect: () => Multiselect;
201+
dots: () => Dots;
199202
}

‎e2e/pages/administration/projectSettings.po/constants.ts‎

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,8 @@ export const elements = {
1010
stepsSwitcher: new UiSwitch(by.css('#steps-feature ui-switch')),
1111
importCompareResultsPattern: new Input(by.css('#compare-result-pattern-feature input')),
1212
projectSelector: new Autocomplete(by.id('project-selector')),
13-
saveFeatures: element(by.id('save-features'))
13+
saveFeatures: element(by.id('save-features')),
14+
stabilityResultsCount: new Input(by.css('#stability_count input'))
1415
};
1516

1617
export const names = {

0 commit comments

Comments
 (0)