Skip to content

Commit

Permalink
[worker] skip CI that needs a worker
Browse files Browse the repository at this point in the history
  • Loading branch information
clangenb committed Apr 10, 2024
1 parent 2c11c03 commit f21fad8
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 7 deletions.
2 changes: 1 addition & 1 deletion packages/worker-api/src/integriteeWorker.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ describe('worker', () => {
// skip it, as this requires a worker (and hence a node) to be running
// To my knowledge jest does not have an option to run skipped tests specifically, does it?
// Todo: add proper CI to test this too.
describe('needs worker and node running', () => {
describe.skip('needs worker and node running', () => {
describe('getWorkerPubKey', () => {
it('should return value', async () => {
const result = await worker.getShieldingKey();
Expand Down
7 changes: 1 addition & 6 deletions packages/worker-api/src/worker.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -3,22 +3,17 @@ import { cryptoWaitReady } from '@polkadot/util-crypto';
import { localDockerNetwork } from './testUtils/networks.js';
import { Worker } from './worker.js';
import WS from 'websocket';
import {KeyringPair} from "@polkadot/keyring/types";

const {w3cwebsocket: WebSocket} = WS;

describe('worker', () => {
const network = localDockerNetwork();
let keyring: Keyring;
let worker: Worker;
let alice: KeyringPair;
let charlie: KeyringPair;
beforeAll(async () => {
jest.setTimeout(90000);
await cryptoWaitReady();
keyring = new Keyring({type: 'sr25519'});
alice = keyring.addFromUri('//Alice', {name: 'Alice default'});
charlie = keyring.addFromUri('//Charlie', {name: 'Bob default'});

worker = new Worker(network.worker, {
keyring: keyring,
Expand All @@ -39,7 +34,7 @@ describe('worker', () => {
// skip it, as this requires a worker (and hence a node) to be running
// To my knowledge jest does not have an option to run skipped tests specifically, does it?
// Todo: add proper CI to test this too.
describe('needs worker and node running', () => {
describe.skip('needs worker and node running', () => {
describe('getWorkerPubKey', () => {
it('should return value', async () => {
const result = await worker.getShieldingKey();
Expand Down

0 comments on commit f21fad8

Please sign in to comment.