Skip to content
This repository was archived by the owner on Jul 29, 2024. It is now read-only.

Provide access to the processed configuration block #1724

Closed
wants to merge 2 commits into from

Conversation

crrobinson14
Copy link
Contributor

This is a continuation of PR #1708 with a few changes:

  1. Un-bork my fork (that OUGHT to be a song).
  2. Use the maintainers' recommended pattern for the call.
  3. On instinct I went with a promise return to be consistent getCapabilities() - let me know if that should change (a promise is not actually required here).
  4. Added a comment block to referenceConfig.js to illustrate its use.

@sjelin
Copy link
Contributor

sjelin commented Jan 14, 2015

Looks good, maybe add a test though? It's pretty simple to do, just put a jasmine/protractor test in /spec/basic and name it something ending in _spec.js and it run automatically.

@crrobinson14
Copy link
Contributor Author

I added a simple test as you suggested.

@juliemr
Copy link
Member

juliemr commented Jan 15, 2015

Looks good to me - I'm going to update the commit message to match with our pattern and merge.


describe('configuration elements', function() {
it('should have access to the processed config block', function() {
browser.getProcessedConfig().then(function(config) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This will actually fail in our CI where the browser is not always Chrome - I'll modify slightly.

@crrobinson14
Copy link
Contributor Author

Thanks for all the help!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants