test(#176): reveal that last smith certification does not postpone expiry

pallets/smith-members/src/tests.rs

@@ -580,21 +580,24 @@ fn certifying_an_online_smith() {
             RuntimeOrigin::signed(1),
             5
         ));
+        Pallet::<Runtime>::on_new_session(3);
         assert_ok!(Pallet::<Runtime>::certify_smith(
             RuntimeOrigin::signed(2),
             5
         ));
         // Smith can expire
+        // TODO: test failing because expires_on is not postponed to session 8
         assert_eq!(
             Smiths::<Runtime>::get(5),
             Some(SmithMeta {
                 status: Smith,
-                expires_on: Some(7),
+                expires_on: Some(8),
                 issued_certs: vec![],
                 received_certs: vec![1, 2]
             })
         );
         assert_eq!(ExpiresOn::<Runtime>::get(7), Some(vec![5]));
+        assert_eq!(ExpiresOn::<Runtime>::get(8), Some(vec![5]));
 
         Pallet::<Runtime>::on_smith_goes_online(5);
         // After going online, the expiration disappears
@@ -609,6 +612,7 @@ fn certifying_an_online_smith() {
         );
         // ExpiresOn is not unscheduled, but as expires_on has switched to None it's not a problem
         assert_eq!(ExpiresOn::<Runtime>::get(7), Some(vec![5]));
+        assert_eq!(ExpiresOn::<Runtime>::get(8), Some(vec![5]));
 
         // We can receive certification without postponing the expiration (because we are online)
         assert_ok!(Pallet::<Runtime>::certify_smith(