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 and 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 4, 2023
1 parent b80eb4a commit 0880977
Show file tree
Hide file tree
Showing 3 changed files with 689 additions and 18 deletions.
Loading

0 comments on commit 0880977

Please sign in to comment.