Skip to content

Commit 7a40c1f

Browse files
committed
[14/1/23] fixed the GetAuthSession() refactor test
1 parent dc5e712 commit 7a40c1f

File tree

2 files changed

+38
-29
lines changed

2 files changed

+38
-29
lines changed

middleware/session_middleware.go

Lines changed: 1 addition & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
package middleware
22

33
import (
4-
"errors"
54
"os"
65

76
"github.com/gorilla/sessions"
@@ -18,33 +17,7 @@ import (
1817
func GetAuth(c echo.Context) (session_gorilla *sessions.Session, err error) {
1918
// Test: session_test = true
2019
if os.Getenv("session_test") == "1" && method.SetSession == false {
21-
if modelsTest.UserSelectTest == "" {
22-
session_gorilla = &sessions.Session{
23-
Values: map[interface{}]interface{}{
24-
"username": "",
25-
"is_auth_type": -1,
26-
},
27-
}
28-
29-
err = errors.New("no session")
30-
return
31-
}
32-
33-
for _, testUser := range modelsTest.UsersTest {
34-
if modelsTest.UserSelectTest == testUser.Username {
35-
session_gorilla = &sessions.Session{
36-
Values: map[interface{}]interface{}{
37-
"username": testUser.Username,
38-
},
39-
}
40-
41-
if testUser.IsAdmin == 1 {
42-
session_gorilla.Values["is_auth_type"] = 1 // admin: 1
43-
} else if testUser.IsAdmin == 0 {
44-
session_gorilla.Values["is_auth_type"] = 2 // user: 2
45-
}
46-
}
47-
}
20+
session_gorilla, err = modelsTest.GetAuthSession()
4821
} else {
4922
if session_gorilla, err = session.Get("session", c); err != nil {
5023
return

tests/models/user.go

Lines changed: 37 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,11 @@
11
package models
22

3-
import "github.com/ockibagusp/golang-website-example/models"
3+
import (
4+
"errors"
5+
6+
"github.com/gorilla/sessions"
7+
"github.com/ockibagusp/golang-website-example/models"
8+
)
49

510
var UserSelectTest string
611

@@ -30,3 +35,34 @@ func UserUsername(user *models.User) {
3035
}
3136
}
3237
}
38+
39+
func GetAuthSession() (session_gorilla *sessions.Session, err error) {
40+
if UserSelectTest == "" {
41+
session_gorilla = &sessions.Session{
42+
Values: map[interface{}]interface{}{
43+
"username": "",
44+
"is_auth_type": -1,
45+
},
46+
}
47+
48+
err = errors.New("no session")
49+
return
50+
}
51+
52+
for _, testUser := range UsersTest {
53+
if UserSelectTest == testUser.Username {
54+
session_gorilla = &sessions.Session{
55+
Values: map[interface{}]interface{}{
56+
"username": testUser.Username,
57+
},
58+
}
59+
60+
if testUser.IsAdmin == 1 {
61+
session_gorilla.Values["is_auth_type"] = 1 // admin: 1
62+
} else if testUser.IsAdmin == 0 {
63+
session_gorilla.Values["is_auth_type"] = 2 // user: 2
64+
}
65+
}
66+
}
67+
return
68+
}

0 commit comments

Comments
 (0)