mapStateToAnyOfProps -> mapStateToCombinatorRendererProps StructuralKeywords should include const

[richturner]

Describe the bug

Currently the mapStateToCombinatorRendererProps function does not consider a validation error relating to a const constraint as invalid and therefore the incorrect indexOfFittingSchema can be returned.

Expected behavior

A const validation failure should be invalid and therefore indexOfFittingSchema should move onto the next schema for validation.

StructuralKeywords should include const

Steps to reproduce the issue

Use a schema containing an anyOf array with const constraints and the indexOfFittingSchema will always return 0

Screenshots

No response

In which browser are you experiencing the issue?

Google Chrome 91.0.4472.164

Framework

No response

RendererSet

No response

Additional context

No response

[sdirix]

Thanks for the report! Would you be willing to contribute the change including a test case? This would be the fastest way to get the change in.

[richturner]

Pull request created; I couldn't get the tests to compile without switching this import:

import { setValidationMode } from '../../lib';

to :

import { setValidationMode } from '../../src/actions';

Don't know where lib comes from? I reverted this change in the PR but wanted to make you aware.

[sdirix]

lib is the output directory of our build. So switching the import path is certainly the right thing to do for the tests. Maybe this came in via the editor's auto import feature.