Write tests for the 8.x-1.x functionality to help prevent regressions.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

DamienMcKenna created an issue. See original summary.

larowlan’s picture

Assigned: Unassigned » larowlan

Working on this (a little bit each day)

DamienMcKenna’s picture

Component: Code » Tests
larowlan’s picture

Status: Active » Needs review
FileSize
6.24 KB

Here's a baseline test.

From here we can require all new features/bugfixes to come with new test asserts - I'd be happy to help with adding them.

Test for #2581351: API flaw - modified defaults don't affect existing records is included but commented out - so once this is committed, that issue should only need to uncomment the line to add coverage.

Also adds schema and removes redundant field settings (strict schema checking happens in tests)

larowlan’s picture

larowlan’s picture

Assigned: larowlan » Unassigned

DamienMcKenna’s picture

Status: Needs review » Fixed

Committed! Thank you so much, larowlan!

BTW I renamed the test to MetatagFieldTest, but that was the only change.

DamienMcKenna’s picture

Status: Fixed » Closed (fixed)

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