Overview

HEAD BROKEN: `\Drupal\Tests\canvas_ai\Kernel\CanvasPackagesDocsFileHashTest` depends on file hash.

We fixed a typo in #3563788: Incorrect documentation link for code components data fetching, which made this test fail.

But it was merged because we don't run phpunit when no code is changing.

Proposed resolution

  • ✅ For now, quickfix updating the hash.
  • We need to determine
    • How we can run tests on CI only if this file changes.
    • Or even if this is testing what it should be testing.

User interface changes

None

Issue fork canvas-3564164

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:

About issue forks

Comments

penyaskito created an issue. See original summary.

penyaskito’s picture

penyaskito
he/him
Primary language Spanish
Location 🧑🏽‍🌾 Seville 💃 Andalusia, UTC+2 🇪🇺
commented
Issue summary: View changes

penyaskito opened merge request !435

penyaskito’s picture

penyaskito
he/him
Primary language Spanish
Location 🧑🏽‍🌾 Seville 💃 Andalusia, UTC+2 🇪🇺
commented
Related issues: +#3548637: Canvas AI: Give AI more context about the libraries present in Canvas

Merged !435 to ensure HEAD passes.

Leaving open as we need to determine a better strategy for this. At least we need to document why we are tracking this file in tests, so we know when its failure is legit, or what action to take to fix it.

Adding #3548637: Canvas AI: Give AI more context about the libraries present in Canvas as related

  • penyaskito committed 25b8caa3 on 1.x 
    fix(AI): #3564164 HEAD BROKEN: `\Drupal\Tests\canvas_ai\Kernel\...
penyaskito’s picture

penyaskito
he/him
Primary language Spanish
Location 🧑🏽‍🌾 Seville 💃 Andalusia, UTC+2 🇪🇺
commented
Title: HEAD BROKEN: `\Drupal\Tests\canvas_ai\Kernel\CanvasPackagesDocsFileHashTest` depends on file hash » `\Drupal\Tests\canvas_ai\Kernel\CanvasPackagesDocsFileHashTest` depends on file hash
Issue summary: View changes
penyaskito’s picture

penyaskito
he/him
Primary language Spanish
Location 🧑🏽‍🌾 Seville 💃 Andalusia, UTC+2 🇪🇺
commented
Assigned: penyaskito » Unassigned