Use actual timeouts in tests for context.Context objects

XMLWordPrintableJSON

    • Type: Improvement
    • Resolution: Won't Do
    • Priority: Major - P3
    • None
    • Affects Version/s: 0.0.2
    • Component/s: Testing
    • None
    • None
    • None
    • None
    • None
    • None
    • None
    • None

      Currently many tests use a context.Background() during their execution. This can cause problems if there is a bug in implementation because it can be difficult to find where a deadlock is occurring. We should ensure that all things that take a context.Context are given a context with a reasonable timeout. Seeing as the maximum time for running tests is 60 seconds in most cases, this timeout should be significantly less than that.

            Assignee:
            Unassigned
            Reporter:
            Kristofer Brandow (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: