-
-
Notifications
You must be signed in to change notification settings - Fork 753
Make specs and features global configuration proof #2602
Conversation
e21fc19
to
51b7b2f
Compare
Hm I was thinking more along the lines of modifying |
41a6129
to
3050e32
Compare
@JonRowe Addressed your concerns, now it's less hacky. Please take another look. |
c1f3081
to
e3a4e67
Compare
If a `~/.config/rspec/options` file contains the following: --color --profile 2 --format progress --require pry specs fail, since `--profile 2` option is picked up and used by the code under test. `$XDG_CONFIG_HOME/rspec/options` has to be isolated for some specs.
If a `~/.config/rspec/options` file contains the following: --color --profile 2 --format progress a number of features fail, since options are picked up and used by the code under test. Resetting `XDG_CONFIG_HOME` for those features isolates the global configuration.
ea7d2df
to
99f9c28
Compare
AppVeyor failure is unrelated. |
This still isn't what I meant, we should set the config required to make specs pass, not ignore the config thats there... |
@JonRowe I can't find a good way around some of the failing features. E.g. in my
and use it
there is nothing that a I can put in that "configuration with defaults" to override the reset the global setting back to default (format for running all is in progress, format for running one is documentation), either of those fails:
Do you have some hints? |
I've already removed |
If a
~/.config/rspec/options
file contains the following:some specs and features fail, since they are run with options that
contradict the default ones, and expectations are not met.
Example failures:
This fixes rspec/rspec-dev#217. Similar issues affect features of
rspec-mocks
andrspec-expectations
, will send out pull requests shortly.NOTE: Unfortunately, this doesn't fix the issue for configuration stored in
~/.rspec
. My advice is to move it over to~/.config/rspec/options
.NOTE: Unfortunately, this doesn't fix the issue for
--require pry
for features.