peerjs icon indicating copy to clipboard operation
peerjs copied to clipboard

test: add new unit test in test case reconnect

Open diaslui opened this issue 1 year ago • 1 comments

What was added? A new test named as: (reconnect => disconnect => destroy) has been added in the "reconnect" test case in the test/peer.spec.ts file

How important is this new test? This new test is important because there were already other tests that analyzed cases such as disconnect => reconnect but not the opposite, reconnect => disconnect, which could allow bugs to pass after reconnecting and disconnecting. This was solved by adding the new test that checks if everything will be ok after reconnecting and disconnecting

What was done after implementing the new test? After implementing the test, I analyzed whether it would fulfill its role by causing it to give an error on purpose, and I confirmed its effectiveness in avoiding future bugs after reconnecting and disconnecting, everything is ok.

It's OK, Tested and Approved I hope you accept the PR with the test to avoid future bugs! Thank you for your attention image

diaslui avatar May 20 '24 17:05 diaslui

the tabulation problem has been resolved image

diaslui avatar May 22 '24 11:05 diaslui