fix(tests): improve error handling in unit tests and Makefile
- Enhanced the Makefile to include additional checks for test failures, specifically for panic messages and general failure indicators. - Updated service tests to dynamically retrieve the server port, ensuring accurate request routing. - Added a clear function call in the dashboard process test setup to ensure a clean state before tests run.
This commit is contained in:
parent
5c1664e882
commit
15a71692a2
3 changed files with 26 additions and 4 deletions
18
Makefile
18
Makefile
|
|
@ -41,6 +41,12 @@ unit-test:
|
||||||
if grep -q "^--- FAIL" tmp.out; then \
|
if grep -q "^--- FAIL" tmp.out; then \
|
||||||
rm tmp.out; \
|
rm tmp.out; \
|
||||||
exit 1; \
|
exit 1; \
|
||||||
|
elif grep -q "^FAIL" tmp.out; then \
|
||||||
|
rm tmp.out; \
|
||||||
|
exit 1; \
|
||||||
|
elif grep -q "^panic:" tmp.out; then \
|
||||||
|
rm tmp.out; \
|
||||||
|
exit 1; \
|
||||||
elif grep -q "build failed" tmp.out; then \
|
elif grep -q "build failed" tmp.out; then \
|
||||||
rm tmp.out; \
|
rm tmp.out; \
|
||||||
exit 1; \
|
exit 1; \
|
||||||
|
|
@ -67,6 +73,12 @@ unit-test-core:
|
||||||
if grep -q "^--- FAIL" tmp.out; then \
|
if grep -q "^--- FAIL" tmp.out; then \
|
||||||
rm tmp.out; \
|
rm tmp.out; \
|
||||||
exit 1; \
|
exit 1; \
|
||||||
|
elif grep -q "^FAIL" tmp.out; then \
|
||||||
|
rm tmp.out; \
|
||||||
|
exit 1; \
|
||||||
|
elif grep -q "^panic:" tmp.out; then \
|
||||||
|
rm tmp.out; \
|
||||||
|
exit 1; \
|
||||||
elif grep -q "build failed" tmp.out; then \
|
elif grep -q "build failed" tmp.out; then \
|
||||||
rm tmp.out; \
|
rm tmp.out; \
|
||||||
exit 1; \
|
exit 1; \
|
||||||
|
|
@ -359,6 +371,12 @@ unit-test-grpc:
|
||||||
if grep -q "^--- FAIL" tmp.out; then \
|
if grep -q "^--- FAIL" tmp.out; then \
|
||||||
rm tmp.out; \
|
rm tmp.out; \
|
||||||
exit 1; \
|
exit 1; \
|
||||||
|
elif grep -q "^FAIL" tmp.out; then \
|
||||||
|
rm tmp.out; \
|
||||||
|
exit 1; \
|
||||||
|
elif grep -q "^panic:" tmp.out; then \
|
||||||
|
rm tmp.out; \
|
||||||
|
exit 1; \
|
||||||
elif grep -q "build failed" tmp.out; then \
|
elif grep -q "build failed" tmp.out; then \
|
||||||
rm tmp.out; \
|
rm tmp.out; \
|
||||||
exit 1; \
|
exit 1; \
|
||||||
|
|
|
||||||
|
|
@ -33,10 +33,13 @@ func TestStartStop(t *testing.T) {
|
||||||
}
|
}
|
||||||
defer srv.Stop()
|
defer srv.Stop()
|
||||||
|
|
||||||
<-srv.Event()
|
port, err := srv.http.Port()
|
||||||
|
if err != nil {
|
||||||
|
t.Fatal(err)
|
||||||
|
}
|
||||||
|
|
||||||
// API Server
|
// API Server
|
||||||
req := test.NewRequest(cfg.Port).Route("/api/__yao/app/setting")
|
req := test.NewRequest(port).Route("/api/__yao/app/setting")
|
||||||
res, err := req.Get()
|
res, err := req.Get()
|
||||||
if err != nil {
|
if err != nil {
|
||||||
t.Fatal(err)
|
t.Fatal(err)
|
||||||
|
|
@ -49,7 +52,7 @@ func TestStartStop(t *testing.T) {
|
||||||
assert.True(t, len(data["name"].(string)) > 0)
|
assert.True(t, len(data["name"].(string)) > 0)
|
||||||
|
|
||||||
// Public
|
// Public
|
||||||
req = test.NewRequest(cfg.Port).Route("/")
|
req = test.NewRequest(port).Route("/")
|
||||||
res, err = req.Get()
|
res, err = req.Get()
|
||||||
if err != nil {
|
if err != nil {
|
||||||
t.Fatal(err)
|
t.Fatal(err)
|
||||||
|
|
@ -58,7 +61,7 @@ func TestStartStop(t *testing.T) {
|
||||||
assert.Equal(t, "Hello World\n", res.Body())
|
assert.Equal(t, "Hello World\n", res.Body())
|
||||||
|
|
||||||
// XGEN
|
// XGEN
|
||||||
req = test.NewRequest(cfg.Port).Route("/admin/")
|
req = test.NewRequest(port).Route("/admin/")
|
||||||
res, err = req.Get()
|
res, err = req.Get()
|
||||||
if err != nil {
|
if err != nil {
|
||||||
t.Fatal(err)
|
t.Fatal(err)
|
||||||
|
|
|
||||||
|
|
@ -32,6 +32,7 @@ func TestProcessComponent(t *testing.T) {
|
||||||
test.Prepare(t, config.Conf)
|
test.Prepare(t, config.Conf)
|
||||||
defer test.Clean()
|
defer test.Clean()
|
||||||
prepare(t)
|
prepare(t)
|
||||||
|
clear(t)
|
||||||
testData(t)
|
testData(t)
|
||||||
|
|
||||||
args := []interface{}{
|
args := []interface{}{
|
||||||
|
|
|
||||||
Loading…
Add table
Reference in a new issue