Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

wim leers created an issue. See original summary.

larowlan’s picture

Title: Automatically transform the `drupal-11.2.2-with-xb-0.7.2-alpha1.filled` fixture when testing non-SQLite databases » [PP-1] Automatically transform the `drupal-11.2.2-with-xb-0.7.2-alpha1.filled` fixture when testing non-SQLite databases
Status: Active » Postponed

#3538487: Don't allow passing uncollapsed inputs if using default expression fixes this, but leaves behind the skip tests. Just need to uncomment them when that's in.
See step 6 in https://git.drupalcode.org/project/experience_builder/-/blob/585efb4733b... which was required to import the DB on mysql - I suspect that's all we were missing in the original issue.

larowlan’s picture

wim leers’s picture

Title: [PP-1] Automatically transform the `drupal-11.2.2-with-xb-0.7.2-alpha1.filled` fixture when testing non-SQLite databases » Automatically transform the `drupal-11.2.2-with-xb-0.7.2-alpha1.filled` fixture when testing non-SQLite databases
Status: Postponed » Active

wim leers’s picture

Assigned: larowlan » Unassigned
Status: Active » Reviewed & tested by the community

Executed what @larowlan described in #2. Passes locally 👍

wim leers’s picture

Title: Automatically transform the `drupal-11.2.2-with-xb-0.7.2-alpha1.filled` fixture when testing non-SQLite databases » Update path tests should also run against SQLite

That passed!

Updating title to reflect that the actual work already happened elsewhere.

  • wim leers committed 175bd2c5 on 1.x
    Issue #3538537 by larowlan: Update path tests should also run against...
wim leers’s picture

Status: Reviewed & tested by the community » Fixed
wim leers’s picture

Assigned: Unassigned » wim leers
Status: Fixed » Needs work

Oops, I hadn't noticed that #3538487: Don't allow passing uncollapsed inputs if using default expression added another skip 😅

wim leers’s picture

Assigned: wim leers » Unassigned
Status: Needs work » Reviewed & tested by the community

wim leers’s picture

Title: Update path tests should also run against SQLite » Update path tests should also run against non-SQLite DBs

Also, hah, I misread this, too 🙈

Clearly, I was a bit too excited/eager by how simple @larowlan's #2 made things sound 😄

  • wim leers committed 2f043302 on 1.x
    Issue #3538537 by wim leers, larowlan: Update path tests should also run...
wim leers’s picture

Status: Reviewed & tested by the community » Fixed

That's better :)

Status: Fixed » Closed (fixed)

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