test: added more test data

IllyaYalovyy · IllyaYalovyy · commit bae0cff82431 · 2025-06-24T12:42:04.000-07:00
diff --git a/src/main.rs b/src/main.rs
@@ -25,10 +25,10 @@ async fn filter_actions(event: Value, _: Context) -> Result<Value, Error> {
 
     let input: Vec<Action> = serde_json::from_value(event)?;
 
-    let mut selected: HashMap<&str, Action> = HashMap::new();
+    let mut selected: HashMap<String, Action> = HashMap::new();
 
     for action in input {
-        let days_since_last = (now - action.last_action_time).num_days();
+        let days_since_last = (now - action.last_action_time).to_std().unwrap().as_secs() / 86400;
         if days_since_last < 7 {
             continue;
         }
@@ -37,9 +37,7 @@ async fn filter_actions(event: Value, _: Context) -> Result<Value, Error> {
             continue;
         }
 
-        selected
-            .entry(&action.entity_id)
-            .or_insert(action);
+        selected.entry(action.entity_id.clone()).or_insert(action);
     }
 
     let mut actions: Vec<Action> = selected.into_iter().map(|(_, v)| v).collect();
diff --git a/testdata/01_sample-input.json b/testdata/01_sample-input.json
@@ -3,24 +3,24 @@
     "entity_id": "entity_1",
     "last_action_time": "2025-06-20T00:00:00Z",
     "next_action_time": "2025-07-10T00:00:00Z",
-    "priority": "high"
+    "priority": "urgent"
   },
   {
     "entity_id": "entity_1",
     "last_action_time": "2025-06-01T00:00:00Z",
     "next_action_time": "2025-07-01T00:00:00Z",
-    "priority": "low"
+    "priority": "normal"
   },
   {
     "entity_id": "entity_2",
     "last_action_time": "2025-03-01T00:00:00Z",
     "next_action_time": "2026-01-01T00:00:00Z",
-    "priority": "high"
+    "priority": "urgent"
   },
   {
     "entity_id": "entity_3",
     "last_action_time": "2025-05-01T00:00:00Z",
     "next_action_time": "2025-07-10T00:00:00Z",
-    "priority": "low"
+    "priority": "normal"
   }
 ]
diff --git a/testdata/02_priority-input.json b/testdata/02_priority-input.json
@@ -0,0 +1,32 @@
+[
+  {
+    "entity_id": "entity_1",
+    "last_action_time": "2025-06-20T00:00:00Z",
+    "next_action_time": "2025-07-10T00:00:00Z",
+    "priority": "urgent"
+  },
+  {
+    "entity_id": "entity_2",
+    "last_action_time": "2025-06-01T00:00:00Z",
+    "next_action_time": "2025-07-01T00:00:00Z",
+    "priority": "normal"
+  },
+  {
+    "entity_id": "entity_3",
+    "last_action_time": "2025-03-01T00:00:00Z",
+    "next_action_time": "2026-01-01T00:00:00Z",
+    "priority": "urgent"
+  },
+  {
+    "entity_id": "entity_4",
+    "last_action_time": "2025-05-01T00:00:00Z",
+    "next_action_time": "2025-07-10T00:00:00Z",
+    "priority": "normal"
+  },
+  {
+    "entity_id": "entity_5",
+    "last_action_time": "2025-05-01T00:00:00Z",
+    "next_action_time": "2025-07-10T00:00:00Z",
+    "priority": "urgent"
+  }
+]
diff --git a/testdata/03_bad-input.json b/testdata/03_bad-input.json
@@ -0,0 +1,26 @@
+[
+  {
+    "entity_id": "entity_1",
+    "last_action_time": "2025-08-20T00:00:00Z",
+    "next_action_time": "2025-07-10T00:00:00Z",
+    "priority": "urgent"
+  },
+  {
+    "entity_id": "entity_2",
+    "last_action_time": "2025-03-01T00:00:00Z",
+    "next_action_time": "2026-01-01T00:00:00Z",
+    "priority": "urgent"
+  },
+  {
+    "entity_id": "entity_3",
+    "last_action_time": "2025-05-01T00:00:00Z",
+    "next_action_time": "2025-07-10T00:00:00Z",
+    "priority": "normal"
+  },
+  {
+    "entity_id": "entity_3",
+    "last_action_time": "2025-05-01T00:00:00Z",
+    "next_action_time": "2025-07-10T00:00:00Z",
+    "priority": "unknown"
+  }
+]
diff --git a/tests/filter_tests.rs b/tests/filter_tests.rs
@@ -1,6 +1 @@
-use lambda_action_filter::main as lambda_main; // Adjust if crate name changes
-// Placeholder for integration tests; candidates can extend
-#[test]
-fn dummy() {
-    assert_eq!(2 + 2, 4);
-}
+

