Skip to content

Commit d0e65df

Browse files
wxiaoguangwxiaoguang
committed
fix
1 parent 300b724 commit d0e65df

File tree

5 files changed

+96
-31
lines changed

5 files changed

+96
-31
lines changed

Diff for: ‎models/actions/run_job.go

+24-19
Original file line numberDiff line numberDiff line change
@@ -153,28 +153,33 @@ func UpdateRunJob(ctx context.Context, job *ActionRunJob, cond builder.Cond, col
153153
}
154154

155155
func AggregateJobStatus(jobs []*ActionRunJob) Status {
156-
allDone := true
157-
allWaiting := true
158-
hasFailure := false
156+
allSuccessOrSkipped := true
157+
var hasFailure, hasCancelled, hasSkipped, hasWaiting, hasRunning, hasBlocked bool
159158
for _, job := range jobs {
160-
if !job.Status.IsDone() {
161-
allDone = false
162-
}
163-
if job.Status != StatusWaiting && !job.Status.IsDone() {
164-
allWaiting = false
165-
}
166-
if job.Status == StatusFailure || job.Status == StatusCancelled {
167-
hasFailure = true
168-
}
159+
allSuccessOrSkipped = allSuccessOrSkipped && (job.Status == StatusSuccess || job.Status == StatusSkipped)
160+
hasFailure = hasFailure || job.Status == StatusFailure
161+
hasCancelled = hasCancelled || job.Status == StatusCancelled
162+
hasSkipped = hasSkipped || job.Status == StatusSkipped
163+
hasWaiting = hasWaiting || job.Status == StatusWaiting
164+
hasRunning = hasRunning || job.Status == StatusRunning
165+
hasBlocked = hasBlocked || job.Status == StatusBlocked
169166
}
170-
if allDone {
171-
if hasFailure {
172-
return StatusFailure
173-
}
167+
switch {
168+
case allSuccessOrSkipped:
174169
return StatusSuccess
175-
}
176-
if allWaiting {
170+
case hasFailure:
171+
return StatusFailure
172+
case hasRunning:
173+
return StatusRunning
174+
case hasWaiting:
177175
return StatusWaiting
176+
case hasBlocked:
177+
return StatusBlocked
178+
case hasCancelled:
179+
return StatusCancelled
180+
case hasSkipped:
181+
return StatusSkipped
182+
default:
183+
return StatusUnknown // it shouldn't happen
178184
}
179-
return StatusRunning
180185
}

Diff for: ‎models/actions/run_job_status_test.go

+63
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,63 @@
1+
// Copyright 2024 The Gitea Authors. All rights reserved.
2+
// SPDX-License-Identifier: MIT
3+
4+
package actions
5+
6+
import (
7+
"testing"
8+
9+
"github.com/stretchr/testify/assert"
10+
)
11+
12+
func TestAggregateJobStatus(t *testing.T) {
13+
testStatuses := func(expected Status, statuses []Status) {
14+
var jobs []*ActionRunJob
15+
for _, v := range statuses {
16+
jobs = append(jobs, &ActionRunJob{Status: v})
17+
}
18+
actual := AggregateJobStatus(jobs)
19+
if !assert.Equal(t, expected, actual) {
20+
var statusStrings []string
21+
for _, s := range statuses {
22+
statusStrings = append(statusStrings, s.String())
23+
}
24+
t.Errorf("AggregateJobStatus(%v) = %v, want %v", statusStrings, statusNames[actual], statusNames[expected])
25+
}
26+
}
27+
28+
cases := []struct {
29+
statuses []Status
30+
expected Status
31+
}{
32+
// success with other status
33+
{[]Status{StatusSuccess}, StatusSuccess},
34+
{[]Status{StatusSuccess, StatusSkipped}, StatusSuccess}, // skipped doesn't affect success
35+
{[]Status{StatusSuccess, StatusFailure}, StatusFailure},
36+
{[]Status{StatusSuccess, StatusCancelled}, StatusCancelled},
37+
{[]Status{StatusSuccess, StatusWaiting}, StatusWaiting},
38+
{[]Status{StatusSuccess, StatusRunning}, StatusRunning},
39+
{[]Status{StatusSuccess, StatusBlocked}, StatusBlocked},
40+
41+
// failure with other status, fail fast (well, should "running" win?)
42+
{[]Status{StatusFailure}, StatusFailure},
43+
{[]Status{StatusFailure, StatusSuccess}, StatusFailure},
44+
{[]Status{StatusFailure, StatusSkipped}, StatusFailure},
45+
{[]Status{StatusFailure, StatusCancelled}, StatusFailure},
46+
{[]Status{StatusFailure, StatusWaiting}, StatusFailure},
47+
{[]Status{StatusFailure, StatusRunning}, StatusFailure},
48+
{[]Status{StatusFailure, StatusBlocked}, StatusFailure},
49+
50+
// skipped with other status
51+
{[]Status{StatusSkipped}, StatusSuccess},
52+
{[]Status{StatusSkipped, StatusSuccess}, StatusSuccess},
53+
{[]Status{StatusSkipped, StatusFailure}, StatusFailure},
54+
{[]Status{StatusSkipped, StatusCancelled}, StatusCancelled},
55+
{[]Status{StatusSkipped, StatusWaiting}, StatusWaiting},
56+
{[]Status{StatusSkipped, StatusRunning}, StatusRunning},
57+
{[]Status{StatusSkipped, StatusBlocked}, StatusBlocked},
58+
}
59+
60+
for _, c := range cases {
61+
testStatuses(c.expected, c.statuses)
62+
}
63+
}

Diff for: ‎templates/repo/actions/status.tmpl

+5-11
Original file line numberDiff line numberDiff line change
@@ -2,28 +2,22 @@
22
Please also update the vue file above if this template is modified.
33
action status accepted: success, skipped, waiting, blocked, running, failure, cancelled, unknown
44
-->
5-
{{- $size := 16 -}}
6-
{{- if .size -}}
7-
{{- $size = .size -}}
8-
{{- end -}}
9-
10-
{{- $className := "" -}}
11-
{{- if .className -}}
12-
{{- $className = .className -}}
13-
{{- end -}}
14-
5+
{{- $size := Iif .size 16 -}}
6+
{{- $className := Iif .className "" -}}
157
<span class="tw-flex tw-items-center" data-tooltip-content="{{ctx.Locale.Tr (printf "actions.status.%s" .status)}}">
168
{{if eq .status "success"}}
179
{{svg "octicon-check-circle-fill" $size (printf "text green %s" $className)}}
1810
{{else if eq .status "skipped"}}
1911
{{svg "octicon-skip" $size (printf "text grey %s" $className)}}
12+
{{else if eq .status "cancelled"}}
13+
{{svg "octicon-stop" $size (printf "text grey %s" $className)}}
2014
{{else if eq .status "waiting"}}
2115
{{svg "octicon-clock" $size (printf "text yellow %s" $className)}}
2216
{{else if eq .status "blocked"}}
2317
{{svg "octicon-blocked" $size (printf "text yellow %s" $className)}}
2418
{{else if eq .status "running"}}
2519
{{svg "octicon-meter" $size (printf "text yellow job-status-rotate %s" $className)}}
26-
{{else if or (eq .status "failure") or (eq .status "cancelled") or (eq .status "unknown")}}
20+
{{else}}{{/*failure, unknown*/}}
2721
{{svg "octicon-x-circle-fill" $size (printf "text red %s" $className)}}
2822
{{end}}
2923
</span>

Diff for: ‎web_src/js/components/ActionRunStatus.vue

+2-1
Original file line numberDiff line numberDiff line change
@@ -21,9 +21,10 @@ withDefaults(defineProps<{
2121
<span class="tw-flex tw-items-center" :data-tooltip-content="localeStatus" v-if="status">
2222
<SvgIcon name="octicon-check-circle-fill" class="text green" :size="size" :class-name="className" v-if="status === 'success'"/>
2323
<SvgIcon name="octicon-skip" class="text grey" :size="size" :class-name="className" v-else-if="status === 'skipped'"/>
24+
<SvgIcon name="octicon-stop" class="text yellow" :size="size" :class-name="className" v-else-if="status === 'cancelled'"/>
2425
<SvgIcon name="octicon-clock" class="text yellow" :size="size" :class-name="className" v-else-if="status === 'waiting'"/>
2526
<SvgIcon name="octicon-blocked" class="text yellow" :size="size" :class-name="className" v-else-if="status === 'blocked'"/>
2627
<SvgIcon name="octicon-meter" class="text yellow" :size="size" :class-name="'job-status-rotate ' + className" v-else-if="status === 'running'"/>
27-
<SvgIcon name="octicon-x-circle-fill" class="text red" :size="size" v-else-if="['failure', 'cancelled', 'unknown'].includes(status)"/>
28+
<SvgIcon name="octicon-x-circle-fill" class="text red" :size="size" v-else/><!-- failure, unknown -->
2829
</span>
2930
</template>

Diff for: ‎web_src/js/svg.ts

+2
Original file line numberDiff line numberDiff line change
@@ -65,6 +65,7 @@ import octiconSidebarCollapse from '../../public/assets/img/svg/octicon-sidebar-
6565
import octiconSidebarExpand from '../../public/assets/img/svg/octicon-sidebar-expand.svg';
6666
import octiconSkip from '../../public/assets/img/svg/octicon-skip.svg';
6767
import octiconStar from '../../public/assets/img/svg/octicon-star.svg';
68+
import octiconStop from '../../public/assets/img/svg/octicon-stop.svg';
6869
import octiconStrikethrough from '../../public/assets/img/svg/octicon-strikethrough.svg';
6970
import octiconSync from '../../public/assets/img/svg/octicon-sync.svg';
7071
import octiconTable from '../../public/assets/img/svg/octicon-table.svg';
@@ -140,6 +141,7 @@ const svgs = {
140141
'octicon-sidebar-expand': octiconSidebarExpand,
141142
'octicon-skip': octiconSkip,
142143
'octicon-star': octiconStar,
144+
'octicon-stop': octiconStop,
143145
'octicon-strikethrough': octiconStrikethrough,
144146
'octicon-sync': octiconSync,
145147
'octicon-table': octiconTable,

0 commit comments

Comments
 (0)