You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
describe('when dealing with nested objects',function(){it('should not throw if registered field is changed',function(){constconfig={nested: {foo: {pleaseChangeMe: 'pleaseChangeMe'}}};constsnapshot=bedrock._snapshotOverrideFields({
config,fields: ['nested']});// change fieldsconfig.nested.foo.pleaseChangeMe='changed';constfn=bedrock._ensureConfigOverride.bind(this,{
config,configOverrideSnapshot: snapshot});expect(fn).to.not.throw();});});
However:
1 failing
1) bedrock
_ensureConfigOverride
when dealing with nested objects
should not throw if registered field is changed:
AssertionError: expected [Function bound _ensureConfigOverride] to not throw an error but 'Error: The config field "nested" must…' was thrown
Config values are set in the map directly, but if parent path values are added to fields to watch then they will not be detected as properly changed. Exact desired behavior here when registering root paths needs to be decided
The text was updated successfully, but these errors were encountered:
tminard
changed the title
BROKEN? ensureConfigOverride doesnt appear to work
verify ensureConfigOverride behaves as expected
Feb 5, 2025
This would be expected to pass:
However:
Config values are set in the map directly, but if parent path values are added to fields to watch then they will not be detected as properly changed. Exact desired behavior here when registering root paths needs to be decided
The text was updated successfully, but these errors were encountered: