Unit tests for models needed to be updated
Opened this issue · 14 comments
Recommended to break down in multiple PR
@garg-anuj Would like to work on this.
@sauravvishal8797 Go on
@garg-anuj
Is this assigned to @sauravvishal8797
or can I work on this?
@free4murad Just assigned this to @sauravvishal8797 to avoid confusion
@garg-anuj Can you elaborate a little more in the description what do you expect here?
@danielpassos Yeah sure
Models under org.hawkular.client.android.backend.model
need to have updated constructors annotated with @VisibleForTesting and unit tests.
Some models don't have tests at present so new tests needed to be created for them
@danielpassos @garg-anuj I think this issue requires significant amount of work. Can I start writing tests for some modules.
@danielpassos @garg-anuj
Actually I haven't completed any GSoC issue yet.
So if @sauravvishal8797 will not be working on it please assign it to me.
@pg301 @free4murad Still working on this one.
@sauravvishal8797 Cool! Please let me know if you need some help. I have updated some constructors in a3f0b95 so if you want you can skip that part.
@sauravvishal8797 The issue description suggests that we break down the issue in multiple PRs. I think we can split some work. Please let me know your progress and we can decide on the remaining modules.
@free4murad GSoC issue only means that given issue will be probably easy one to tackle and nothing more. So find issues and register them. Thats it 😉
@sauravvishal8797 @garg-anuj The recently opened pull requests 9e85a15 only fixes 'updating visibleForTesting annotation'. There are a number of models which do not have corresponding tests. I would like to write tests for the remaining models and will send a PR.