Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use global executionContext #30

Closed
wants to merge 1 commit into from
Closed

Conversation

bcarter97
Copy link
Member

Description

Related Issues

  • Connects to #

Definition of Done:

The Below tasks should be completed before marking a PR as ready for review.

  • Added / Removed / Updated relevant tests
  • Scaladoc updated

@bcarter97 bcarter97 force-pushed the global-execution-context branch from 1cfd462 to 07e7271 Compare October 3, 2023 11:38
@bcarter97 bcarter97 force-pushed the global-execution-context branch from 07e7271 to cbe8d05 Compare October 23, 2023 08:51
import scala.concurrent.duration.*

trait AsyncIntSpec[F[_]] extends AsyncWordSpec with AsyncIOSpec with Matchers with OptionValues with Eventually {
override given patienceConfig: PatienceConfig = PatienceConfig(30.seconds, 500.millis)

override given executionContext: ExecutionContext = ExecutionContext.global

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is only needed when using the CatsResource trait i thought?

see typelevel/cats-effect-testing#300

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The last test in TopicLoaderIntSpec, loadAndRun sometimes fails and I'm trying to rule out it's to do with the resource being terminated before it can finish... but you're right, the global EC is only needed with the CatsResource.

@bcarter97 bcarter97 closed this Dec 1, 2023
@bcarter97 bcarter97 deleted the global-execution-context branch December 1, 2023 23:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants