Skip to main content

You are not logged in. Your edit will be placed in a queue until it is peer reviewed.

We welcome edits that make the post easier to understand and more valuable for readers. Because community members review edits, please try to make the post substantially better than how you found it, for example, by fixing grammar or adding additional resources and hyperlinks.

13
  • 1
    And you're definitely using threads and not tasks, right? Commented Oct 28, 2019 at 0:50
  • In the current codebase yes Commented Oct 28, 2019 at 0:51
  • This test seems like its pointing to bigger problems, this shouldn't really need to be tested for, and if you do, it should probably be covered in another test Commented Oct 28, 2019 at 0:55
  • I don't know what you mean, I want to cover this in all tests. It may or may not be a test code that starts the threads, but in either case they should be cleaned up once the test is complete. If they're not this seems like something we should be aware of hence the test should fail. Commented Oct 28, 2019 at 1:02
  • It would be awesome if you could share a minimal reproducible example. Commented Oct 28, 2019 at 1:17