dependentSchemas and propertyDependencies don't "define" properties

[gregsdennis]

Coming out of a Slack discussion and #624, we need tests that show that properties listed in dependentSchemas and propertyDependencies don't affect the validation results of additionalProperties or unevaluatedProperties.

[Vinit-Pandit]

@gregsdennis Can you please check if this test matches the requirements for this issue?

[
    {
        "description": "dependent schema with additionalProperties",
        "schema": {
            "$schema": "https://json-schema.org/draft/2019-09/schema",
            "dependentSchemas": {
                "foo": {}
            },
            "additionalProperties": false
        },
        "tests": [
            {
                "description": "dependentSchemas properties are counted as additional properties",
                "data": {
                    "foo": ""
                },
                "valid": false
            }
        ]
    },
    {
        "description": "dependent schema with unevaluatedProperties",
        "schema": {
            "$schema": "https://json-schema.org/draft/2019-09/schema",
            "dependentSchemas": {
                "foo": {}
            },
            "unevaluatedProperties": false
        },
        "tests": [
            {
                "description": "dependentSchemas properties are counted as unevaluated properties",
                "data": {
                    "foo": ""
                },
                "valid": false
            }
        ]
    }
]

[gregsdennis]

Yeah, those should do. We'll need similar tests for the new propertyDependencies in draft/next.

[Vinit-Pandit]

@gregsdennis ?

[{
        "description" : "propertyDependencies with additionalProperties",
        "schema" : {
            "propertyDependencies" : {
                "foo" : {}
            },
            "additionalProperties" : false
        },
        "tests" : [
            {
                "description": "propertyDependencies properties are count as additionalproperties" , 
                "data" : {"foo" : ""},
                "valid" : false
            }
        ]
    },
    {
        "description" : "propertyDependencies with unevaluatedProperties" ,
        "schema" : {
            "propertyDependencies" : {
                "foo" : {}
            },
            "unevaluatedProperties" : false
        },

        "test" : [
            {
                "description": "propertyDependencies properties are count as the unevaluatedProperties" , 
                "data": {"foo": ""},
                "valid": false
            }
        ]
    }]

[Vinit-Pandit]

Is this test also require for dependentSchemas in draft/next or this behaviour is going to be change in next ?

[Vinit-Pandit]

@gregsdennis i think this one need to be close as it done in #732

[gregsdennis]

Yes.