Merge pull request #281 from redbadger/must_use

must_use on app.update()

examples/bridge_echo/shared/src/app.rs

@@ -76,9 +76,9 @@ mod test {
         let app = AppTester::<App, _>::default();
         let mut model = Model::default();
 
-        app.update(Event::Tick, &mut model);
-        app.update(Event::Tick, &mut model);
-        app.update(Event::Tick, &mut model);
+        let _ = app.update(Event::Tick, &mut model);
+        let _ = app.update(Event::Tick, &mut model);
+        let _ = app.update(Event::Tick, &mut model);
 
         let actual_view = app.view(&model);
         let expected_view = ViewModel { count: 3 };
@@ -91,14 +91,14 @@ mod test {
         let app = AppTester::<App, _>::default();
         let mut model = Model::default();
 
-        app.update(Event::Tick, &mut model);
-        app.update(Event::Tick, &mut model);
-        app.update(Event::Tick, &mut model);
-        app.update(Event::NewPeriod, &mut model);
-        app.update(Event::Tick, &mut model);
-        app.update(Event::Tick, &mut model);
-        app.update(Event::NewPeriod, &mut model);
-        app.update(Event::Tick, &mut model);
+        let _ = app.update(Event::Tick, &mut model);
+        let _ = app.update(Event::Tick, &mut model);
+        let _ = app.update(Event::Tick, &mut model);
+        let _ = app.update(Event::NewPeriod, &mut model);
+        let _ = app.update(Event::Tick, &mut model);
+        let _ = app.update(Event::Tick, &mut model);
+        let _ = app.update(Event::NewPeriod, &mut model);
+        let _ = app.update(Event::Tick, &mut model);
 
         let expected = Model {
             log: vec![3, 2],

examples/cat_facts/shared/src/app.rs

@@ -271,7 +271,7 @@ mod tests {
         assert_eq!(update.events, vec![Event::SetFact(Ok(expected_response))]);
 
         for event in update.events {
-            app.update(event, &mut model);
+            let _ = app.update(event, &mut model);
         }
 
         assert_let!(Effect::Http(request), effects.next().unwrap());
@@ -288,7 +288,7 @@ mod tests {
             .resolve(request, response)
             .expect("should resolve successfully");
         for event in update.events {
-            app.update(event, &mut model);
+            let _ = app.update(event, &mut model);
         }
 
         assert_eq!(model.cat_fact, Some(a_fact));

examples/cat_facts/shared/src/app/platform.rs

@@ -67,7 +67,7 @@ mod tests {
             .resolve(request, response)
             .expect("should resolve successfully");
         for event in update.events {
-            app.update(event, &mut model);
+            let _ = app.update(event, &mut model);
         }
 
         assert_eq!(model.platform, "platform");

examples/counter/shared/src/app.rs

@@ -258,7 +258,7 @@ mod tests {
             .expect("Update to succeed");
 
         // send the generated (internal) `Set` event back into the app
-        app.update(update.events[0].clone(), &mut model);
+        let _ = app.update(update.events[0].clone(), &mut model);
 
         // check that the model has been updated correctly
         insta::assert_yaml_snapshot!(model, @r###"
@@ -319,7 +319,7 @@ mod tests {
         // run the event loop in order to send the (internal) `Set` event
         // back into the app
         for event in update.events {
-            app.update(event, &mut model);
+            let _ = app.update(event, &mut model);
         }
 
         // check that the model has been updated correctly