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.

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

Make global mock experimental feature and disable it by default #160

Merged
merged 2 commits into from
Oct 8, 2020

Conversation

surik
Copy link
Contributor

@surik surik commented Oct 8, 2020

Even though all tests passed and the global mock was tested with an external application,
it was pretty optimistic to enabled that by default.
This commit changes default behavior that mocks every cassette and adds an ability to enable
the experimental feature. To do so, extend your test.exs:

config :exvcr, [
  global_mock: true
]

In Travis CI we run the same test for both default behavior and global
mock.

Even though all tests passed and the global mock was tested with an external application,
it was pretty optimistic to enabled that by default.
This commit changes default behavior that mocks every cassette and adds an ability to enable
the experimental feature. To do so, extend your `test.exs`:

```elixir
config :exvcr, [
  global_mock: true
]
```

In Travis CI we run the same test for both default behaviour and global
mock.
@surik surik mentioned this pull request Oct 8, 2020
@coveralls
Copy link

coveralls commented Oct 8, 2020

Coverage Status

Coverage decreased (-1.0%) to 92.308% when pulling 982b6b7 on surik:global-mock-flag into f518f13 on parroty:master.

@surik surik changed the title Make global mock experimental feature and disable it by default WIP: Make global mock experimental feature and disable it by default Oct 8, 2020
@surik surik changed the title WIP: Make global mock experimental feature and disable it by default Make global mock experimental feature and disable it by default Oct 8, 2020
@surik
Copy link
Contributor Author

surik commented Oct 8, 2020

Also, here I'm addressing the issue with handle_body_request in a hackney adapter that I missed in the initial PR: 982b6b7

@parroty parroty merged commit 603a8e3 into parroty:master Oct 8, 2020
@parroty
Copy link
Owner

parroty commented Oct 8, 2020

Thanks!

tcitworld added a commit to framasoft/mobilizon that referenced this pull request Oct 8, 2020
parroty/exvcr#160 fixes an issue with v0.12.0

Signed-off-by: Thomas Citharel <tcit@tcit.fr>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants