Skip to content

Flow transitions should have required properties #1856

@markscott-ms

Description

@markscott-ms

Bug Report

Steps to Reproduce:

  1. Define a simple architecture of two nodes and one relationship
  2. Add a flow to this architecture
  3. Add one transition to the flow, but use incorrect property names of relationship and summary instead of relationship-unique-id and description.
  4. Use calm validate to validate the architecture against the schema

Expected Result:

A validation error saying that relationship-unique-id and description are required fields of a transition

Actual Result:

The architecture is valid.

Environment:

calm CLI 1.16.0
Ubuntu 24.0
Node 20.x

Additional Context:

A bug exists in the CALM schema in flow.json. It may look like it does, but it does NOT define required properties for the transition element. It defines a property called required. The required block needs to be a level higher, as a sibling of properties, not a child.

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't workingcalm

    Type

    Projects

    Status

    Todo

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions