i hate borrow

secert-hitler/src/controller/gamemaster/gamemaster.rs

@@ -33,25 +33,27 @@ impl GameMaster {
       self.run_game()
    }
 
-   fn run_lobby(&mut self) -> Result<usize, &str> {
-      self.run_lobby_loop()?;
-      self.lobby.ready()
-   }
-
-   fn run_lobby_loop(&mut self) -> Result<usize, &str> {
-      while !self.lobby.locked {
-         self.run_lobby_scrape()?;
+   fn run_lobby(&mut self) -> Result<usize, String> {
+      let mut r = self.run_lobby_scrape();
+      while r.unwrap_or(0) == 0 {
+      //while r.is_ok() && r.unwrap() == 0 {
          thread::sleep(time::Duration::new(1, 0));
+         r = self.run_lobby_scrape();
       }
-      Ok(0)
+         return Ok(r.unwrap().clone());
+      /*
+      if r.is_ok() {
+         return Ok(r.unwrap());
+      }
+      let e = r.err();
+      Err(e.unwrap())
+      */
    }
 
-   fn run_lobby_scrape(&mut self) -> Result<usize, &str> {
-      /*
-      let rollback = self.room.since();
+   fn run_lobby_scrape(&mut self) -> Result<usize, String> {
       let events = self.room.sync();
       for e in &events {
-         let ge = GameEvent::new(e.body.clone());
+         let ge = GameEvent::new(e.clone());
          if ge.mode() == GameEventType::GameStart {
             self.room.rollback(e.since.clone());
             self.lobby.lock();
@@ -59,7 +61,6 @@ impl GameMaster {
          }
          self.lobby.eat(e.clone());
       }
-      */
       Ok(0)
    }
 
@@ -131,11 +132,15 @@ mod tests {
    #[test]
    fn run_lobby() {
       init();
+         println!("run_lobby 1");
       let mut mrs = MockRooms::new();
       let r1 = mrs.create();
+         println!("run_lobby 2");
       let room_id = r1.room_id();
       let mut gm = GameMaster::new(r1);
+         println!("run_lobby 3");
       for i in 0..4 {
+            println!("run_lobby attempt stop #{}", i);
          let mut r2 = mrs.join(room_id.clone()).unwrap();
          r2.send(format!(r#"{{
             "msgtype": "m.text",
@@ -160,7 +165,6 @@ mod tests {
       init();
       let mut mrs = MockRooms::new();
       let r1 = mrs.create();
-      let room_id = r1.room_id();
       let mut gm = GameMaster::new(r1);
       assert!(gm.run_game_setup().is_ok());
    }