X-Git-Url: https://ruderich.org/simon/gitweb/?a=blobdiff_plain;f=main_test.go;h=145abe91dfe5b8a0c53989e90133856feeeea528;hb=999fa3508eab3bba561202b91c520aaa5893bf11;hp=31c9e50064f21276776f5e344edc62da53f9aad0;hpb=a8a8f9daebe164d067d004458679dd7e7f0dc087;p=nsscash%2Fnsscash.git

diff --git a/main_test.go b/main_test.go
index 31c9e50..145abe9 100644
--- a/main_test.go
+++ b/main_test.go
@@ -156,8 +156,8 @@ func mustMakeOld(t *testing.T, paths ...string) {
 	}
 }
 
-// mustMakeOld verifies that all paths have a modification time in the past,
-// as set by mustMakeOld().
+// mustBeOld verifies that all paths have a modification time in the past, as
+// set by mustMakeOld.
 func mustBeOld(t *testing.T, paths ...string) {
 	for _, p := range paths {
 		i, err := os.Stat(p)
@@ -198,6 +198,7 @@ func TestMainFetch(t *testing.T) {
 		// Perform most tests with passwd for simplicity
 		fetchPasswdCacheFileDoesNotExist,
 		fetchPasswd404,
+		fetchPasswdUnexpected304,
 		fetchPasswdEmpty,
 		fetchPasswdInvalid,
 		fetchPasswdLimits,
@@ -326,7 +327,25 @@ func fetchPasswd404(a args) {
 		"status code 404")
 
 	mustNotExist(t, statePath, plainPath, groupPath)
-	mustBeOld(a.t, passwdPath)
+	mustBeOld(t, passwdPath)
+}
+
+func fetchPasswdUnexpected304(a args) {
+	t := a.t
+	mustWritePasswdConfig(t, a.url)
+	mustCreate(t, passwdPath)
+
+	*a.handler = func(w http.ResponseWriter, r *http.Request) {
+		// 304
+		w.WriteHeader(http.StatusNotModified)
+	}
+
+	err := mainFetch(configPath)
+	mustBeErrorWithSubstring(t, err,
+		"status code 304 but did not send If-Modified-Since")
+
+	mustNotExist(t, statePath, plainPath, groupPath)
+	mustBeOld(t, passwdPath)
 }
 
 func fetchPasswdEmpty(a args) {
@@ -696,6 +715,7 @@ func fetchStateCannotRead(a args) {
 		statePath+": permission denied")
 
 	mustNotExist(t, passwdPath, plainPath, groupPath)
+	mustBeOld(t, statePath)
 }
 
 func fetchStateInvalid(a args) {