Original file line number	Diff line number	Diff line change
`@@ -25,10 +25,10 @@ async fn filter_actions(event: Value, _: Context) -> Result<Value, Error> {`
`25`	`25`
`26`	`26`	`let input: Vec<Action> = serde_json::from_value(event)?;`
`27`	`27`
`28`		`- let mut selected: HashMap<&str, Action> = HashMap::new();`
	`28`	`+ let mut selected: HashMap<String, Action> = HashMap::new();`
`29`	`29`
`30`	`30`	`for action in input {`
`31`		`- let days_since_last = (now - action.last_action_time).num_days();`
	`31`	`+ let days_since_last = (now - action.last_action_time).to_std().unwrap().as_secs() / 86400;`
`32`	`32`	`if days_since_last < 7 {`
`33`	`33`	`continue;`
`34`	`34`	`}`
`@@ -37,9 +37,7 @@ async fn filter_actions(event: Value, _: Context) -> Result<Value, Error> {`
`37`	`37`	`continue;`
`38`	`38`	`}`
`39`	`39`
`40`		`- selected`
`41`		`- .entry(&action.entity_id)`
`42`		`- .or_insert(action);`
	`40`	`+ selected.entry(action.entity_id.clone()).or_insert(action);`
`43`	`41`	`}`
`44`	`42`
`45`	`43`	`let mut actions: Vec<Action> = selected.into_iter().map(\|(_, v)\| v).collect();`
Original file line number	Diff line number	Diff line change
`@@ -3,24 +3,24 @@`
`3`	`3`	`"entity_id": "entity_1",`
`4`	`4`	`"last_action_time": "2025-06-20T00:00:00Z",`
`5`	`5`	`"next_action_time": "2025-07-10T00:00:00Z",`
`6`		`- "priority": "high"`
	`6`	`+ "priority": "urgent"`
`7`	`7`	`},`
`8`	`8`	`{`
`9`	`9`	`"entity_id": "entity_1",`
`10`	`10`	`"last_action_time": "2025-06-01T00:00:00Z",`
`11`	`11`	`"next_action_time": "2025-07-01T00:00:00Z",`
`12`		`- "priority": "low"`
	`12`	`+ "priority": "normal"`
`13`	`13`	`},`
`14`	`14`	`{`
`15`	`15`	`"entity_id": "entity_2",`
`16`	`16`	`"last_action_time": "2025-03-01T00:00:00Z",`
`17`	`17`	`"next_action_time": "2026-01-01T00:00:00Z",`
`18`		`- "priority": "high"`
	`18`	`+ "priority": "urgent"`
`19`	`19`	`},`
`20`	`20`	`{`
`21`	`21`	`"entity_id": "entity_3",`
`22`	`22`	`"last_action_time": "2025-05-01T00:00:00Z",`
`23`	`23`	`"next_action_time": "2025-07-10T00:00:00Z",`
`24`		`- "priority": "low"`
	`24`	`+ "priority": "normal"`
`25`	`25`	`}`
`26`	`26`	`]`