From 274d44990e8973a069f8214dfa742d4b5e1420b0 Mon Sep 17 00:00:00 2001
From: Bart Veneman <bartveneman@users.noreply.github.com>
Date: Mon, 9 Mar 2020 23:15:47 +0100
Subject: [PATCH 1/3] fix error when referencing a css file directly

---
 src/index.js                 |   8 ++++
 test/index.js                |   7 ++++
 test/snapshots/index.js.md   |  73 +----------------------------------
 test/snapshots/index.js.snap | Bin 605 -> 117 bytes
 4 files changed, 17 insertions(+), 71 deletions(-)

diff --git a/src/index.js b/src/index.js
index c17fc6a..546f9cd 100644
--- a/src/index.js
+++ b/src/index.js
@@ -36,6 +36,14 @@ module.exports = async (url, {waitUntil = 'networkidle0'} = {}) => {
 		)
 	}
 
+	// If the response is a CSS file, return that file
+	// instead of running our complicated setup
+	const headers = response.headers()
+
+	if (headers['content-type'].includes('text/css')) {
+		return Promise.resolve(response.text())
+	}
+
 	const coverage = await page.coverage.stopCSSCoverage()
 
 	// Get all CSS generated with the CSSStyleSheet API
diff --git a/test/index.js b/test/index.js
index fc8681c..32c0e72 100644
--- a/test/index.js
+++ b/test/index.js
@@ -84,6 +84,13 @@ test('it finds inline styles - JS', async t => {
 	t.snapshot(actual)
 })
 
