a7693cc · typeable/oauth2-proxy

git.iko.soy/typeable/oauth2-proxy

on master help sign in

Tranfser all cookies in tests

a7693ccJoel Speed7 years ago

pkg/sessions/session_store_test.go+10−5

compare with master

unifiedsplit

@@ -115,8 +115,9 @@ var _ = Describe("NewSessionStore", func() {

115 err := ss.Save(saveResp, req, session)

116 Expect(err).ToNot(HaveOccurred())

117

118− resultCookie := saveResp.Result().Cookies()[0]

118+ for _, c := range saveResp.Result().Cookies() {

119− request.AddCookie(resultCookie)

119+ request.AddCookie(c)

120+ }

120 err = ss.Clear(response, request)

121 err = ss.Clear(response, request)

121 Expect(err).ToNot(HaveOccurred())

122 Expect(err).ToNot(HaveOccurred())

122 })

123 })

@@ -177,14 +178,18 @@ var _ = Describe("NewSessionStore", func() {

177 err := ss.Save(saveResp, req, session)

178 err := ss.Save(saveResp, req, session)

178 Expect(err).ToNot(HaveOccurred())

179 Expect(err).ToNot(HaveOccurred())

179

180

180− resultCookie := saveResp.Result().Cookies()[0]

181+ resultCookies := saveResp.Result().Cookies()

182+ for _, c := range resultCookies {

181− request.AddCookie(resultCookie)

183+ request.AddCookie(c)

184+ }

182 err = ss.Clear(response, request)

185 err = ss.Clear(response, request)

183 Expect(err).ToNot(HaveOccurred())

186 Expect(err).ToNot(HaveOccurred())

184

187

185 // The following should only be for server stores

188 // The following should only be for server stores

186 loadReq := httptest.NewRequest("GET", "http://example.com/", nil)

189 loadReq := httptest.NewRequest("GET", "http://example.com/", nil)

190+ for _, c := range resultCookies {

187− loadReq.AddCookie(resultCookie)

191+ loadReq.AddCookie(c)

192+ }

188 loadedAfterClear, err = ss.Load(loadReq)

193 loadedAfterClear, err = ss.Load(loadReq)

189 })

194 })

190

195