Skip to content

Commit 92e259d

Browse files
committed
test task fix and updated version
1 parent f691111 commit 92e259d

File tree

5 files changed

+18
-4
lines changed

5 files changed

+18
-4
lines changed

CHANGELOG.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,8 @@
1+
## [0.3.18] - 12 October 2023
2+
- Extension version bump to 0.3.18
3+
- *Test Task* version bump to "0.3.4"
4+
- *Test Task* logic change to fix false negative test case
5+
16
## [0.3.17] - 12 October 2023
27
- Extension version bump to 0.3.17
38
- Updated support links in readme

tasks/test/index.js

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -141,7 +141,8 @@ function createTestCase(suite, output, globalFailure) {
141141
const testRunRegex = /\s*\d\d:\d\d (\+\d+)?(\s+\-\d+)?:\s*(.*)/;
142142
let match = testRunRegex.exec(output);
143143
if (match) {
144-
var title = match[3];
144+
var tSplits = match[3].split(': ');
145+
var title = tSplits[tSplits.length - 1];
145146
var successes = Number(match[1]);
146147
var failures = match[2] ? -Number(match[2]) : suite.failed;
147148
var newCase = {
@@ -170,6 +171,10 @@ function createTestCase(suite, output, globalFailure) {
170171
newCase.isSuccess = true;
171172
hasNewCase = true;
172173
}
174+
else {
175+
newCase.isSuccess = true;
176+
hasNewCase = true;
177+
}
173178
if (hasNewCase) {
174179
if (suite.cases.length > 0) {
175180
suite.cases[suite.cases.length - 1].ended = newCase.started;

tasks/test/index.ts

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -172,7 +172,8 @@ function createTestCase(suite: any, output: string, globalFailure: number) {
172172
const testRunRegex = /\s*\d\d:\d\d (\+\d+)?(\s+\-\d+)?:\s*(.*)/;
173173
let match = testRunRegex.exec(output);
174174
if (match) {
175-
var title = match[3];
175+
var tSplits = match[3].split(': ');
176+
var title = tSplits[tSplits.length - 1];
176177
var successes = Number(match[1]);
177178
var failures = match[2] ? -Number(match[2]) : suite.failed;
178179

@@ -203,6 +204,9 @@ function createTestCase(suite: any, output: string, globalFailure: number) {
203204
// everything is 0 meaning it's a new case
204205
newCase.isSuccess = true;
205206
hasNewCase = true;
207+
} else {
208+
newCase.isSuccess = true;
209+
hasNewCase = true;
206210
}
207211

208212
if (hasNewCase) {

tasks/test/task.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99
"version": {
1010
"Major": 0,
1111
"Minor": 3,
12-
"Patch": 3
12+
"Patch": 4
1313
},
1414
"instanceNameFormat": "Flutter Test",
1515
"inputs": [

vss-extension.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
{
22
"manifestVersion": 1,
33
"id": "flutter",
4-
"version": "0.3.17",
4+
"version": "0.3.18",
55
"name": "Flutter Tasks",
66
"description": "Flutter extension for Azure DevOps. Install, build, analyze, command and env tasks for easier Flutter DevOps.",
77
"publisher": "hey24sheep",

0 commit comments

Comments
 (0)