From fec54c9c5ddf587febc11027d79ca70b0e6b3dac Mon Sep 17 00:00:00 2001 From: Shelley Vohr Date: Wed, 7 Aug 2019 11:41:56 -0700 Subject: [PATCH] test: remove ffi specs (#19661) --- spec/modules-spec.js | 20 -------------------- 1 file changed, 20 deletions(-) diff --git a/spec/modules-spec.js b/spec/modules-spec.js index 90c2376a4e..ac63c01651 100644 --- a/spec/modules-spec.js +++ b/spec/modules-spec.js @@ -54,26 +54,6 @@ describe('modules support', () => { } }) - // TODO(alexeykuzmin): Disabled during the Chromium 62 (Node.js 9) upgrade. - // Enable it back when "ffi" module supports Node.js 9. - // https://github.com/electron/electron/issues/11274 - xdescribe('ffi', () => { - before(function () { - if (!nativeModulesEnabled || process.platform === 'win32' || - process.arch === 'arm64') { - this.skip() - } - }) - - it('does not crash', () => { - const ffi = require('ffi') - const libm = ffi.Library('libm', { - ceil: ['double', ['double']] - }) - expect(libm.ceil(1.5)).to.equal(2) - }) - }) - describe('q', () => { const Q = require('q') describe('Q.when', () => {