Also I noticed in some other tests we were inconsistent about `{}` vs `None` as the argument to `set_config_data`