Skip to content

Commit

Permalink
Merge pull request #14092 from eduardoj/fix/remove_vcr_default_for_fe…
Browse files Browse the repository at this point in the history
…atures

 Remove enforcing "vcr: true" for feature specs
  • Loading branch information
eduardoj committed Apr 4, 2023
2 parents d285d52 + 3def3e1 commit 34d6534
Show file tree
Hide file tree
Showing 65 changed files with 17 additions and 28,886 deletions.
5 changes: 2 additions & 3 deletions src/api/spec/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -143,10 +143,9 @@ Otherwise the cassette will not match and VCR tries record a new cassette each t
let(:apache_project) { create(:project, name: 'Apache') }
```

#### Enable VCR for Model and Controller Specs
#### Enable VCR

To make loading tests faster, we only include VCR in feature tests by default.
However, sometimes you also get and want to verify a backend response in a model or controller test.
You may want to store a backend response in a spec test.
Make sure you enable VCR in the test metadata like this:

```
Expand Down

This file was deleted.

This file was deleted.

This file was deleted.

Loading

0 comments on commit 34d6534

Please sign in to comment.