In https://github.com/electron/electron/commit/7613afe6d70eadd7446db921918dca086cd324c0 I had to disable the check that validates that the constructor is called on the right thread, because the check fails when the tests run.

Let's make sure the check doesn't fail and re-enable it. Or refactor so that the thread affinity is not required and remove the check.
atom/browser/api/atom_api_app.cc
}
  
  if (!jump_list.Begin())
    return JumpListResult::GENERIC_ERROR;
  JumpListResult result = jump_list.AppendCategories(categories);
  
  
  if (!jump_list.Commit()) {
    LOG(ERROR) << "Failed to commit changes to custom Jump List.";
    
    
    
    if (result == JumpListResult::SUCCESS)
      result = JumpListResult::GENERIC_ERROR;
  }
  return result;
}
#endif  
void App::GetFileIcon(const base::FilePath& path,
                      mate::Arguments* args) {
  mate::Dictionary options;
  IconLoader::IconSize icon_size;
  FileIconCallback callback;
  v8::Locker locker(isolate());
  v8::HandleScope handle_scope(isolate());
  base::FilePath normalized_path = path.NormalizePathSeparators();
  if (!args->GetNext(&options)) {
    icon_size = IconLoader::IconSize::NORMAL;
  } else {
    std::string icon_size_string;