Skip to content

Commit 8e29a13

Browse files
suwakeindeloof
authored andcommitted
improved test
Signed-off-by: keitosuwahara <[email protected]>
1 parent 3c8da0a commit 8e29a13

File tree

1 file changed

+20
-18
lines changed

1 file changed

+20
-18
lines changed

pkg/progress/json_test.go

Lines changed: 20 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -44,19 +44,21 @@ func TestJsonWriter_Event(t *testing.T) {
4444
}
4545
w.Event(event)
4646

47-
var msg jsonMessage
48-
err := json.Unmarshal(out.Bytes(), &msg)
47+
var actual jsonMessage
48+
err := json.Unmarshal(out.Bytes(), &actual)
4949
assert.NilError(t, err)
5050

51-
assert.Equal(t, true, msg.DryRun)
52-
assert.Equal(t, false, msg.Tail)
53-
assert.Equal(t, "service1", msg.ID)
54-
assert.Equal(t, "project", msg.ParentID)
55-
assert.Equal(t, "Creating", msg.Text)
56-
assert.Equal(t, "Working", msg.Status)
57-
assert.Equal(t, int64(50), msg.Current)
58-
assert.Equal(t, int64(100), msg.Total)
59-
assert.Equal(t, 50, msg.Percent)
51+
expected := jsonMessage{
52+
DryRun: true,
53+
ID: event.ID,
54+
ParentID: event.ParentID,
55+
Text: event.Text,
56+
Status: event.StatusText,
57+
Current: event.Current,
58+
Total: event.Total,
59+
Percent: event.Percent,
60+
}
61+
assert.DeepEqual(t, expected, actual)
6062
}
6163

6264
func TestJsonWriter_TailMsgf(t *testing.T) {
@@ -75,13 +77,13 @@ func TestJsonWriter_TailMsgf(t *testing.T) {
7577

7678
w.Stop()
7779

78-
var msg jsonMessage
79-
err := json.Unmarshal(out.Bytes(), &msg)
80+
var actual jsonMessage
81+
err := json.Unmarshal(out.Bytes(), &actual)
8082
assert.NilError(t, err)
8183

82-
assert.Equal(t, false, msg.DryRun)
83-
assert.Equal(t, true, msg.Tail)
84-
assert.Equal(t, "hello world", msg.Text)
85-
assert.Equal(t, "", msg.ID)
86-
assert.Equal(t, "", msg.Status)
84+
expected := jsonMessage{
85+
Tail: true,
86+
Text: "hello world",
87+
}
88+
assert.DeepEqual(t, expected, actual)
8789
}

0 commit comments

Comments
 (0)