fix file backend pushinfo

storage/file/file_test.go

@@ -0,0 +1,18 @@
+package file
+
+import (
+	"context"
+	"testing"
+
+	"github.com/micromdm/nanomdm/storage/test"
+)
+
+func TestFileStorage(t *testing.T) {
+	s, err := New(t.TempDir())
+	if err != nil {
+		t.Fatal(err)
+	}
+
+	test.TestQueue(t, "EA4E19F1-7F8B-493D-BEAB-264B33BCF4E6", s)
+	test.TestRetrievePushInfo(t, context.Background(), s)
+}

storage/file/push.go

@@ -3,6 +3,7 @@ package file
 import (
 	"context"
 	"errors"
+	"os"
 
 	"github.com/micromdm/nanomdm/mdm"
 )
@@ -13,7 +14,11 @@ func (s *FileStorage) RetrievePushInfo(_ context.Context, ids []string) (map[str
 	for _, id := range ids {
 		e := s.newEnrollment(id)
 		tokenUpdate, err := e.readFile(TokenUpdateFilename)
-		if err != nil {
+		if errors.Is(err, os.ErrNotExist) {
+			// TokenUpdate file missing could be a non-existent or
+			// incomplete enrollment which should not trigger an error.
+			continue
+		} else if err != nil {
 			return nil, err
 		}
 		msg, err := mdm.DecodeCheckin(tokenUpdate)

storage/storage.go

@@ -40,9 +40,15 @@ type ServiceStore interface {
 	BootstrapTokenStore
 }
 
-// PushStore stores and retrieves APNs push-related data.
+// PushStore retrieves APNs push-related data.
 type PushStore interface {
-	RetrievePushInfo(context.Context, []string) (map[string]*mdm.Push, error)
+	// RetrievePushInfo retrieves push data for the given ids.
+	//
+	// If an ID does not exist or is not enrolled properly then
+	// implementations should silently skip returning any push data for
+	// them. It is up to the caller to discern any missing IDs from the
+	// returned map.
+	RetrievePushInfo(ctx context.Context, ids []string) (map[string]*mdm.Push, error)
 }
 
 // PushCertStore stores and retrieves APNs push certificates.