Fix #255 distance reading discarded state

• Fixed the pallet-distance issue where CurrentPoolIndex was unset during the first evaluation period, which would cause errors if the inherent_data_provider was correctly implemented.
• Fixed #255 (closed) by removing all the expect calls from the implementation. Errors will now propagate using the inherent_data_provider error mechanism without crashing the node or being shadowed by default values.

client/distance/src/lib.rs

@@ -40,17 +40,22 @@ where
     Backend: sc_client_api::Backend<B>,
     IdtyIndex: Decode + Encode + PartialEq + TypeInfo,
 {
-    let pool_index = client
+    let pool_index = if let Some(index) = client
         .storage(
             parent,
             &StorageKey(
                 frame_support::storage::storage_prefix(b"Distance", b"CurrentPoolIndex").to_vec(),
             ),
         )
-        .expect("CurrentIndex is Err")
-        .map_or(0, |raw| {
-            u32::decode(&mut &raw.0[..]).expect("cannot decode CurrentIndex")
-        });
+        .ok()
+        .flatten()
+        .and_then(|raw| u32::decode(&mut &raw.0[..]).ok())
+    {
+        index
+    } else {
+        log::error!("🧙 [distance oracle] CurrentPoolIndex cannot be decoded");
+        return Ok(sp_distance::InherentDataProvider::<IdtyIndex>::new(None));
+    };
 
     let published_results = client
         .storage(
@@ -67,22 +72,24 @@ where
                 )
                 .to_vec(),
             ),
-        )?
-        .map_or_else(Default::default, |raw| {
-            pallet_distance::EvaluationPool::<AccountId32, IdtyIndex>::decode(&mut &raw.0[..])
-                .expect("cannot decode EvaluationPool")
+        )
+        .ok()
+        .flatten()
+        .and_then(|raw| {
+            pallet_distance::EvaluationPool::<AccountId32, IdtyIndex>::decode(&mut &raw.0[..]).ok()
         });
 
     // Have we already published a result for this period?
     // The block author is guaranteed to be in the owner_keys.
-    let owner_keys = owner_keys
-        .iter()
-        .map(|&key| sp_runtime::AccountId32::new(key.0))
-        .any(|key| published_results.evaluators.contains(&key));
-
-    if owner_keys {
-        log::debug!("🧙 [distance oracle] Already published a result for this period");
-        return Ok(sp_distance::InherentDataProvider::<IdtyIndex>::new(None));
+    if let Some(results) = published_results {
+        if owner_keys
+            .iter()
+            .map(|&key| sp_runtime::AccountId32::new(key.0))
+            .any(|key| results.evaluators.contains(&key))
+        {
+            log::debug!("🧙 [distance oracle] Already published a result for this period");
+            return Ok(sp_distance::InherentDataProvider::<IdtyIndex>::new(None));
+        }
     }
 
     // Read evaluation result from file, if it exists