Problem/Motivation

I think it would be useful to have some automated tests to verify contributions and reduce the hassle of manual testing.

Proposed resolution

Write some automated tests.

Remaining tasks

Patch!

User interface changes

None.

API changes

None.

Data model changes

None.

CommentFileSizeAuthor
#2 2684759-testing-2.patch397 bytesSam152
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Sam152 created an issue. See original summary.

Sam152’s picture

Status: Active » Needs review
FileSize
397 bytes

It looks like there are already tests in the module. I didn't realise because they aren't enabled here. We should enable them if possible.

Also, now that #2680953: Ensure simple_sitemap has appropriate config schema. is in, we should be able to turn on strict checking of config schema, to ensure the schema and the config don't go out of sync in the future.

  • gbyte.co committed 42a9c46 on 8.x-2.x
    Issue #2684759 by Sam152: Turn on strict checking on config schema.
    
gbyte’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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