Add missing default annotations

Author: simolus3

crates/core/build.rs

@@ -1,7 +1,10 @@
 use std::process::Command;
 fn main() {
     // note: add error checking yourself.
-    let output = Command::new("git").args(&["rev-parse", "HEAD"]).output().unwrap();
+    let output = Command::new("git")
+        .args(&["rev-parse", "HEAD"])
+        .output()
+        .unwrap();
     let git_hash = String::from_utf8(output.stdout).unwrap();
     println!("cargo:rustc-env=GIT_HASH={}", git_hash);
 }

crates/core/src/ext.rs

@@ -1,4 +1,4 @@
-use sqlite_nostd::{Connection, Destructor, ManagedStmt, ResultCode, sqlite3};
+use sqlite_nostd::{sqlite3, Connection, Destructor, ManagedStmt, ResultCode};
 
 pub trait SafeManagedStmt {
     fn exec(&self) -> Result<(), ResultCode>;
@@ -23,7 +23,6 @@ impl SafeManagedStmt for ManagedStmt {
     }
 }
 
-
 pub trait ExtendedDatabase {
     fn exec_text(&self, sql: &str, param: &str) -> Result<(), ResultCode>;
 }

crates/core/src/sync/line.rs

@@ -54,6 +54,7 @@ pub struct CheckpointDiff<'a> {
     pub updated_buckets: Vec<BucketChecksum<'a>>,
     #[serde(borrow)]
     pub removed_buckets: Vec<&'a str>,
+    #[serde(default)]
     #[serde(deserialize_with = "deserialize_optional_string_to_i64")]
     pub write_checkpoint: Option<i64>,
 }
@@ -75,11 +76,13 @@ pub struct CheckpointPartiallyComplete {
 pub struct BucketChecksum<'a> {
     pub bucket: &'a str,
     pub checksum: Checksum,
+    #[serde(default)]
     pub priority: Option<BucketPriority>,
+    #[serde(default)]
     pub count: Option<i64>,
-//    #[serde(default)]
-//    #[serde(deserialize_with = "deserialize_optional_string_to_i64")]
-//    pub last_op_id: Option<i64>,
+    //    #[serde(default)]
+    //    #[serde(deserialize_with = "deserialize_optional_string_to_i64")]
+    //    pub last_op_id: Option<i64>,
 }
 
 #[derive(Deserialize, Debug)]
@@ -364,15 +367,23 @@ mod tests {
         assert_eq!(diff.removed_buckets.len(), 0);
     }
 
+    #[test]
+    fn parse_checkpoint_diff_no_write_checkpoint() {
+        let SyncLine::CheckpointDiff(diff) = deserialize(
+            r#"{"checkpoint_diff":{"last_op_id":"12","updated_buckets":[{"bucket":"a","count":12,"checksum":0,"priority":3}],"removed_buckets":[]}}"#,
+        ) else {
+            panic!("Expected checkpoint diff")
+        };
+    }
+
     #[test]
     fn parse_checkpoint_complete() {
         assert_matches!(
             deserialize(r#"{"checkpoint_complete": {"last_op_id": "10"}}"#),
-            SyncLine::CheckpointComplete(CheckpointComplete { 
-                //last_op_id: 10 
-                })
+            SyncLine::CheckpointComplete(CheckpointComplete {
+                // last_op_id: 10
+            })
         );
-
     }
 
     #[test]