We've enabled automated testing, but now we need tests.

CommentFileSizeAuthor
#2 3175519-2.patch1.2 KBvuil

Comments

colan created an issue. See original summary.

vuil’s picture

vuil
Location Bulgaria 🇧🇬 🇪🇺 🌍
commented
Status: Active » Needs review
StatusFileSize
new 3175519-2.patch1.2 KB

I added a blank test base.

  • vuil committed 42b9678 on 2.x 
    Issue #3175519 by vuil, colan: Add tests
vuil’s picture

vuil
Location Bulgaria 🇧🇬 🇪🇺 🌍
commented
Status: Needs review » Needs work

The module needs more tests.

vuil’s picture

vuil
Location Bulgaria 🇧🇬 🇪🇺 🌍
commented
Status: Needs work » Fixed

I close the issue as Fixed because there are not activity for 3 months period.

colan’s picture

colan
he/him
Primary language English
Location Toronto 🇨🇦
commented
Status: Fixed » Active

But we should still add tests, at least basic functional ones, to ensure nothing breaks when we get patches from contributors.

vuil’s picture

vuil
Location Bulgaria 🇧🇬 🇪🇺 🌍
commented
Status: Active » Needs work
vuil’s picture

vuil
Location Bulgaria 🇧🇬 🇪🇺 🌍
commented
Status: Needs work » Fixed

I close the issue as Done. We'll continue a later stage.

Now that this issue is closed, please review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, please credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

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