Skip to content

[28.x] testutil/fixtures/plugin: makePluginBundle: fix invalid spec#50820

Merged
thaJeztah merged 1 commit intomoby:28.xfrom
thaJeztah:28.x_fix_plugin_fixture
Aug 27, 2025
Merged

[28.x] testutil/fixtures/plugin: makePluginBundle: fix invalid spec#50820
thaJeztah merged 1 commit intomoby:28.xfrom
thaJeztah:28.x_fix_plugin_fixture

Conversation

@thaJeztah
Copy link
Member

relates to:

The fixture was using an invalid formatted Capability, which wasn't detected in the old API code, but fails with the API module, for example:

=== RUN   TestPluginsWithRuntimes
    plugin_test.go:269: assertion failed: error is not nil: json: error calling MarshalText for type plugin.CapabilityID: capability "docker.dummy/1.0" cannot contain a dot
--- FAIL: TestPluginsWithRuntimes (0.63s)
=== RUN   TestPluginBackCompatMediaTypes
    plugin_test.go:331: assertion failed: error is not nil: json: error calling MarshalText for type plugin.CapabilityID: capability "docker.dummy/1.0" cannot contain a dot
--- FAIL: TestPluginBackCompatMediaTypes (0.11s)
FAIL

This patch applies the same changes as ee560a3 in the master branch.

- What I did

- How I did it

- How to verify it

- Human readable description for the release notes

- A picture of a cute animal (not mandatory but encouraged)

The fixture was using an invalid formatted Capability, which wasn't
detected in the old API code, but fails with the API module, for example:

    === RUN   TestPluginsWithRuntimes
        plugin_test.go:269: assertion failed: error is not nil: json: error calling MarshalText for type plugin.CapabilityID: capability "docker.dummy/1.0" cannot contain a dot
    --- FAIL: TestPluginsWithRuntimes (0.63s)
    === RUN   TestPluginBackCompatMediaTypes
        plugin_test.go:331: assertion failed: error is not nil: json: error calling MarshalText for type plugin.CapabilityID: capability "docker.dummy/1.0" cannot contain a dot
    --- FAIL: TestPluginBackCompatMediaTypes (0.11s)
    FAIL

This patch applies the same changes as ee560a3
in the master branch.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah thaJeztah merged commit 74e4ea4 into moby:28.x Aug 27, 2025
235 of 236 checks passed
@thaJeztah thaJeztah deleted the 28.x_fix_plugin_fixture branch August 27, 2025 10:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants