Do not expose $this->connection()

When adding the timeout to waitForUpdate in 🐛 Add timeout to waitForTask() Active discovered that the waitForTask is sometimes directly called, which should not happen. That also means the timeout set was not always applied. Checked if there are any other instance where $this->connection is directly called, found some in tests but other than that the only issue was with waitForTask. Again, test are expected to fail as the 🐛 Add timeout to waitForTask() Active first needs to be merged to add the timeout.
Setting to needs review to not cause confusion.

Rebased and merged, thanks.

Automatically closed - issue fixed for 2 weeks with no activity.