diff --git a/pallets/smith-members/src/lib.rs b/pallets/smith-members/src/lib.rs
index 691e68e99b3aaca90a7ad26390c31e68b9719cc8..cce2dbaeadb5460bc110d1d65234a9633d787900 100644
--- a/pallets/smith-members/src/lib.rs
+++ b/pallets/smith-members/src/lib.rs
@@ -461,7 +461,7 @@ impl<T: Config> Pallet<T> {
                     let new_expires_on =
                         CurrentSession::<T>::get() + T::SmithInactivityMaxDuration::get();
                     smith_meta.expires_on = Some(new_expires_on);
-                    // ExpiresOn::<T>::append(new_expires_on, receiver);
+                    ExpiresOn::<T>::append(new_expires_on, receiver);
                 }
 
                 // - if the status is smith but wasn't, notify that smith gained membership
diff --git a/pallets/smith-members/src/tests.rs b/pallets/smith-members/src/tests.rs
index 60a8786a10eeed18fad4e2af05f85b380da419c3..a200d0551e9685aab6b283bb2d291030a4905c6f 100644
--- a/pallets/smith-members/src/tests.rs
+++ b/pallets/smith-members/src/tests.rs
@@ -594,7 +594,6 @@ fn certifying_an_online_smith() {
                 received_certs: vec![1, 2]
             })
         );
-        // TODO: test is failing here because ExpiresOn is not correctly updated
         assert_eq!(ExpiresOn::<Runtime>::get(7), Some(vec![5]));
 
         Pallet::<Runtime>::on_smith_goes_online(5);