From 252ab8ce7296c205d67d7c875582464d65eebbf6 Mon Sep 17 00:00:00 2001 From: Jussi Kukkonen Date: Thu, 5 Sep 2019 14:23:02 +0300 Subject: [PATCH] react: Test removing callback from nonexistent cell RemoveCallbackError::NonexistentCell was not tested yet, add new test. Fixes #465 --- exercises/react/tests/react.rs | 24 ++++++++++++++++++++++++ 1 file changed, 24 insertions(+) diff --git a/exercises/react/tests/react.rs b/exercises/react/tests/react.rs index cd26a361e..90eeef283 100644 --- a/exercises/react/tests/react.rs +++ b/exercises/react/tests/react.rs @@ -191,6 +191,30 @@ fn error_adding_callback_to_nonexistent_cell() { ); } +#[test] +#[ignore] +fn error_removing_callback_from_nonexisting_cell() { + let mut dummy_reactor = Reactor::new(); + let dummy_input = dummy_reactor.create_input(1); + let _ = dummy_reactor + .create_compute(&[CellID::Input(dummy_input)], |_| 0) + .unwrap(); + let dummy_output = dummy_reactor + .create_compute(&[CellID::Input(dummy_input)], |_| 0) + .unwrap(); + + let mut reactor = Reactor::new(); + let input = reactor.create_input(1); + let output = reactor + .create_compute(&[CellID::Input(input)], |_| 0) + .unwrap(); + let callback = reactor.add_callback(output, |_| ()).unwrap(); + assert_eq!( + reactor.remove_callback(dummy_output, callback), + Err(RemoveCallbackError::NonexistentCell) + ); +} + #[test] #[ignore] fn callbacks_only_fire_on_change() {