Skip to content

Commit

Permalink
Write black-box tests for DefaultAbly.connect
Browse files Browse the repository at this point in the history
These document the current behaviour of this method.

I wrote them having only seen the Ably interface (which DefaultAbly
implements), and knowing the dependencies that DefaultAbly has.

I intentionally did not look at the code whilst writing these tests because I
wanted to limit myself to writing test cases based on scenarios that I could
imagine might occur, instead of ones based on logic that I know to exist.

Once I’ve written tests like this for the whole class, then I’d like to look at
its implementation and add any further test cases for logic that wasn’t clear
from the public interface.

This is part of #869.
  • Loading branch information
lawrence-forooghian committed Jan 11, 2023
1 parent 27f5862 commit dd979da
Show file tree
Hide file tree
Showing 3 changed files with 883 additions and 18 deletions.
Loading

0 comments on commit dd979da

Please sign in to comment.