update per requested changes

Author: marionbarker

Loop/Managers/LoopDataManager.swift

@@ -1157,7 +1157,7 @@ extension LoopDataManager {
             throw LoopError.glucoseTooOld(date: glucose.startDate)
         }
 
-        guard lastGlucoseDate.timeIntervalSince(now()) <= LoopCoreConstants.inputDataRecencyInterval else {
+        guard lastGlucoseDate.timeIntervalSince(now()) <= LoopCoreConstants.futureGlucoseDataInterval else {
             throw LoopError.glucoseInFuture(date: lastGlucoseDate)
         }
 

Loop/Models/LoopError.swift

@@ -192,7 +192,7 @@ extension LoopError: LocalizedError {
             return String(format: NSLocalizedString("Glucose data is %1$@ old", comment: "The error message when glucose data is too old to be used. (1: glucose data age in minutes)"), minutes)
         case .glucoseInFuture(let date):
             let minutes = formatter.string(from: -date.timeIntervalSinceNow) ?? ""
-            return String(format: NSLocalizedString("Glucose data is %1$@ in the future", comment: "The error message when glucose data is in the future to be used. (1: glucose data time in future in minutes)"), minutes)
+            return String(format: NSLocalizedString("Invalid glucose reading with a timestamp that is %1$@ in the future", comment: "The error message when glucose data is in the future. (1: glucose data time in future in minutes)"), minutes)
         case .pumpDataTooOld(let date):
             let minutes = formatter.string(from: -date.timeIntervalSinceNow) ?? ""
             return String(format: NSLocalizedString("Pump data is %1$@ old", comment: "The error message when pump data is too old to be used. (1: pump data age in minutes)"), minutes)

Loop/View Models/BolusEntryViewModel.swift

@@ -674,7 +674,7 @@ final class BolusEntryViewModel: ObservableObject {
             switch error {
             case LoopError.missingDataError(.glucose), LoopError.glucoseTooOld:
                 notice = .staleGlucoseData
-            case LoopError.missingDataError(.glucose), LoopError.glucoseInFuture:
+            case LoopError.glucoseInFuture:
                 notice = .futureGlucoseData
             case LoopError.pumpDataTooOld:
                 notice = .stalePumpData

Loop/Views/BolusEntryView.swift

@@ -336,8 +336,8 @@ struct BolusEntryView: View {
             )
         case .futureGlucoseData:
             return WarningView(
-                title: Text("Glucose in Future", comment: "Title for bolus screen notice when glucose data is in the future"),
-                caption: Text("Check for data in the future on phone.", comment: "Caption for bolus screen notice when glucose data is in the future")
+                title: Text("Invalid Future Glucose", comment: "Title for bolus screen notice when glucose data is in the future"),
+                caption: Text("Check for device time and/or remove any invalid data from Apple Health.", comment: "Caption for bolus screen notice when glucose data is in the future")
             )
         case .stalePumpData:
             return WarningView(

LoopCore/LoopCoreConstants.swift

@@ -13,6 +13,9 @@ public enum LoopCoreConstants {
     /// The amount of time since a given date that input data should be considered valid
     public static let inputDataRecencyInterval = TimeInterval(minutes: 15)
 
+    /// The amount of time in the future a glucose value should be considered valid
+    public static let futureGlucoseDataInterval = TimeInterval(minutes: 5)
+
     public static let defaultCarbAbsorptionTimes: CarbStore.DefaultAbsorptionTimes = (fast: .minutes(30), medium: .hours(3), slow: .hours(5))
 
     /// How much historical glucose to include in a dosing decision