2186165210

Build Type

github

Committed by GitHub

Commit Message

<a href="https://github.com/gulpjs/remove-bom-stream/commit/<a class=hub.com/gulpjs/remove-bom-stream/commit/<a class="double-link" href="https://git"><a class=hub.com/gulpjs/remove-bom-stream/commit/<a class="double-link" href="https://git"><a class=hub.com/gulpjs/remove-bom-stream/commit/<a class="double-link" href="https://git"><a class=hub.com/gulpjs/remove-bom-stream/commit/a8439dc4faa11e04774e59ccfb9f712521f4421a">a8439dc4f">&lt;a href=&quot;https://github.com/gulpjs/remove-bom-stream/commit/</a><a class="double-link" href="https://github.com/gulpjs/remove-bom-stream/commit/&lt;a class=&quot;double-link&quot; href=&quot;https://git">&lt;a class=</a>hub.com/gulpjs/remove-bom-stream/commit/&lt;a class=&quot;double-link&quot; href=&quot;https://git">&lt;a class=</a>hub.com/gulpjs/remove-bom-stream/commit/&lt;a class=&quot;double-link&quot; href=&quot;https://git">&lt;a class=</a>hub.com/gulpjs/remove-bom-stream/commit/&lt;a class=&quot;double-link&quot; href=&quot;https://git">&lt;a class=</a>hub.com/gulpjs/remove-bom-stream/commit/a8439dc4faa11e04774e59ccfb9f712521f4421a">a8439dc4f</a><a href="https://github.com/gulpjs/remove-bom-stream/commit/a8439dc4faa11e04774e59ccfb9f712521f4421a">&quot;&gt;&amp;lt;a href=&amp;quot;https://github.com/gulpjs/remove-bom-stream/commit/&lt;/a&gt;&lt;a class=&quot;double-link&quot; href=&quot;https://github.com/gulpjs/remove-bom-stream/commit/&amp;lt;a class=&amp;quot;double-link&amp;quot; href=&amp;quot;https://git&quot;&gt;&amp;lt;a class=&lt;/a&gt;hub.com/gulpjs/remove-bom-stream/commit/&amp;lt;a class=&amp;quot;double-link&amp;quot; href=&amp;quot;https://git&quot;&gt;&amp;lt;a class=&lt;/a&gt;hub.com/gulpjs/remove-bom-stream/commit/&amp;lt;a class=&amp;quot;double-link&amp;quot; href=&amp;quot;https://git&quot;&gt;&amp;lt;a class=&lt;/a&gt;hub.com/gulpjs/remove-bom-stream/commit/&amp;lt;a class=&amp;quot;double-link&amp;quot; href=&amp;quot;https://git&quot;&gt;&amp;lt;a class=&lt;/a&gt;hub.com/gulpjs/remove-bom-stream/commit/a8439dc4faa11e04774e59ccfb9f712521f4421a&quot;&gt;a8439dc4f&lt;/a&gt;&lt;a href=&quot;https://github.com/gulpjs/remove-bom-stream/commit/a8439dc4faa11e04774e59ccfb9f712521f4421a&quot;&gt;&amp;quot;&amp;gt;&amp;amp;lt;a href=&amp;amp;quot;https://github.com/gulpjs/remove-bom-stream/commit/&amp;lt;/a&amp;gt;&amp;lt;a... (continued)

Pull Request Pull Request #8: feat!: Require encoding option to avoid inspecting chunks for UTF-8 encoding

Run Details

11 of 12 branches covered (91.67%)

Branch coverage included in aggregate %.

22 of 23 new or added lines in 1 file covered. (95.65%)

25 of 26 relevant lines covered (96.15%)

123.92 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

94.74

/index.js

'use strict';

var through = require('through2');
var TextDecoder = require('util').TextDecoder;

var BOM = '\ufeff';

function removeBomStream(encoding) {
  encoding = (encoding || '').toLowerCase();
  var isUTF8 = (encoding === 'utf-8' || encoding === 'utf8');

  // Needed due to https://github.com/nodejs/node/pull/42779
  if (!isUTF8) {
    return through();
  }

  // Only used if encoding is UTF-8
  var decoder = new TextDecoder('utf-8', { ignoreBOM: false });

  var state = 0; // 0:Not removed, -1:In removing, 1:Already removed

  return through(onChunk);

  function onChunk(data, _, cb) {
    if (state === 1) {
      cb(null, data);
      return;
    }

    try {
      state = -1;

      var chunk = decoder.decode(data, { stream: true });

      // The first time we have data after a decode, it should have already removed the BOM
      if (chunk !== '') {
        chunk += decoder.decode(); // end of stream mode and clear inner buffer.

        // Node<=v12, TextDecoder#decode returns a BOM if it receives a BOM separately.
        // Ref https://github.com/nodejs/node/pull/30132
        if (chunk !== BOM) {
          state = 1;
          var buffer = Buffer.from(chunk, 'utf-8');

          cb(null, buffer);
          return;
        }
      }

      cb();
    } catch (err) {
      cb(err);
    }
  }
}

module.exports = removeBomStream;

1	'use strict';
2
3	var through = require('through2');	12✔
4	var TextDecoder = require('util').TextDecoder;	12✔
5
6	var BOM = '\ufeff';	12✔
7
8	function removeBomStream(encoding) {
9	encoding = (encoding \|\| '').toLowerCase();	108!
10	var isUTF8 = (encoding === 'utf-8' \|\| encoding === 'utf8');	108✔
11
12	// Needed due to https://github.com/nodejs/node/pull/42779
13	if (!isUTF8) {	108✔
14	return through();	36✔
15	}
16
17	// Only used if encoding is UTF-8
18	var decoder = new TextDecoder('utf-8', { ignoreBOM: false });	72✔
19
20	var state = 0; // 0:Not removed, -1:In removing, 1:Already removed	72✔
21
22	return through(onChunk);	72✔
23
24	function onChunk(data, _, cb) {
25	if (state === 1) {	636✔
26	cb(null, data);	528✔
27	return;	528✔
28	}
29
30	try {	108✔
31	state = -1;	108✔
32
33	var chunk = decoder.decode(data, { stream: true });	108✔
34
35	// The first time we have data after a decode, it should have already removed the BOM
36	if (chunk !== '') {	108✔
37	chunk += decoder.decode(); // end of stream mode and clear inner buffer.	75✔
38
39	// Node<=v12, TextDecoder#decode returns a BOM if it receives a BOM separately.
40	// Ref https://github.com/nodejs/node/pull/30132
41	if (chunk !== BOM) {	75✔
42	state = 1;	72✔
43	var buffer = Buffer.from(chunk, 'utf-8');	72✔
44
45	cb(null, buffer);	72✔
46	return;	72✔
47	}
48	}
49
50	cb();	36✔
51	} catch (err) {
NEW 52	cb(err);	×
53	}
54	}
55	}
56
57	module.exports = removeBomStream;	12✔

gulpjs / remove-bom-stream / 2186165210

Source File Press 'n' to go to next uncovered line, 'b' for previous

Source File
Press 'n' to go to next uncovered line, 'b' for previous