Problem/Motivation

We could use some phpunit tests for this module.

A very quick/simple initial win would be a Functional test that installs group_notify, tries to load the front page, and makes sure we don't blow up. ;)

This will catch things like:
- PHP fatal errors.
- Config schema mis-matches.
...

Proposed resolution

Add a trivial tests/src/Functional/LoadTest.php test for group_notify.

Remaining tasks

  1. Upload.
  2. Commit.
CommentFileSizeAuthor
#3 3164924.2_3.interdiff.txt341 bytesdww
#3 3164924-3.patch1.23 KBdww
#2 3164924-2.patch1.23 KBdww

Comments

dww created an issue. See original summary.

dww’s picture

Issue summary: View changes
Status: Active » Needs review
StatusFileSize
new1.23 KB
dww’s picture

StatusFileSize
new1.23 KB
new341 bytes

While looking this over via Slack, we discovered a typo in the namespace. New patch for completeness.

  • gregcube committed 5ea0c60 on 8.x-1.x authored by dww
    Issue #3164924 by dww: Add a trivial "load test"
    
dww’s picture

Issue summary: View changes
Status: Needs review » Fixed

Great, thanks! We can sort out the project node running tests at #3164923: [META] Add test coverage for Group Notify. Calling this one fixed, too. ;)

Status: Fixed » Closed (fixed)

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