fix: improve error message when _session_store is missing from context (#67)

* fix: add proper error message when _session_store is missing from context

We need to know when _session_store is missing from context and the session
middleware is not properly initialized.

Author: imclem

session/session.go

@@ -36,7 +36,7 @@ var (
 func Get(name string, c echo.Context) (*sessions.Session, error) {
 	s := c.Get(key)
 	if s == nil {
-		return nil, fmt.Errorf("%q session not found", name)
+		return nil, fmt.Errorf("%q session store not found", key)
 	}
 	store := s.(sessions.Store)
 	return store.Get(c.Request(), name)

session/session_test.go

@@ -1,6 +1,7 @@
 package session
 
 import (
+	"fmt"
 	"net/http"
 	"net/http/httptest"
 	"testing"
@@ -50,4 +51,15 @@ func TestMiddleware(t *testing.T) {
 	h = mw(handler)
 	assert.NoError(t, h(c))
 	assert.Contains(t, rec.Header().Get(echo.HeaderSetCookie), "labstack.com")
+
+}
+
+func TestGetSessionMissingStore(t *testing.T) {
+	e := echo.New()
+	req := httptest.NewRequest(echo.GET, "/", nil)
+	rec := httptest.NewRecorder()
+	c := e.NewContext(req, rec)
+	_, err := Get("test", c)
+
+	assert.EqualError(t, err, fmt.Sprintf("%q session store not found", key))
 }