Disable audio notifications on Windows 7 and lower

This commit is contained in:
Daniel Gasienica 2018-05-02 19:58:23 -04:00
parent a102016ed8
commit ecf23b6b2e
2 changed files with 18 additions and 1 deletions

View file

@ -23,6 +23,21 @@ describe('Settings', () => {
});
context('on Windows', () => {
context('version 7', () => {
beforeEach(() => {
sandbox.stub(process, 'platform').value('win32');
sandbox.stub(os, 'release').returns('7.0.0');
});
afterEach(() => {
sandbox.restore();
});
it('should return false', () => {
assert.isFalse(Settings.isAudioNotificationSupported());
});
});
context('version 8+', () => {
beforeEach(() => {
sandbox.stub(process, 'platform').value('win32');

View file

@ -1,4 +1,6 @@
import * as OS from '../OS';
const MIN_WINDOWS_VERSION = '8.0.0';
export const isAudioNotificationSupported = () =>
OS.isWindows() || OS.isMacOS();
OS.isWindows(MIN_WINDOWS_VERSION) || OS.isMacOS();