Skip to content

Commit 1164daf

Browse files
author
Stanislav Fedii
committed
Re-arranged panicObject handling.
1 parent 1c9aa91 commit 1164daf

File tree

2 files changed

+6
-7
lines changed

2 files changed

+6
-7
lines changed

before.go

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -29,20 +29,19 @@ func BeforeTest(t *testing.T, testOptions ...Option) {
2929

3030
defer func() {
3131
panicObject := recover()
32+
33+
beforeSubContainer.Stop = getTimestampMs()
34+
beforeSubContainer.Status = getTestStatus(t)
35+
beforeSubContainer.Stage = "finished"
36+
3237
if panicObject != nil {
3338
t.Fail()
3439
beforeSubContainer.StatusDetails = &statusDetails{
3540
Message: fmt.Sprintf("%+v", panicObject),
3641
Trace: filterStackTrace(debug.Stack()),
3742
}
3843
beforeSubContainer.Status = broken
39-
}
4044

41-
beforeSubContainer.Stop = getTimestampMs()
42-
beforeSubContainer.Status = getTestStatus(t)
43-
beforeSubContainer.Stage = "finished"
44-
45-
if panicObject != nil {
4645
r := newResult()
4746
r.Stop = getTimestampMs()
4847
r.Name = strings.Join(camelcase.Split(t.Name())[1:], " ")

example/example_before_fail_test.go renamed to example/example_before_panic_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ import (
55
"testing"
66
)
77

8-
func TestBeforeFail(t *testing.T) {
8+
func TestBeforePanic(t *testing.T) {
99
allure.BeforeTest(t, allure.Action(func() {
1010
panic("panic at the before statement! (disco)")
1111
}))

0 commit comments

Comments
 (0)