Extend unit testing for elo-ranking

src/mock.rs

@@ -147,6 +147,16 @@ pub fn new_test_ext() -> sp_io::TestExternalities {
                 frame_benchmarking::account("Bob", 0, 1),
                 asset_min_balance * 100,
             ),
+            (
+                asset_id,
+                frame_benchmarking::account("Charlie", 0, 2),
+                asset_min_balance * 100,
+            ),
+            (
+                asset_id,
+                frame_benchmarking::account("Dave", 0, 3),
+                asset_min_balance * 100,
+            ),
         ],
     }
     .assimilate_storage(&mut storage)
@@ -156,6 +166,8 @@ pub fn new_test_ext() -> sp_io::TestExternalities {
         elo: vec![
             (frame_benchmarking::account("Alice", 0, 0), 2000),
             (frame_benchmarking::account("Bob", 0, 1), 2400),
+            (frame_benchmarking::account("Charlie", 0, 2), 1000),
+            (frame_benchmarking::account("Dave", 0, 3), 1000),
         ],
     }
     .assimilate_storage(&mut storage)

src/tests.rs

@@ -534,7 +534,51 @@ fn check_elo_stronger_looses() {
 }
 
 #[test]
-fn check_elo_player_aborts() {
+fn check_elo_player_draw() {
+    new_test_ext().execute_with(|| {
+        System::set_block_number(1);
+        let charlie: u64 = account("Charlie", 0, 2);
+        let dave:u64  = account("Dave", 0, 3);
+
+        let bet_asset_id = AssetId::get();
+        let bet_amount = AssetMinBalance::get() * 5;
+
+        assert_ok!(Chess::create_match(
+            RuntimeOrigin::signed(charlie),
+            dave,
+            MatchStyle::Bullet,
+            bet_asset_id,
+            bet_amount
+        ));
+
+        let match_id = Chess::chess_match_id_from_nonce(0).unwrap();
+
+        assert_ok!(Chess::join_match(RuntimeOrigin::signed(dave), match_id));
+
+        // check elo before the match finishes
+        assert_eq!(Chess::player_elo(charlie), 1000);
+        assert_eq!(Chess::player_elo(dave), 1000);
+
+        assert_ok!(Chess::force_board_state(
+            match_id,
+            "8/8/8/8/8/5K2/Q7/7k w - - 1 68".into()
+        ));
+        // this move forces draws
+        assert_ok!(Chess::make_move(
+            RuntimeOrigin::signed(charlie),
+            match_id,
+            "a2f2".into()
+        ));
+        assert_eq!(Chess::chess_matches(match_id), None);
+
+        // check the elo after the match is complete
+        assert_eq!(Chess::player_elo(charlie), 1000);
+        assert_eq!(Chess::player_elo(dave), 1000);
+    });
+}
+
+#[test]
+fn check_elo_stronger_abandon() {
     new_test_ext().execute_with(|| {
         System::set_block_number(1);
 
@@ -560,24 +604,37 @@ fn check_elo_player_aborts() {
         assert_eq!(Chess::player_elo(alice), 2000);
         assert_eq!(Chess::player_elo(bob), 2400);
 
-        assert_ok!(Chess::force_board_state(
+        assert_ok!(Chess::make_move(
+            RuntimeOrigin::signed(bob),
             match_id,
-            "8/8/8/8/8/5K2/Q7/7k w - - 1 68".into()
+            "f2f3".into()
         ));
-        // this move forces draws
         assert_ok!(Chess::make_move(
-            RuntimeOrigin::signed(bob),
+            RuntimeOrigin::signed(alice),
             match_id,
-            "a2f2".into()
+            "e7e5".into()
+        ));
+
+        // advance the block number to the point where bob's time-to-move is expired
+        System::set_block_number(
+            System::block_number() + <Test as Config>::BulletPeriod::get() + 1,
+        );
+
+        // Bob abandom, alice claims victory 
+        assert_ok!(Chess::clear_abandoned_match(
+            RuntimeOrigin::signed(alice),
+            match_id
         ));
+
         assert_eq!(Chess::chess_matches(match_id), None);
 
-        // check the elo after the match is complete
-        assert_eq!(Chess::player_elo(alice), 2013);
-        assert_eq!(Chess::player_elo(bob), 2387);
+        // check the elo after match is complete
+        assert_eq!(Chess::player_elo(alice), 2029);
+        assert_eq!(Chess::player_elo(bob), 2371);
     });
 }
 
+
 const BOARD_STATE: &str = "Q7/5Q2/8/8/3k4/6P1/6BP/7K b - - 0 67";
 
 #[test]