test: run expectation tests in previous Safari

chrisguttandin · Aug 2, 2023 · 08b27db · 08b27db
1 parent 323f317
commit 08b27db
Show file tree

Hide file tree

Showing 6 changed files with 107 additions and 4 deletions.
diff --git a/config/grunt/aliases.js b/config/grunt/aliases.js
@@ -19,7 +19,8 @@ module.exports = {
             ...filter(isTarget(), 'sh:test-expectation-firefox-developer'),
             ...filter(isTarget('firefox-penultimate'), 'sh:test-expectation-firefox-penultimate'),
             ...filter(isTarget('firefox-previous'), 'sh:test-expectation-firefox-previous'),
-            ...filter(isTarget('safari'), 'sh:test-expectation-safari')
+            ...filter(isTarget('safari'), 'sh:test-expectation-safari'),
+            ...filter(isTarget('safari-previous'), 'sh:test-expectation-firefox-previous')
         ),
         ...filter(isType('integration'), 'sh:test-integration'),
         ...filter(isType('unit'), 'sh:test-unit')

diff --git a/config/grunt/sh.js b/config/grunt/sh.js
@@ -47,6 +47,9 @@ module.exports = (grunt) => {
         'test-expectation-safari': {
             cmd: `karma start config/karma/config-expectation-safari.js ${continuous ? '--concurrency Infinity' : '--single-run'}`
         },
+        'test-expectation-safari-previous': {
+            cmd: `karma start config/karma/config-expectation-safari-previous.js ${continuous ? '--concurrency Infinity' : '--single-run'}`
+        },
         'test-integration': {
             cmd: `karma start config/karma/config-integration.js ${continuous ? '--concurrency Infinity' : '--single-run'}`
         },

diff --git a/config/karma/config-expectation-safari-previous.js b/config/karma/config-expectation-safari-previous.js
@@ -0,0 +1,70 @@
+const { env } = require('process');
+const { DefinePlugin } = require('webpack');
+
+module.exports = (config) => {
+    config.set({
+        basePath: '../../',
+
+        browserDisconnectTimeout: 100000,
+
+        browserNoActivityTimeout: 100000,
+
+        browsers: ['WebkitHeadless'],
+
+        client: {
+            mocha: {
+                bail: true,
+                timeout: 20000
+            }
+        },
+
+        concurrency: 1,
+
+        files: ['test/expectation/safari/previous/**/*.js'],
+
+        frameworks: ['mocha', 'sinon-chai'],
+
+        preprocessors: {
+            'test/expectation/safari/previous/**/*.js': 'webpack'
+        },
+
+        reporters: ['dots'],
+
+        webpack: {
+            mode: 'development',
+            module: {
+                rules: [
+                    {
+                        test: /\.ts?$/,
+                        use: {
+                            loader: 'ts-loader',
+                            options: {
+                                compilerOptions: {
+                                    declaration: false,
+                                    declarationMap: false
+                                }
+                            }
+                        }
+                    }
+                ]
+            },
+            plugins: [
+                new DefinePlugin({
+                    'process.env': {
+                        CI: JSON.stringify(env.CI)
+                    }
+                })
+            ],
+            resolve: {
+                extensions: ['.js', '.ts'],
+                fallback: { util: false }
+            }
+        },
+
+        webpackMiddleware: {
+            noInfo: true
+        }
+    });
+
+    env.WEBKIT_HEADLESS_BIN = 'webkit-v16-4/pw_run.sh';
+};
diff --git a/config/karma/config-expectation-safari.js b/config/karma/config-expectation-safari.js
@@ -18,12 +18,12 @@ module.exports = (config) => {
 
         concurrency: 1,
 
-        files: ['test/expectation/safari/**/*.js'],
+        files: ['test/expectation/safari/current/**/*.js'],
 
         frameworks: ['mocha', 'sinon-chai'],
 
         preprocessors: {
-            'test/expectation/safari/**/*.js': 'webpack'
+            'test/expectation/safari/current/**/*.js': 'webpack'
         },
 
         reporters: ['dots'],

diff --git a/test/expectation/safari/media-stream.js → ...xpectation/safari/current/media-stream.js b/test/expectation/safari/media-stream.js → ...xpectation/safari/current/media-stream.js
@@ -1,5 +1,5 @@
 import { AudioContext } from 'standardized-audio-context';
-import { createMediaStreamWithAudioTrack } from '../../helpers/create-media-stream-with-audio-track';
+import { createMediaStreamWithAudioTrack } from '../../../helpers/create-media-stream-with-audio-track';
 
 describe('MediaStream', () => {
     let audioContext;

diff --git a/test/expectation/safari/previous/media-stream.js b/test/expectation/safari/previous/media-stream.js
@@ -0,0 +1,29 @@
+import { AudioContext } from 'standardized-audio-context';
+import { createMediaStreamWithAudioTrack } from '../../../helpers/create-media-stream-with-audio-track';
+
+describe('MediaStream', () => {
+    let audioContext;
+    let mediaStream;
+
+    afterEach(() => {
+        audioContext.close();
+        mediaStream.getTracks().forEach((track) => track.stop());
+    });
+
+    beforeEach(async () => {
+        audioContext = new AudioContext();
+        mediaStream = await createMediaStreamWithAudioTrack(audioContext);
+    });
+
+    // bug #15
+
+    it('should not expose the channelCount as part of the settings', () => {
+        expect(mediaStream.getAudioTracks()[0].getSettings().channelCount).to.be.undefined;
+    });
+
+    // bug #16
+
+    it('should not expose the sampleRate as part of the settings', () => {
+        expect(mediaStream.getAudioTracks()[0].getSettings().sampleRate).to.be.undefined;
+    });
+});