Ensure dbs tests don't leave hanging temporary files (#796)

Summary:
Pull Request resolved: #796

Extra read only databases were leaving temporary resources hanging and not cleaning up. Fixed to ensure they are manually handled.

Reviewed By: tymmsc

Differential Revision: D74211454

fbshipit-source-id: 01ce2c77b5b1443d86a0deba5c4ecae17a680ea2

Author: JasonKChow

tests/test_db.py

@@ -648,6 +648,8 @@ def test_get_dataframe_extra_data(self):
         np.testing.assert_equal(df.iloc[0]["trial_number"], np.nan)
         self.assertEqual(df.iloc[1]["trial_number"], 2)
 
+        extra_info_data.cleanup()
+
     def test_combine_dbs_list(self):
         """Test the combine_dbs function with a list of database paths."""
         # Get paths to existing test databases
@@ -693,6 +695,8 @@ def test_combine_dbs_list(self):
                 "single_stimuli.db" in origin_db or "extra_info.db" in origin_db
             )
 
+        combined_db.cleanup()
+
 
 if __name__ == "__main__":
     unittest.main()