Skip to content

Commit 8ca1fb5

Browse files
author
Jalander Ramagiri
committed
handle error case for refUpdated event translation
Signed-off-by: Jalander Ramagiri <[email protected]>
1 parent 790cd85 commit 8ca1fb5

File tree

2 files changed

+43
-40
lines changed

2 files changed

+43
-40
lines changed

pkg/gerrit/event_test.go

+40-40
Original file line numberDiff line numberDiff line change
@@ -1,40 +1,40 @@
1-
package gerrit
2-
3-
import (
4-
"net/http"
5-
"os"
6-
"testing"
7-
)
8-
9-
func TestHandleTranslateProjectCreatedEvent(t *testing.T) {
10-
event, err := os.ReadFile("testdata/project-created.json")
11-
if err != nil {
12-
t.Fatalf("Failed to read project-created.json file: %v", err)
13-
}
14-
headers := http.Header{}
15-
headers.Set("X-Origin-Url", "http://gerrit.est.tech")
16-
17-
cdEvent, err := HandleTranslateGerritEvent(string(event), headers)
18-
if err != nil {
19-
t.Errorf("Expected RepositoryCreated CDEvent to be successful.")
20-
return
21-
}
22-
Log().Info("Handle project-created gerrit event into dev.cdevents.repository.created is successful ", cdEvent)
23-
}
24-
25-
func TestHandleTranslateProjectHeadUpdatedEvent(t *testing.T) {
26-
27-
event, err := os.ReadFile("testdata/project-head-updated.json")
28-
if err != nil {
29-
t.Fatalf("Failed to read project-head-updated.json file: %v", err)
30-
}
31-
headers := http.Header{}
32-
headers.Set("X-Origin-Url", "http://gerrit.est.tech")
33-
34-
cdEvent, err := HandleTranslateGerritEvent(string(event), headers)
35-
if err != nil {
36-
t.Errorf("Expected RepositoryModified CDEvent to be successful.")
37-
return
38-
}
39-
Log().Info("Handle project-head-updated gerrit event into dev.cdevents.repository.modified is successful ", cdEvent)
40-
}
1+
package gerrit
2+
3+
import (
4+
"net/http"
5+
"os"
6+
"testing"
7+
)
8+
9+
func TestHandleTranslateProjectCreatedEvent(t *testing.T) {
10+
event, err := os.ReadFile("testdata/project-created.json")
11+
if err != nil {
12+
t.Fatalf("Failed to read project-created.json file: %v", err)
13+
}
14+
headers := http.Header{}
15+
headers.Set("X-Origin-Url", "http://gerrit.est.tech")
16+
17+
cdEvent, err := HandleTranslateGerritEvent(string(event), headers)
18+
if err != nil {
19+
t.Errorf("Expected RepositoryCreated CDEvent to be successful.")
20+
return
21+
}
22+
Log().Info("Handle project-created gerrit event into dev.cdevents.repository.created is successful ", cdEvent)
23+
}
24+
25+
func TestHandleTranslateProjectHeadUpdatedEvent(t *testing.T) {
26+
27+
event, err := os.ReadFile("testdata/project-head-updated.json")
28+
if err != nil {
29+
t.Fatalf("Failed to read project-head-updated.json file: %v", err)
30+
}
31+
headers := http.Header{}
32+
headers.Set("X-Origin-Url", "http://gerrit.est.tech")
33+
34+
cdEvent, err := HandleTranslateGerritEvent(string(event), headers)
35+
if err != nil {
36+
t.Errorf("Expected RepositoryModified CDEvent to be successful.")
37+
return
38+
}
39+
Log().Info("Handle project-head-updated gerrit event into dev.cdevents.repository.modified is successful ", cdEvent)
40+
}

pkg/gerrit/translate_events.go

+3
Original file line numberDiff line numberDiff line change
@@ -82,6 +82,9 @@ func (pEvent *GerritEvent) HandleRefUpdatedEvent() (string, error) {
8282
return "", err
8383
}
8484
Log().Info("Translated ref-updated gerrit event into dev.cdevents.branch.deleted CDEvent: ", cdEvent)
85+
} else {
86+
Log().Info("Ignoring handling ref-updated gerrit event for refName : ", refUpdated.RefUpdate.RefName)
87+
return "", errors.New("ignoring translating ref-updated gerrit event")
8588
}
8689

8790
return cdEvent, nil

0 commit comments

Comments
 (0)