Skip to content
This repository has been archived by the owner on Dec 13, 2018. It is now read-only.

Config-loading tests #601

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

the-overengineer
Copy link

Adds tests for config loading. Tests these features:

  • Is config loaded from the default config.yml
  • Is config loaded from a local stormpath.yml
  • Is config loaded from a local stormpath.json
  • Is config loaded from the environment
  • Is config loaded from the config object
  • Is configuration overridden in the proper order
  • Are errors thrown if the id/secret/app_href are missing

Needs a tiny bit more work, however - when testing the application href missing line, the application is complaining that the ID/Secret are incorrect, after being generated for a new account created for that very application. Spent some time trying to track it, but running into a wall. Maybe a fresh pair of eyes would be helpful.

Fixes #286

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

Successfully merging this pull request may close these issues.

1 participant