Since we have already completed the API connection and changes in module configurations, it would be nice to have some of the functional/Unit tests which would help in detection of bugs in case new features are added/modified.

CommentFileSizeAuthor
#2 3154091-2.patch2.5 KBgaurav.kapoor

Comments

GabrielMartinelli created an issue. See original summary.

gaurav.kapoor’s picture

gaurav.kapoor commented
StatusFileSize
new 3154091-2.patch2.5 KB

I have added the test and resolved errors in the module based on test output.

gaurav.kapoor’s picture

gaurav.kapoor commented
Status: Active » Needs review
GabrielMartinelli’s picture

GabrielMartinelli commented
Status: Needs review » Reviewed & tested by the community

Thanks for the patch. Looks good to me.

  • gaurav.kapoor authored 7420e61 on 8.x-2.x 
    Issue #3154091 by gaurav.kapoor: Add test coverage for the module
gaurav.kapoor’s picture

gaurav.kapoor commented
Status: Reviewed & tested by the community » Fixed
gaurav.kapoor’s picture

gaurav.kapoor commented

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.