He-Pin opened a new pull request, #3679: URL: https://github.com/apache/pekko/pull/3679
### Motivation `SameThreadExecutionContextSpec` still exists after the implementation it originally covered was removed in #2208. It now checks the Scala standard library, including an assertion on an implementation class name. ### Modification Remove the obsolete spec and consolidate its dispatcher coverage in `ExecutionContextSpec`. Register a `scala.concurrent.ExecutionContext.parasitic` callback before completing the promise, then assert that the dispatcher uses a different thread from the caller and the callback stays on that dispatcher thread. The coverage is adapted from the existing in-repository spec; the destination retains its existing license and copyright notices. ### Result Retain focused dispatcher integration coverage without a standalone test for the removed implementation. Production code is unchanged. ### Tests - `sbt "+actor-tests / Test / testOnly org.apache.pekko.dispatch.ExecutionContextSpec"` — passed on Scala 2.13.18 and 3.3.8, 14 tests each (JDK 17). - `scalafmt --mode diff-ref=origin/main` and `scalafmt --list --mode diff-ref=origin/main` — passed. - `sbt headerCreateAll checkCodeStyle` — passed, including Java formatting and `+headerCheckAll` (JDK 17). - `git diff --check` — passed. - `validatePullRequest` — local run stopped at the contributor's request; full validation is delegated to PR CI. ### References Refs #2208 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