+test.only('it returns a direct link to a CSS file', async t => {
+	const actual = await extractCss(server.url + '/import-in-css.css')
+
+	t.true(actual.includes('.css-imported-with-css {}'))
+	t.snapshot(actual)
+})
+
 test('it rejects if the url has an HTTP error status', async t => {
 	server.get('/404-page', (req, res) => {
 		res.status(404).send()
diff --git a/test/snapshots/index.js.md b/test/snapshots/index.js.md
index 69c6d1b..982e040 100644
--- a/test/snapshots/index.js.md
+++ b/test/snapshots/index.js.md
@@ -4,77 +4,8 @@ The actual snapshot is saved in `index.js.snap`.
 
 Generated by [AVA](https://ava.li).
 
-## it finds CSS implemented in a mixed methods (inline, links, style tags)
+## it returns a direct link to a CSS file
 
 > Snapshot 1
 
-    `@import url("import-in-css.css");␊
-    ␊
-    .link-in-html { }␊
-    .css-imported-with-css {}␊
-    @import url("import-in-css.css");␊
-    ␊
-    .link-tag-created-with-js {}␊
-    ␊
-    .css-imported-with-css {}␊
-    .style-tag-in-html { color: green; }␊
-    .js-insertRule { color: red; }␊
-    .style-tag-js { }␊
-    [x-extract-css-inline-style] { background-image: url('background-image-inline-style-attribute-in-html'); }␊
-    [x-extract-css-inline-style] { background-image: url("background-image-inline-style-js-cssText"); }␊
-    [x-extract-css-inline-style] { background-image: url("background-image-inline-style-js-with-prop"); }`
-
-## it finds css in a <link> tag - HTML
-
-> Snapshot 1
-
-    `@import url("import-in-css.css");␊
-    ␊
-    .link-in-html { }␊
-    .css-imported-with-css {}`
-
-## it finds css in a <link> tag - JS
-
-> Snapshot 1
-
-    `@import url("import-in-css.css");␊
-    ␊
-    .link-tag-created-with-js {}␊
-    ␊
-    .css-imported-with-css {}`
-
-## it finds css in a <style> tag - HTML
-
-> Snapshot 1
-
-    `.css-imported-with-css {}␊
-    @import url("import-in-css.css");␊
-    .fixture { color: red; }`
-
-## it finds css in a <style> tag - JS
-
-> Snapshot 1
-
-    `.css-imported-with-js {}␊
-    @import url("import-in-js.css");␊
-    .fixture { color: red; }`
-
-## it finds css-in-js
-
-> Snapshot 1
-
-    '.bcMPWx { color: blue; }'
-
-## it finds inline styles - HTML
-
-> Snapshot 1
-
-    `[x-extract-css-inline-style] { color: red; font-size: 12px; }␊
-    [x-extract-css-inline-style] { color: blue }`
-
-## it finds inline styles - JS
-
-> Snapshot 1
-
-    `[x-extract-css-inline-style] { color: red; font-size: 12px; border-style: solid; }␊
-    [x-extract-css-inline-style] { border-color: blue; border-width: 1px; }`
+    '.css-imported-with-css {}'
diff --git a/test/snapshots/index.js.snap b/test/snapshots/index.js.snap
index 70c133c98f293638669ba0814833e4a102561207..3cae3c50cc5f5c4dbdb58219ed72601d1cf45904 100644
GIT binary patch
literal 117
zcmZ<^b5sb<ODrhP$S+YSGvZ=m=*;J{5bD&GXX#})`Kg)5M82DY0SuUf6M!Hk;z0T8
zbAGM!-PPCi*em^VG*D*dSix*)%xpd5?1@tfCp1=0kBM0`duH^6_>f7n7DdmCoV1Wz
RByhz#RtAeQAsKg|SpcfsCu9Hs

literal 605
zcmV-j0;2svRzV<BZeeh9Xm4~Nb}|YA0A6{2{s}>3O1ZAy#T$P~)gOxp00000000BK
z#=yY9!SJB=l8cbBtkZ@I^IBb061OlifPga-1H-X6?fPl+tc%y!PVA_dJ28t9EIJ>E
zg}L<QqErw0>~`Re`O0x%FC$o#k(q&kS6o>0S=I5!XI%1(Ual(@U<8Y*1F@W?!=m}t
z{p}kie{2k{vpmBH7QFz(uZn*sIOir<h8$r2&A<5hBu22P2MYtkT~A*9?|EWBB6*v>
zURdN;#t0VO0L0p9%Xt<{S?v6i_rL02Y1TJJu&4kl0|PSyJJ<(|tPFxoCQJ^Qxdr(}
zB?_fQIT}h3wr*ygZgO$49uO;OT61yf<z(h%g9I~5a&r`_6>7OaQo0busVTbUnI#z@
z4GPt@TtpjJl9;ZWT$Gv!Hy{hFACJ*`#U+(FsbGx=w<PE1<QG{fq!*>8=2?SWmIc(E
zSDac@5>%R#icnCLngW(am;tg6C=y+vn_5v)l$czi3${5A=(<#0u(DX7f~3Uc?DV4i
z(!3O)qY~3ot-wB0M;AuXr<+((Qk0ogT9OKLsJf;#xmGFRw+iT8pjjb6bCqam2RI%I
zit-D<wjd`VMnCGNAQZDhC`o8~X_*xzr9~Kt0L2Cgy`*H{fba^0qNJSCR5Vpgvls}3
zE3yS?`FSO}#hF#Wz%?`~sIXQ@$}dVuErKYvQYg;P$xI<8svx>hTm=;?&rB)F0GbXm
r9MwBnl(>_ilaSnkW^WYj@;!<@L0FV!;Y$x$1d{^*f*2%|<^=!%xquCr


From 3a1181b8c7362917611d6625ca1162ba9788d212 Mon Sep 17 00:00:00 2001
From: Bart Veneman <bartveneman@users.noreply.github.com>
Date: Mon, 9 Mar 2020 23:17:04 +0100
Subject: [PATCH 2/3] hey computer plz run all tests

---
 test/index.js                |   2 +-
 test/snapshots/index.js.md   |  75 +++++++++++++++++++++++++++++++++++
 test/snapshots/index.js.snap | Bin 117 -> 639 bytes
 3 files changed, 76 insertions(+), 1 deletion(-)

diff --git a/test/index.js b/test/index.js
index 32c0e72..0fe24b6 100644
--- a/test/index.js
+++ b/test/index.js
@@ -84,7 +84,7 @@ test('it finds inline styles - JS', async t => {
 	t.snapshot(actual)
 })
 
-test.only('it returns a direct link to a CSS file', async t => {
+test('it returns a direct link to a CSS file', async t => {
 	const actual = await extractCss(server.url + '/import-in-css.css')
 
 	t.true(actual.includes('.css-imported-with-css {}'))
diff --git a/test/snapshots/index.js.md b/test/snapshots/index.js.md
index 982e040..9cd150a 100644
--- a/test/snapshots/index.js.md
+++ b/test/snapshots/index.js.md
@@ -9,3 +9,78 @@ Generated by [AVA](https://ava.li).
 > Snapshot 1
 
     '.css-imported-with-css {}'
+
+## it finds CSS implemented in a mixed methods (inline, links, style tags)
+
+> Snapshot 1
+
+    `@import url("import-in-css.css");␊
+    ␊
+    .link-in-html { }␊
+    .css-imported-with-css {}␊
+    @import url("import-in-css.css");␊
+    ␊
+    .link-tag-created-with-js {}␊
+    ␊
+    .css-imported-with-css {}␊
+    .style-tag-in-html { color: green; }␊
+    .js-insertRule { color: red; }␊
+    .style-tag-js { }␊
+    [x-extract-css-inline-style] { background-image: url('background-image-inline-style-attribute-in-html'); }␊
+    [x-extract-css-inline-style] { background-image: url("background-image-inline-style-js-cssText"); }␊
+    [x-extract-css-inline-style] { background-image: url("background-image-inline-style-js-with-prop"); }`
+
+## it finds css in a <link> tag - HTML
+
+> Snapshot 1
+
+    `@import url("import-in-css.css");␊
+    ␊
+    .link-in-html { }␊
+    .css-imported-with-css {}`
+
+## it finds css in a <link> tag - JS
+
+> Snapshot 1
+
+    `@import url("import-in-css.css");␊
+    ␊
+    .link-tag-created-with-js {}␊
+    ␊
+    .css-imported-with-css {}`
+
+## it finds css in a <style> tag - HTML
+
+> Snapshot 1
+
+    `.css-imported-with-css {}␊
+    @import url("import-in-css.css");␊
+    .fixture { color: red; }`
+
+## it finds css in a <style> tag - JS
+
+> Snapshot 1
+
+    `.css-imported-with-js {}␊
+    @import url("import-in-js.css");␊
+    .fixture { color: red; }`
+
+## it finds css-in-js
+
+> Snapshot 1
+
+    '.bcMPWx { color: blue; }'
+
+## it finds inline styles - HTML
+
+> Snapshot 1
+
+    `[x-extract-css-inline-style] { color: red; font-size: 12px; }␊
+    [x-extract-css-inline-style] { color: blue }`
+
+## it finds inline styles - JS
+
+> Snapshot 1
+
+    `[x-extract-css-inline-style] { color: red; font-size: 12px; border-style: solid; }␊
+    [x-extract-css-inline-style] { border-color: blue; border-width: 1px; }`
diff --git a/test/snapshots/index.js.snap b/test/snapshots/index.js.snap
index 3cae3c50cc5f5c4dbdb58219ed72601d1cf45904..bce390c9847cb58781b5e8912ff9a90f9c267daf 100644
GIT binary patch
delta 622
zcmV-!0+Ic7{{(+QRzV<BZeeh9Xm4~Nb}|YA0EgG;XBLCDvTjg&z#6=@gdd9t00000
z000B~z`(%3$*_R$R-K&+KaWma8Gl6dgJ4Dm5Ksr=yPmxI-}A(NMDjL$y|Boyj1er_
z55%vEe<wKSCRm0XVE)a&`1vG8uqZoFt1y?oT$Jh|pWS~B+%aD{4(w$Fi)I4xgW5|j
zLdLRA8!pUibx}#&!Uz^!!_2^-owl53v6RKmKY9PF{*`8ZV+4!xurM$ji_@;3HqW|v
zjqSvain$ZB7{Q_$K+G#Htof|!_~SD!c}6eSl?pI|Mf-qQ&eCDgeCz)9jgmh$2G?1h
zVFZhsvoZ%TFf*`&{l<|_9e+hp7Ucj@qEK3tqoD+0>t^PGl!6RU(zNE{(#y%r%hoMP
zOxH~=N=<~DmIX2lC`*(*S<w}`sTCzfiOD6pU=4XdLsNB&ODc0xV->0ulJj%&i>wri
zQd6uI((?04bc-{qQmqsWjS4EP6>7PN)18!4nyP?mZwOB3aS`^pUVmC<MM-H<Dw_LH
zY+y2BB030xf*B>bIY8rTaR(a_?jkovxb(mw2G)ph3)E}rMX9NIps?1<0tIk!YEem0
zX%2?pk<9?vM>ukl5|gvji}FkJQh<(1Ob5m)$Y<*4!YKN56H7{pGLuS6Qeh5N*CZkx
zV7E#Mzg0l*0?i5mnqR9#OFO{vP*9X#0Ja4skp^Lju`GOPA&aCW%IHVk6pCWDgkDmz
zZ$NkjYA!_2B(oSO$^S|DMJcI85XDvs#rZjzDa1q+L>G#ypkn2jDJ2;|)4_!c0O{dj
IRwo7k0Q77e6951J

literal 117
zcmZ<^b5sb<ODrhP$S+YSGvZ=m=*;J{5bD&GXX#})`Kg)5M82DY0SuUf6M!Hk;z0T8
zbAGM!-PPCi*em^VG*D*dSix*)%xpd5?1@tfCp1=0kBM0`duH^6_>f7n7DdmCoV1Wz
RByhz#RtAeQAsKg|SpcfsCu9Hs


From 5392cc88cf411a0a834939d8706fcf9c3239d80a Mon Sep 17 00:00:00 2001
From: Bart Veneman <bartveneman@users.noreply.github.com>
Date: Mon, 9 Mar 2020 23:17:53 +0100
Subject: [PATCH 3/3] hey computer plz run all tests

---
 test/snapshots/index.js.md   |  12 ++++++------
 test/snapshots/index.js.snap | Bin 639 -> 636 bytes
 2 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/test/snapshots/index.js.md b/test/snapshots/index.js.md
index 9cd150a..3d93769 100644
--- a/test/snapshots/index.js.md
+++ b/test/snapshots/index.js.md
@@ -4,12 +4,6 @@ The actual snapshot is saved in `index.js.snap`.
 
 Generated by [AVA](https://ava.li).
 
-## it returns a direct link to a CSS file
-
-> Snapshot 1
-
-    '.css-imported-with-css {}'
-
 ## it finds CSS implemented in a mixed methods (inline, links, style tags)
 
 > Snapshot 1
@@ -84,3 +78,9 @@ Generated by [AVA](https://ava.li).
 
     `[x-extract-css-inline-style] { color: red; font-size: 12px; border-style: solid; }␊
     [x-extract-css-inline-style] { border-color: blue; border-width: 1px; }`
+
+## it returns a direct link to a CSS file
+
+> Snapshot 1
+
+    '.css-imported-with-css {}'
diff --git a/test/snapshots/index.js.snap b/test/snapshots/index.js.snap
index bce390c9847cb58781b5e8912ff9a90f9c267daf..fd6263aa3ca98e773bfd4845261a6085c15b59d0 100644
GIT binary patch
literal 636
zcmV-?0)zcQRzV<BZeeh9Xm4~Nb}|YA0L`kff-i6467Nf&yX6tXNFR#`00000000B~
zz`(%3$#5)AyMEd{>*6)G6FVyAPRwFt0D%}FzU#@W|2<FaM<j34*9(jM${4|-M}SzE
zOJ6QZ^^nhQ2kw}!90&F?f<@JUTD8-b^DLIK*!d^#f7QRztZ$59(KaA{P<zQm$XM2C
z!-aXRE-Hyz7{Q_^nHd;(#f3GWRULnP#wE|_<+@S<MzH96AeOUqSTx_dzkQ?RkBz}~
zmS-5jqBB?+7+w|sPH@gmunalC{F{IA^GS?gQ2|y4h6Q}L>g-hbd355+_#>hp1T%s~
z&4HMifgS8OMpgzvMn8wl+=Bd~5{1&D91SH1TQ@ULH@Uc24~UgCt+}}Lax(L>L4p}2
zxj72e3bkAyDP4%-)D+$F%#sX{28HTc6tkm<HoGJ-T{pQXH4$!F7RWFzBJBymsf>%T
z3-r=5D@sa>Qi1+T&d<p&vQj8YO|e!$wIK+L4O#ds$Rf#nCKD#=MgtN4CO71{^omO=
zb5g+?5pIEcExjl;HP0I4vMiwPyyDcNlAzKY48J3r0kRJ$5?!I2T2WGzm|UU@wmA>z
zx>Q}TvRI&kq{QUx^rHOIycD3L64O(yz&=w)7e>*in^;m(l$lgok_vOEx~4U`Rw?1P
z3g}&+Ss_4km1t=PI35a$@(aMWpd?WVy`*H{fba@LSSRI_rlKdbSqubRh8$06`FSO}
z#hFz==NTFmR9Gt{<rk%-7C{tSDHP}DWTp_~S%@waS3$+fGgC@3fTn{CNA*q?CGI5X
WBqX<>*(-@N;R67hnZTzf1^@srL?bT%

literal 639
zcmV-_0)YKNRzV<BZeeh9Xm4~Nb}|YA0EgG;XBLCDvTjg&z#6=@gdd9t00000000B~
zz`(%3$*_R$R-K&+KaWma8Gl6dgJ4Dm5Ksr=yPmxI-}A(NMDjL$y|Boyj1er_55%vE
ze<wKSCRm0XVE)a&`1vG8uqZoFt1y?oT$Jh|pWP1JF<&_j>}3RtW&-hp+Dk4%#<ETu
zF3f9nQAymw2o_z#%)p?Xww!0Nl*P_JdH<{am1cco1dH;pFfbg8)2^R3&$@Vx?Zl3X
zxf8P(!J-*J%quRe`K;>r<1;RKMlaWu3NV62`+!)^(qYkj>;CqQl0P;E*IAxn1dE!p
zGB7YRu!H@^$jTtdD5;lRT&$a!TaaH=lA5Aho>`Ki3lvePu0>H6<p5ElP+F9up#)*;
zX6AvEf(%g7wC3W{%gM~k)-6d)*G(=;O@y141u_gMOO!oX(G|L>6(vQ9$tAjA4S7IA
zQ+10=Dsxg}6{;1I^K<fxtQ3k;Q>+!z^7Be`i!-ZItrQH63M#A>YPpEhos?6Ws(@;5
z2u|m55%#%WT4qH_X;CVg`%r9PGGQV*2!VnbCAm33<7#mS8xigzH%7Shz##_Kh;R$k
zYw1O)sd=EV*2@9~aB*r;Nl<AHhToCR0NF=4a*`60v(t<6OY>5Ij!H}i#wy5X>gd8K
z`g9XZN{TX*N=s5<4prAAA{}72N(sMJK<@(03IUp{L`yrs@la5dUjViRC6NYUiLoqv
zX(5ZGB+BSV-4u#qwuD|%vTs0m1!^ut&m^-LD9Qgx`9&$IMG(bS3dQ+3nJL6X6hs$_
ZtDs`#nJFb1K-0m63jpciVOA#w007nzC2jx!