Discussion: This library has no tests at all?

[nerder]

After reporting a bug I was attempting to fix the issue myself but I don't really feel confident in contribute to it since there are no tests. Keeping in mind that we really do appreciate the work done here, since we are using this from more than a year in production. I'm not attempting in any way to undermine the quality or the effort taken in making this library, but at this stage is really not easy to be to contribute it.

How other people feeling about this? There is any plan to add some tests, at least in a basic unitary level, for the core functionalities such as the ones in calendar_controller code?

[aleksanderwozniak]

That's a valid observation. In that case maybe you could contribute some tests? I'd be more than happy to review them.

On a side note - CalendarController is removed in v3.0.0, which is right around the corner. See the changelog for more details.

[swissonid]

@aleksanderwozniak I saw that the CalendarController is removed, what is the alternative for it?

[aleksanderwozniak]

@swissonid
Rebuilding the widget with updated values - check this readme and #434

[burhanrashid52]

@aleksanderwozniak if you are interested then I can contribute some tests for CellContnent or 1-2 end-to-end integration tests.

[aleksanderwozniak]

@burhanrashid52
Thanks. I'm already working on a test suite myself, but if I end up needing some extra help, I'll reach out to you.

[aleksanderwozniak]

A test suite has now been added - closing.