Commit d18d515
committed
Fix Capybara screenshots in feature specs
@chrislo & I accidentally removed the screenshotting capability in #6669 at the
same time as we removed the redundant configuration for running feature specs
in a browser using the cuprite driver, because we were under the mistaken
impression that rack-test driver did not support screenshots.
As @murugapl & @misaka pointed out, capybara-screenshot does actually support
HTML screenshots when using rack-test. These screenshots are improved by
setting `Capybara.asset_host = "http://localhost:4000"` so that assets can be
loaded if you have the Rails app running locally.
This commit partially reverts this commit [1] to restore the screenshotting
behaviour.
[1]: 0a95de71 parent e00291c commit d18d515
3 files changed
Lines changed: 13 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
117 | 117 | | |
118 | 118 | | |
119 | 119 | | |
| 120 | + | |
120 | 121 | | |
121 | 122 | | |
122 | 123 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
240 | 240 | | |
241 | 241 | | |
242 | 242 | | |
| 243 | + | |
| 244 | + | |
| 245 | + | |
243 | 246 | | |
244 | 247 | | |
245 | 248 | | |
246 | 249 | | |
247 | 250 | | |
248 | 251 | | |
249 | 252 | | |
| 253 | + | |
| 254 | + | |
250 | 255 | | |
251 | 256 | | |
252 | 257 | | |
| |||
435 | 440 | | |
436 | 441 | | |
437 | 442 | | |
| 443 | + | |
| 444 | + | |
| 445 | + | |
| 446 | + | |
438 | 447 | | |
439 | 448 | | |
440 | 449 | | |
| |||
935 | 944 | | |
936 | 945 | | |
937 | 946 | | |
| 947 | + | |
938 | 948 | | |
939 | 949 | | |
940 | 950 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
108 | 108 | | |
109 | 109 | | |
110 | 110 | | |
| 111 | + | |
111 | 112 | | |
112 | 113 | | |
113 | 114 | | |
114 | 115 | | |
115 | 116 | | |
| 117 | + | |
116 | 118 | | |
117 | 119 | | |
118 | 120 | | |
| |||
0 commit comments