Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Updating testParametersContainingSubDefinitions #5

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Updating testParametersContainingSubDefinitions #5

wants to merge 1 commit into from

Commits on Dec 11, 2015

  1. Updating testParametersContainingSubDefinitions 

    I removed the `"foo"=>"bar"` part of the definitions array.
A DefinitionProvider should return an array of `DefinitionInterface`, indexed by key. `"bar"` is not a valid `DefinitionInterface`.

Note: the `ArrayDefinitionProvider` used in this test is NOT the one from Assembly (that wraps values not implementing the `DefinitionInterface` into `ParameterDefinition` objects). It is one I wrote for the tests that does pass the array "as is".
    @moufmouf
    moufmouf committed Dec 11, 2015
    Configuration menu
    Copy the full SHA
    ca3e607 View commit details
    Browse the repository at this point in the history