Uploaded image for project: 'Camel'
  1. Camel
  2. CAMEL-19519

camel-file: replace Thread.sleep in tests

Attach filesAttach ScreenshotVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Novice

    Description

      We have many tests which use Thread.sleep for synchronization. This is bug prone and can introduce flakiness when running on environments with different capacities.

      Ideally we should replace these with:

      • Awaitility
      • Java's native syncronization mechanism (Latches, Phasers, Locks, etc)
      • Nothing (i.e.; in some cases the sleep can simply be removed)

       

       

      Attachments

        Issue Links

        is a clone of

        Task - A task that needs to be done. CAMEL-19514 camel-aws: replace Thread.sleep in tests

        • Minor - Minor loss of function, or other problem where easy workaround is present.
        • Open
        links to

        Web Link GitHub Pull Request #12844

        Activity
          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            var Vaishnavi R
            orpiske Otavio Rodolfo Piske
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment