- 🇺🇸United States dcam
I don't understand this issue. I've been doing git and issue archaeology for the last hour to try and get the point because it doesn't make sense. At first I couldn't understand what @quietone was talking about because those aren't the class names of Aggregator's upgrade tests and never were. Also, neither test has a @todo and never did. But I understood that the migration tests were copied from migrate_drupal_ui so that they would continue working for Aggregator even after it was removed from Core. So I kept digging. While the copied tests were in development they did have the copied class names along with the @todo. All that was changed by @quietone in the MR with commit https://git.drupalcode.org/project/drupal/-/merge_requests/1849/diffs?co.... But that was nine days before this issue was created. So what's up with this?
I think we can close this. It doesn't seem applicable. Also in 📌 Add content in tests only when needed Needs work @benjifisher asked at least twice if this issue could be closed as outdated without any response that I saw. If I'm wrong, then please feel free to re-open it, but please also explain exactly what needs to be done and where because I don't understand.