Skip to content

Commit

Permalink
Run browser tests on Windows
Browse files Browse the repository at this point in the history
Current CI images support this, so we no longer need to skip it.
  • Loading branch information
sgravrock committed Aug 12, 2023
1 parent 7ae9f8f commit e284c49
Show file tree
Hide file tree
Showing 6 changed files with 1 addition and 31 deletions.
2 changes: 1 addition & 1 deletion .circleci/config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -64,7 +64,7 @@ jobs:
command: echo 'console.log(require("jasmine-core").version())' | node
- run:
name: Run tests
command: JASMINE_NO_BROWSER_TESTS=y npm test
command: npm test

saucelabs_integration_test:
executor: node18
Expand Down
6 changes: 0 additions & 6 deletions spec/esmIntegrationSpec.js
Original file line number Diff line number Diff line change
@@ -1,12 +1,6 @@
const { exec } = require('child_process');

describe('ESM integration', function() {
beforeEach(function() {
if (process.env.JASMINE_NO_BROWSER_TESTS) {
pending('skipping because the JASMINE_NO_BROWSER_TESTS env var is set');
}
});

it(
'supports ES modules as specs, helpers, and sources',
function(done) {
Expand Down
6 changes: 0 additions & 6 deletions spec/exitCodeIntegrationSpec.js
Original file line number Diff line number Diff line change
@@ -1,12 +1,6 @@
const { exec } = require('child_process');

describe('Exit code integration', function() {
beforeEach(function() {
if (process.env.JASMINE_NO_BROWSER_TESTS) {
pending('skipping because the JASMINE_NO_BROWSER_TESTS env var is set');
}
});

it(
'exits 0 on success',
async function() {
Expand Down
6 changes: 0 additions & 6 deletions spec/importMapSpec.js
Original file line number Diff line number Diff line change
Expand Up @@ -3,12 +3,6 @@
const { exec } = require('child_process');

describe('Import Map Sample Project', function() {
beforeEach(function() {
if (process.env.JASMINE_NO_BROWSER_TESTS) {
pending('skipping because the JASMINE_NO_BROWSER_TESTS env var is set');
}
});

it(
'executes example specs',
function(done) {
Expand Down
6 changes: 0 additions & 6 deletions spec/randomizationIntegrationSpec.js
Original file line number Diff line number Diff line change
@@ -1,12 +1,6 @@
const { exec } = require('child_process');

describe('Randomization integration', function() {
beforeEach(function() {
if (process.env.JASMINE_NO_BROWSER_TESTS) {
pending('skipping because the JASMINE_NO_BROWSER_TESTS env var is set');
}
});

it(
'reports the command to reproduce the random seed',
async function() {
Expand Down
6 changes: 0 additions & 6 deletions spec/reporterIntegrationSpec.js
Original file line number Diff line number Diff line change
@@ -1,12 +1,6 @@
const { exec } = require('child_process');

describe('Reporter integration', function() {
beforeEach(function() {
if (process.env.JASMINE_NO_BROWSER_TESTS) {
pending('skipping because the JASMINE_NO_BROWSER_TESTS env var is set');
}
});

it(
'evaluates relative reporter paths in config files relative to the CWD',
function(done) {
Expand Down

0 comments on commit e284c49

Please sign in to comment.