Page MenuHomePhabricator

Quibble runs core *integration* tests against Parsoid-as-an-extension, not *unit* tests
Closed, ResolvedPublic

Description

The integration tests from core are not guaranteed to pass once extensions are loaded, since they can have arbitrary side-effects on core functions via hooks.

For example, installing Scribunto makes core integration tests fail, because (among other things) one of the tests enumerates all the registered namespaces, and Scribunto adds the 'Module' and 'Module_talk' namespaces.

For this reason, integration-config only runs extension tests and core *unit tests* when extensions are installed.

But Parsoid at this moment is running the full test suite including integration tests.

Event Timeline

Change 655760 had a related patch set uploaded (by C. Scott Ananian; owner: C. Scott Ananian):
[integration/config@master] Zuul: Remove parsoid dependencies

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

Change 655742 had a related patch set uploaded (by C. Scott Ananian; owner: C. Scott Ananian):
[integration/config@master] Ensure Parsoid's tests include the Disambiguator extension (take 2)

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

Change 655732 had a related patch set uploaded (by C. Scott Ananian; owner: C. Scott Ananian):
[integration/quibble@master] Test Parsoid as if it were an extension

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

Change 655760 merged by jenkins-bot:
[integration/config@master] Zuul: Remove parsoid dependencies

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

Change 655732 merged by jenkins-bot:
[integration/quibble@master] Test Parsoid as if it were an extension

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

Change 685502 had a related patch set uploaded (by Hashar; author: Hashar):

[integration/quibble@master] Release Quibble 0.0.47

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

Change 685502 merged by jenkins-bot:

[integration/quibble@master] Release Quibble 0.0.47

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

Change 655742 merged by jenkins-bot:

[integration/config@master] Ensure Parsoid's tests include the Disambiguator extension (take 2)

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

Mentioned in SAL (#wikimedia-releng) [2021-05-05T23:52:38Z] <James_F> Zuul: Ensure Parsoid's tests include the Disambiguator extension (take 2) T271863 T237538