Skip to content
This repository has been archived by the owner on Jun 3, 2024. It is now read-only.

Allow reconfiguration of behat.yml context params. #392

Merged
merged 9 commits into from
Jul 27, 2017
Merged

Conversation

dkinzer
Copy link
Contributor

@dkinzer dkinzer commented Jul 23, 2017

REF CIVIC-6614 GetDKAN/dkan#2025 #332

Adds some new build tools to allow sites to override behat cotext attributes
with custom settings set in cofig.yml

See .ahoy/site/tests/test_config-behat.rb for details of how this works, but the gist of it is
you add overriding configurations in config.yml

behat:
  contexts:
    datasets:
      defaults: {}
      fields: {}
      labels: {}
      sets: {}
    services:
      request_fields_map: {}

And this gets incorporated into the various behat.yml settings by running

ahoy build config`

QA Steps

  • run ahoy build test, if when tests pass... check out file in .ahoy/site/tests/test_config-behat.rb verify you understand what is going on and how this works.
  • If you wish recreate those tests manually yourself and verify they work.

Required PRS.

as per usual.

REF CIVIC-6614 GetDKAN/dkan#2025 #332

Adds some new build tools to allow sites to override behat cotext attributes
with custom settings set in cofig.yml

See .ahoy/site/tests/test_config-behat.rb for details of how this works, but the gist of it is
you add overriding configurations in config.yml

```
behat:
  contexts:
    datasets:
      defaults: {}
      fields: {}
      labels: {}
      sets: {}
    services:
      request_fields_map: {}
```

And this gets incorporated into the various behat.yml settings by running

```
ahoy build config`
```

as per usual.
@dkinzer dkinzer force-pushed the civic-6614 branch 2 times, most recently from 5bd51bc to c69325a Compare July 26, 2017 16:12
@dafeder dafeder merged commit da26953 into master Jul 27, 2017
@dafeder dafeder deleted the civic-6614 branch July 27, 2017 15:12
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants