[GOFLOW-31] Unit testing and coverage are missing

[giorgiozoppi]

Goflow: Unit testing and coverage

Goal

Provide a 95% unit test mocking where it's opportune with gomock. No tests are present in the current code base.

[s8sg]

@giorgiozoppi This is where I need help. I didn't had enough bandwidth to look at the project recently.
What I'm thinking is s simplify a lot of implementation and add test

Do you have any idea how we can start ?

[giorgiozoppi]

I would start from the lower level possibile. Is that the datastore? I also use this way of doing tickets for matching branches.

[s8sg]

We need to test this first
https://github.com/s8sg/goflow/tree/master/core/sdk

This has all core functionalities

[giorgiozoppi]

Ok, i branch out, please use the same schema to rename different issues. The name of teh banch will be feature/goflow-31-unit-testing.

[s8sg]

Lets use - issue-31

…

On Sat, 11 Mar 2023 at 23:01, Giorgio Zoppi ***@***.***> wrote: Ok, i branch out, please use the same schema to rename different issues. The name of teh banch will be feature/goflow-31-unit-testing. — Reply to this email directly, view it on GitHub <#31 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABYIJ2REFCFQQQTV2TPKO73W3SOUJANCNFSM6AAAAAASGAASJE> . You are receiving this because you commented.Message ID: ***@***.***>

[giorgiozoppi]

Branched out, i start working on this.