Skip to content

Commit

Permalink
fix lint
Browse files Browse the repository at this point in the history
  • Loading branch information
kvakil committed Jul 22, 2022
1 parent e3a0650 commit 2ade30a
Showing 1 changed file with 6 additions and 4 deletions.
10 changes: 6 additions & 4 deletions test/pummel/test-v8-serialize-buffer-too-large.js
Expand Up @@ -6,7 +6,7 @@ if (common.isIBMi)
common.skip('On IBMi, the rss memory always returns zero');

const v8 = require('v8');
const {kMaxLength} = require('buffer');
const { kMaxLength } = require('buffer');

const PADDING_STRING = `Lorem ipsum dolor sit amet, consectetur adipiscing elit. Proin non quam in diam laoreet rhoncus condimentum quis neque. Sed luctus arcu eget velit tincidunt rhoncus. Mauris eros libero, lobortis et dolor quis, interdum sagittis dolor. Maecenas sit amet nulla at risus ullamcorper gravida. Sed varius nulla vel faucibus accumsan. Sed luctus purus felis, sagittis vehicula justo sollicitudin sed. Duis laoreet lobortis condimentum. Nunc ac nisi quis dolor malesuada aliquet eget in quam.
Expand All @@ -28,10 +28,12 @@ for (let j = 0; j < 2 ** i; j++) {
const assert = require('assert');

const rssBefore = process.memoryUsage.rss();

// Fail to serialize a few times.
assert.throws(() => v8.serialize(toSerialize));
assert.throws(() => v8.serialize(toSerialize));
assert.throws(() => v8.serialize(toSerialize));
const expectedError = { code: 'ERR_BUFFER_TOO_LARGE' };
assert.throws(() => v8.serialize(toSerialize), expectedError);
assert.throws(() => v8.serialize(toSerialize), expectedError);
assert.throws(() => v8.serialize(toSerialize), expectedError);

// Check that (at least some of) the memory got freed.
const rssAfter = process.memoryUsage.rss();
Expand Down

0 comments on commit 2ade30a

Please sign in to comment.