Insufficient tests

Open esafak opened this issue 9 months ago • 2 comments

A library that hopes to be commercially adopted needs tests.

Mar 13 '25 02:03 esafak

A library that hopes to be commercially adopted needs tests.

Thank you! This is a priority and we will be adding better code coverage in the coming days.

Mar 13 '25 11:03 saqadri

@saqadri So start with unittest coverage of the main classes ? pytest and pytest-asyncio ? then integration test and mocked llm reponses. add a workflow to run tests etc. Starting with the critical paths.

Mar 18 '25 07:03 corroleaus

@esafak following up here to close this task. We're now at > 80% test coverage, with many new updates incoming. Hope you and others can have additional confidence in relying on this library. Thank you for raising this issue. And thanks @StreetLamb for driving this.

May 28 '25 20:05 saqadri