[go: up one dir, main page]

Page MenuHomePhabricator

Failing tests related to bug fix in schema.js
Closed, ResolvedPublicBUG REPORT

Description

Description

With function-schemata patch 876335 in place, the orchestrator's unit test 'Test that reify avoids infinite expansions' is still failing; needs diagnosis.

Background info: it was working prior to patch 862320. 862320 exposed bug T326357, which caused this test to fail for a different reason. Patch 876335 fixes T326357, but the test is still failing.

Patch 876335 also causes 3 WikiLambda integration tests to fail:

  • Request specifies JSON for new implementation
  • Request specifies JSON for edited version of existing implementation
  • Request specifies non-implementation as implementation, by reference

as shown here: https://integration.wikimedia.org/ci/job/quibble-vendor-mysql-php81-noselenium-docker/3507/console

Steps to reproduce (step by step instructions, with links, commands and necessary data to reproduce the error)

  1. Unskip and run the unit test named 'Test that reify avoids infinite expansions' (which is in mswOrchestrateTest.js)

Observed behavior

  • It returns the error below in Z22K2 (metadata value for "errors").
{
  Z1K1: 'Z5',
  Z5K1: 'Z518',
  Z5K2: {
    Z1K1: { Z1K1: 'Z7', Z7K1: 'Z885', Z885K1: 'Z518' },
    Z518K1: 'Z20015',
    Z518K2: { Z1K1: 'Z20015', Z20015K1: '1' },
    Z518K3: { Z1K1: 'Z5', Z5K1: 'Z502', Z5K2: [Object] }
  }
}

The error for Z518 (object_type_mismatch) is constructed in execute.js.
Expected behavior/Acceptance criteria (returned value, expected error, performance expectations, etc.)

  • It should pass.

Remove all the non-applicable tags from the "Tags" field, leave only the tags of the projects/repositories related to this task


Completion checklist

Event Timeline

DMartin-WMF renamed this task from 'Test that reify avoids infinite expansions' fails currently to Failing tests related to bug fix in schema.js.Jan 11 2023, 5:06 PM
DMartin-WMF updated the task description. (Show Details)
DMartin-WMF updated the task description. (Show Details)

Change 879097 had a related patch set uploaded (by Jforrester; author: Jforrester):

[mediawiki/services/function-schemata@master] Revert "Fix ZObjectKeyFactory.create() not to throw an error when input is Z24"

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

Change 879097 merged by jenkins-bot:

[mediawiki/services/function-schemata@master] Revert "Fix ZObjectKeyFactory.create() not to throw an error when input is Z24"

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

Change 879128 had a related patch set uploaded (by Jforrester; author: Jforrester):

[mediawiki/tools/wikilambda-cli@master] Update function-schemata sub-module to HEAD (4093097)

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

Change 879129 had a related patch set uploaded (by Jforrester; author: Jforrester):

[mediawiki/services/function-evaluator@master] Update function-schemata sub-module to HEAD (4093097)

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

Change 879130 had a related patch set uploaded (by Jforrester; author: Jforrester):

[mediawiki/services/function-orchestrator@master] Update function-schemata sub-module to HEAD (4093097)

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

Change 878138 had a related patch set uploaded (by Jforrester; author: Jforrester):

[mediawiki/extensions/WikiLambda@master] Update function-schemata sub-module to HEAD (4093097)

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

Change 878138 merged by jenkins-bot:

[mediawiki/extensions/WikiLambda@master] Update function-schemata sub-module to HEAD (4093097)

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

Change 879128 merged by jenkins-bot:

[mediawiki/tools/wikilambda-cli@master] Update function-schemata sub-module to HEAD (4093097)

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

Change 879129 merged by jenkins-bot:

[mediawiki/services/function-evaluator@master] Update function-schemata sub-module to HEAD (4093097)

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

Change 879130 merged by jenkins-bot:

[mediawiki/services/function-orchestrator@master] Update function-schemata sub-module to HEAD (4093097)

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

Looks like this is working now; moving to QA

DVrandecic claimed this task.

Test is still there I am told, so all good