Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add isElectronMain env test #13

Merged
merged 11 commits into from
Sep 19, 2019
Merged

feat: add isElectronMain env test #13

Show file tree
Hide file tree
Changes from 2 commits
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
ae7767d
feat: support isElectronMain
hugomrdias Sep 18, 2019
ec643db
fix: fix env tests
hugomrdias Sep 18, 2019
8cafe1c
fix: fix env tests assertions
hugomrdias Sep 19, 2019
8fed5e2
fix: fix typo
hugomrdias Sep 19, 2019
7a2a278
fix: fix typo
hugomrdias Sep 19, 2019
57bcb6c
fix: fix typo
hugomrdias Sep 19, 2019
d1c7adf
fix: fix typo
hugomrdias Sep 19, 2019
44a4e61
fix: fix typo
hugomrdias Sep 19, 2019
5506ac3
fix: fix typo
hugomrdias Sep 19, 2019
1ebea77
chore: fix linter
hugomrdias Sep 19, 2019
748ad4c
Merge branch 'feat/electron-main' of github.com:ipfs/js-ipfs-utils in…
hugomrdias Sep 19, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions src/env.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@ const isElectron = require('is-electron')
const IS_ENV_WITH_DOM = typeof window === 'object' && typeof document === 'object' && document.nodeType === 9
const IS_ELECTRON = isElectron()
const IS_BROWSER = IS_ENV_WITH_DOM && !IS_ELECTRON
const IS_ELECTRON_MAIN = IS_ELECTRON && !IS_ENV_WITH_DOM
const IS_ELECTRON_RENDERER = IS_ELECTRON && IS_ENV_WITH_DOM
const IS_NODE = typeof require === 'function' && typeof process !== 'undefined' && typeof process.release !== 'undefined' && process.release.name === 'node' && !IS_ELECTRON
// eslint-disable-next-line no-undef
Expand All @@ -13,6 +14,7 @@ const IS_TEST = typeof process !== 'undefined' && typeof process.env !== 'undefi
module.exports = {
isTest: IS_TEST,
isElectron: IS_ELECTRON,
isElectronMain: IS_ELECTRON_MAIN,
isElectronRenderer: IS_ELECTRON_RENDERER,
isNode: IS_NODE,
/**
Expand Down
43 changes: 36 additions & 7 deletions test/env.spec.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ const expect = chai.expect

describe('env', function () {
it('isElectron should have the correct value in each env', function () {
if (env.isElectron) {
if (env.isElectronMain) {
return expect(env.isElectron).to.be.true()
}
if (env.isElectronRenderer) {
Expand All @@ -25,15 +25,37 @@ describe('env', function () {
if (env.isWebWorker) {
return expect(env.isElectron).to.be.false()
}

throw new Error('Should not fail')
})

it('isElectronRenderer should have the correct value in each env', function () {
if (env.isElectron && !env.isElectronRenderer) {
return expect(env.isElectronRenderer).to.be.false()
it('isElectronMain should have the correct value in each env', function () {
if (env.isElectronMain) {
return expect(env.isElectronMain).to.be.true()
hugomrdias marked this conversation as resolved.
Show resolved Hide resolved
}
if (env.isElectronRenderer) {
return expect(env.isElectronMain).to.be.false()
}
if (env.isBrowser) {
return expect(env.isElectronMain).to.be.false()
}
if (env.isNode) {
return expect(env.isElectronMain).to.be.false()
}
if (env.isWebWorker) {
return expect(env.isElectronMain).to.be.false()
}

throw new Error('Should not fail')
achingbrain marked this conversation as resolved.
Show resolved Hide resolved
})

it('isElectronRenderer should have the correct value in each env', function () {
if (env.isElectronRenderer) {
return expect(env.isElectronRenderer).to.be.true()
}
if (env.isElectronMain) {
return expect(env.isElectronRenderer).to.be.false()
}
if (env.isBrowser) {
return expect(env.isElectronRenderer).to.be.false()
}
Expand All @@ -43,10 +65,12 @@ describe('env', function () {
if (env.isWebWorker) {
return expect(env.isElectronRenderer).to.be.false()
}

throw new Error('Should not fail')
})

it('isNode should have the correct value in each env', function () {
if (env.isElectron) {
if (env.isElectronMain) {
return expect(env.isNode).to.be.false()
}
if (env.isElectronRenderer) {
Expand All @@ -61,10 +85,12 @@ describe('env', function () {
if (env.isWebWorker) {
return expect(env.isNode).to.be.false()
}

throw new Error('Should not fail')
})

it('isBrowser should have the correct value in each env', function () {
if (env.isElectron) {
if (env.isElectronMain) {
return expect(env.isBrowser).to.be.false()
}
if (env.isElectronRenderer) {
Expand All @@ -79,10 +105,11 @@ describe('env', function () {
if (env.isWebWorker) {
return expect(env.isBrowser).to.be.false()
}
throw new Error('Should not fail')
})

it('isWebWorker should have the correct value in each env', function () {
if (env.isElectron) {
if (env.isElectronMain) {
return expect(env.isWebWorker).to.be.false()
}
if (env.isElectronRenderer) {
Expand All @@ -97,5 +124,7 @@ describe('env', function () {
if (env.isWebWorker) {
return expect(env.isWebWorker).to.be.true()
}

throw new Error('Should not fail')
})
})