Page MenuHomePhabricator

Migrate tests to constraint statement parameters
Closed, ResolvedPublic

Description

Many of the constraint checker tests still use template-style parameters, e. g. [ 'property' => 'P2' ]. Before we can remove support for constraint templates, we’ll have to migrate those tests to use statement-style parameters. This can be done with helper methods in the ConstraintParameter trait, like the existing methods propertyParameter, propertiesParameter, rangeParameter, namespaceParameter, and formatParameter.

Changes:

Event Timeline

Change 362395 had a related patch set uploaded (by Lucas Werkmeister (WMDE); owner: Lucas Werkmeister (WMDE)):
[mediawiki/extensions/WikibaseQualityConstraints@master] Add {class,relation}Parameter to ConstraintParameters

https://gerrit.wikimedia.org/r/362395

Change 369909 had a related patch set uploaded (by Lucas Werkmeister (WMDE); owner: Lucas Werkmeister (WMDE)):
[mediawiki/extensions/WikibaseQualityConstraints@master] Improve rangeParameter helper and use it in RangeCheckerTest

https://gerrit.wikimedia.org/r/369909

Change 369914 had a related patch set uploaded (by Lucas Werkmeister (WMDE); owner: Lucas Werkmeister (WMDE)):
[mediawiki/extensions/WikibaseQualityConstraints@master] Migrate ConnectionChecker tests to statement parameters

https://gerrit.wikimedia.org/r/369914

Change 369917 had a related patch set uploaded (by Lucas Werkmeister (WMDE); owner: Lucas Werkmeister (WMDE)):
[mediawiki/extensions/WikibaseQualityConstraints@master] Migrate OneOfChecker tests to statement parameters

https://gerrit.wikimedia.org/r/369917

Change 369946 had a related patch set uploaded (by Lucas Werkmeister (WMDE); owner: Lucas Werkmeister (WMDE)):
[mediawiki/extensions/WikibaseQualityConstraints@master] Migrate MandatoryQualifiersChecker tests to constraint statements

https://gerrit.wikimedia.org/r/369946

Change 369955 had a related patch set uploaded (by Lucas Werkmeister (WMDE); owner: Lucas Werkmeister (WMDE)):
[mediawiki/extensions/WikibaseQualityConstraints@master] Remove constraint_status from API tests

https://gerrit.wikimedia.org/r/369955

Change 369946 merged by jenkins-bot:
[mediawiki/extensions/WikibaseQualityConstraints@master] Migrate MandatoryQualifiersChecker tests to statement parameters

https://gerrit.wikimedia.org/r/369946

Change 369955 merged by jenkins-bot:
[mediawiki/extensions/WikibaseQualityConstraints@master] Remove constraint_status from API tests

https://gerrit.wikimedia.org/r/369955

Change 372162 had a related patch set uploaded (by Lucas Werkmeister (WMDE); owner: Lucas Werkmeister (WMDE)):
[mediawiki/extensions/WikibaseQualityConstraints@master] WIP: migrate DelegatingConstraintCheckerTest to statement params

https://gerrit.wikimedia.org/r/372162

Change 372171 had a related patch set uploaded (by Lucas Werkmeister (WMDE); owner: Lucas Werkmeister (WMDE)):
[mediawiki/extensions/WikibaseQualityConstraints@master] Migrate SpecialConstraintReportTest to statement parameters

https://gerrit.wikimedia.org/r/372171

Change 362395 merged by jenkins-bot:
[mediawiki/extensions/WikibaseQualityConstraints@master] Add {class,relation}Parameter to ConstraintParameters

https://gerrit.wikimedia.org/r/362395

Change 369909 merged by jenkins-bot:
[mediawiki/extensions/WikibaseQualityConstraints@master] Improve rangeParameter helper and use it in RangeCheckerTest

https://gerrit.wikimedia.org/r/369909

Change 369914 merged by jenkins-bot:
[mediawiki/extensions/WikibaseQualityConstraints@master] Migrate ConnectionChecker tests to statement parameters

https://gerrit.wikimedia.org/r/369914

Change 369917 merged by jenkins-bot:
[mediawiki/extensions/WikibaseQualityConstraints@master] Migrate OneOfChecker tests to statement parameters

https://gerrit.wikimedia.org/r/369917

Change 372171 merged by jenkins-bot:
[mediawiki/extensions/WikibaseQualityConstraints@master] Migrate SpecialConstraintReportTest to statement parameters

https://gerrit.wikimedia.org/r/372171

Change 372162 merged by jenkins-bot:
[mediawiki/extensions/WikibaseQualityConstraints@master] Migrate DelegatingConstraintCheckerTest to statement params

https://gerrit.wikimedia.org/r/372162