Skip to content

Commit 678f7da

Browse files
seydxseydx
committed
Remove skipInCI from hardware tests
1 parent 74aae47 commit 678f7da

File tree

4 files changed

+37
-37
lines changed

4 files changed

+37
-37
lines changed

‎test/encoder.test.ts‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ import { describe, it } from 'node:test';
33

44
import { AV_PIX_FMT_YUV420P, AV_SAMPLE_FMT_FLTP, Encoder, FF_ENCODER_AAC, FF_ENCODER_LIBX264, HardwareContext } from '../src/index.js';
55
import { Frame, Rational } from '../src/lib/index.js';
6-
import { encodeFrame, encodeFrameSync, skipInCI } from './index.js';
6+
import { encodeFrame, encodeFrameSync } from './index.js';
77

88
describe('Encoder', () => {
99
describe('create', () => {
@@ -593,7 +593,7 @@ describe('Encoder', () => {
593593
});
594594

595595
describe('hardware encoding', () => {
596-
it('should create hardware encoder with hardware context', skipInCI, async () => {
596+
it('should create hardware encoder with hardware context', async () => {
597597
// Try to get hardware context
598598
const hw = HardwareContext.auto();
599599
if (!hw) {

‎test/hardware.test.ts‎

Lines changed: 27 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ import {
1818
HardwareContext,
1919
type AVHWDeviceType,
2020
} from '../src/index.js';
21-
import { decodeFirstFrame, decodePacket, decodePacketSync, encodeFrame, encodeFrameSync, getInputFile, prepareTestEnvironment, skipInCI } from './index.js';
21+
import { decodeFirstFrame, decodePacket, decodePacketSync, encodeFrame, encodeFrameSync, getInputFile, prepareTestEnvironment } from './index.js';
2222

2323
prepareTestEnvironment();
2424

@@ -32,7 +32,7 @@ describe('HardwareContext', () => {
3232
console.log('Available hardware:', available.join(', ') || 'none');
3333
});
3434

35-
it('should auto-detect hardware', skipInCI, async () => {
35+
it('should auto-detect hardware', async () => {
3636
const hw = HardwareContext.auto();
3737
if (hw) {
3838
console.log(`Auto-detected hardware: ${hw.deviceTypeName}`);
@@ -72,7 +72,7 @@ describe('HardwareContext', () => {
7272
});
7373

7474
describe('instance methods', () => {
75-
it('should provide device information', skipInCI, () => {
75+
it('should provide device information', () => {
7676
const hw = HardwareContext.auto();
7777
if (hw) {
7878
assert.ok(hw.deviceTypeName, 'Should have device type name');
@@ -82,7 +82,7 @@ describe('HardwareContext', () => {
8282
}
8383
});
8484

85-
it('should get encoder codec for base codec name', skipInCI, async () => {
85+
it('should get encoder codec for base codec name', async () => {
8686
const hw = HardwareContext.auto();
8787
if (hw) {
8888
// Test getting hardware encoder codec
@@ -102,7 +102,7 @@ describe('HardwareContext', () => {
102102
}
103103
});
104104

105-
it('should support Symbol.dispose', skipInCI, () => {
105+
it('should support Symbol.dispose', () => {
106106
const hw = HardwareContext.auto();
107107
if (hw) {
108108
{
@@ -114,7 +114,7 @@ describe('HardwareContext', () => {
114114
}
115115
});
116116

117-
it('should check if hardware is disposed', skipInCI, () => {
117+
it('should check if hardware is disposed', () => {
118118
const hw = HardwareContext.auto();
119119
if (hw) {
120120
assert.equal(hw.isDisposed, false, 'Should not be disposed initially');
@@ -124,7 +124,7 @@ describe('HardwareContext', () => {
124124
});
125125
});
126126

127-
describe('specific hardware types', skipInCI, () => {
127+
describe('specific hardware types', () => {
128128
it('should handle CUDA if available', () => {
129129
const cuda = HardwareContext.create(AV_HWDEVICE_TYPE_CUDA);
130130
if (cuda) {
@@ -209,7 +209,7 @@ describe('HardwareContext', () => {
209209
});
210210
});
211211

212-
describe('hardware validation', skipInCI, () => {
212+
describe('hardware validation', () => {
213213
it('should validate decoder with testDecoder()', () => {
214214
const hw = HardwareContext.auto();
215215
if (!hw) {
@@ -344,7 +344,7 @@ describe('HardwareContext', () => {
344344
});
345345

346346
describe('hardware disposal', () => {
347-
it('should safely dispose hardware multiple times', skipInCI, () => {
347+
it('should safely dispose hardware multiple times', () => {
348348
const hw = HardwareContext.auto();
349349
if (!hw) {
350350
console.log('No hardware available - skipping test');
@@ -364,7 +364,7 @@ describe('HardwareContext', () => {
364364
assert.ok(hw.isDisposed, 'Should still be disposed');
365365
});
366366

367-
it('should dispose hardware when decoder closes', skipInCI, async () => {
367+
it('should dispose hardware when decoder closes', async () => {
368368
const hw = HardwareContext.auto();
369369
if (!hw) {
370370
console.log('No hardware available - skipping test');
@@ -391,7 +391,7 @@ describe('HardwareContext', () => {
391391
media.close();
392392
});
393393

394-
it('should dispose hardware when encoder closes', skipInCI, async () => {
394+
it('should dispose hardware when encoder closes', async () => {
395395
const hw = HardwareContext.auto();
396396
if (!hw) {
397397
console.log('No hardware available - skipping test');
@@ -418,7 +418,7 @@ describe('HardwareContext', () => {
418418
}
419419
});
420420

421-
it('should allow sharing hardware between multiple decoders', skipInCI, async () => {
421+
it('should allow sharing hardware between multiple decoders', async () => {
422422
const hw = HardwareContext.auto();
423423
if (!hw) {
424424
console.log('No hardware available - skipping test');
@@ -457,7 +457,7 @@ describe('HardwareContext', () => {
457457
});
458458

459459
describe('hardware integration', () => {
460-
it('should work with Decoder when hardware is available (async)', skipInCI, async () => {
460+
it('should work with Decoder when hardware is available (async)', async () => {
461461
// Try to get hardware context
462462
const hw = HardwareContext.auto();
463463

@@ -491,7 +491,7 @@ describe('HardwareContext', () => {
491491
}
492492
});
493493

494-
it('should work with Decoder when hardware is available (sync)', skipInCI, async () => {
494+
it('should work with Decoder when hardware is available (sync)', async () => {
495495
// Try to get hardware context
496496
const hw = HardwareContext.auto();
497497

@@ -525,7 +525,7 @@ describe('HardwareContext', () => {
525525
}
526526
});
527527

528-
it('should work with Decoder using auto hardware detection (async)', skipInCI, async () => {
528+
it('should work with Decoder using auto hardware detection (async)', async () => {
529529
const media = await Demuxer.open(inputFile);
530530
const videoStream = media.video(0);
531531
assert.ok(videoStream, 'Should have video stream');
@@ -544,7 +544,7 @@ describe('HardwareContext', () => {
544544
// hw is disposed by decoder.close()
545545
});
546546

547-
it('should work with Decoder using auto hardware detection (sync)', skipInCI, async () => {
547+
it('should work with Decoder using auto hardware detection (sync)', async () => {
548548
const media = await Demuxer.open(inputFile);
549549
const videoStream = media.video(0);
550550
assert.ok(videoStream, 'Should have video stream');
@@ -574,7 +574,7 @@ describe('HardwareContext', () => {
574574
// hw is disposed by decoder.close()
575575
});
576576

577-
it('should work with Encoder when hardware is available', skipInCI, async () => {
577+
it('should work with Encoder when hardware is available', async () => {
578578
const hw = HardwareContext.auto();
579579

580580
if (!hw) {
@@ -599,7 +599,7 @@ describe('HardwareContext', () => {
599599
hw.dispose();
600600
});
601601

602-
it('should work with Encoder using auto hardware detection', skipInCI, async () => {
602+
it('should work with Encoder using auto hardware detection', async () => {
603603
const hw = HardwareContext.auto();
604604
if (!hw) {
605605
return;
@@ -630,7 +630,7 @@ describe('HardwareContext', () => {
630630
});
631631

632632
describe('hardware derivation', () => {
633-
it('should derive hardware context from another (with AVHWDeviceType)', skipInCI, () => {
633+
it('should derive hardware context from another (with AVHWDeviceType)', () => {
634634
const hw = HardwareContext.auto();
635635
if (!hw) {
636636
console.log('No hardware available - skipping derivation test');
@@ -661,7 +661,7 @@ describe('HardwareContext', () => {
661661
hw.dispose();
662662
});
663663

664-
it('should return null for invalid derivation', skipInCI, () => {
664+
it('should return null for invalid derivation', () => {
665665
const hw = HardwareContext.auto();
666666
if (!hw) {
667667
console.log('No hardware available - skipping test');
@@ -679,7 +679,7 @@ describe('HardwareContext', () => {
679679
hw.dispose();
680680
});
681681

682-
it('should return null when source is disposed', skipInCI, () => {
682+
it('should return null when source is disposed', () => {
683683
const hw = HardwareContext.auto();
684684
if (!hw) {
685685
console.log('No hardware available - skipping test');
@@ -693,7 +693,7 @@ describe('HardwareContext', () => {
693693
assert.equal(derived, null, 'Should return null when source is disposed');
694694
});
695695

696-
it('should test common derivation paths', skipInCI, () => {
696+
it('should test common derivation paths', () => {
697697
// Common derivation paths:
698698
// - VAAPI -> OpenCL
699699
// - CUDA -> Vulkan
@@ -730,7 +730,7 @@ describe('HardwareContext', () => {
730730
hw.dispose();
731731
});
732732

733-
it('should work with derived context in decoder', skipInCI, async () => {
733+
it('should work with derived context in decoder', async () => {
734734
const hw = HardwareContext.auto();
735735
if (!hw) {
736736
console.log('No hardware available - skipping test');
@@ -776,7 +776,7 @@ describe('HardwareContext', () => {
776776
});
777777

778778
describe('zero-copy GPU transfer', () => {
779-
it('should transfer frames from decoder to encoder on GPU (zero-copy) (async)', skipInCI, async () => {
779+
it('should transfer frames from decoder to encoder on GPU (zero-copy) (async)', async () => {
780780
// This test demonstrates zero-copy GPU frame transfer
781781
// where frames stay on GPU between decode and encode
782782

@@ -871,7 +871,7 @@ describe('HardwareContext', () => {
871871
}
872872
});
873873

874-
it('should transfer frames from decoder to encoder on GPU (zero-copy) (sync)', skipInCI, async () => {
874+
it('should transfer frames from decoder to encoder on GPU (zero-copy) (sync)', async () => {
875875
// This test demonstrates zero-copy GPU frame transfer with sync methods
876876
// where frames stay on GPU between decode and encode
877877

@@ -966,7 +966,7 @@ describe('HardwareContext', () => {
966966
}
967967
});
968968

969-
it('should demonstrate GPU memory efficiency with multiple streams', skipInCI, async () => {
969+
it('should demonstrate GPU memory efficiency with multiple streams', async () => {
970970
const hw = HardwareContext.auto();
971971
if (!hw) {
972972
console.log('No hardware acceleration available - skipping test');
@@ -1013,7 +1013,7 @@ describe('HardwareContext', () => {
10131013
}
10141014
});
10151015

1016-
it('should verify hardware codec selection', skipInCI, async () => {
1016+
it('should verify hardware codec selection', async () => {
10171017
const hw = HardwareContext.auto();
10181018
if (!hw) {
10191019
console.log('No hardware acceleration available - skipping test');

‎test/pipeline.test.ts‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -861,7 +861,7 @@ describe('Pipeline - Comprehensive Tests', () => {
861861
});
862862
});
863863

864-
describe('Hardware Acceleration', skipInCI, () => {
864+
describe('Hardware Acceleration', () => {
865865
it('should support hardware decoder in pipeline', async function (t) {
866866
// Check if hardware acceleration is available
867867
const hw = HardwareContext.auto();

‎test/transcode.test.ts‎

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ import assert from 'node:assert';
22
import { describe, it } from 'node:test';
33

44
import { AV_PIX_FMT_NV12, AV_PIX_FMT_YUV420P, Decoder, Demuxer, Encoder, FF_ENCODER_LIBX264, FilterAPI, FilterPreset, HardwareContext } from '../src/index.js';
5-
import { decodePacket, encodeFrame, filterFrame, getInputFile, prepareTestEnvironment, skipInCI } from './index.js';
5+
import { decodePacket, encodeFrame, filterFrame, getInputFile, prepareTestEnvironment } from './index.js';
66

77
prepareTestEnvironment();
88

@@ -108,7 +108,7 @@ describe('Transcode Scenarios', () => {
108108
assert.ok(frameCount > 0, 'Should process at least one frame');
109109
});
110110

111-
it('should handle HW Decode -> HW Filter -> SW Encode', skipInCI, async () => {
111+
it('should handle HW Decode -> HW Filter -> SW Encode', async () => {
112112
using hw = HardwareContext.auto();
113113
if (!hw) {
114114
console.log('No hardware available - skipping test');
@@ -142,7 +142,7 @@ describe('Transcode Scenarios', () => {
142142
assert.ok(frameCount > 0, 'Should process at least one frame');
143143
});
144144

145-
it('should handle SW Decode -> HW Filter -> HW Encode', skipInCI, async () => {
145+
it('should handle SW Decode -> HW Filter -> HW Encode', async () => {
146146
using hw = HardwareContext.auto();
147147
if (!hw) {
148148
console.log('No hardware available - skipping test');
@@ -189,7 +189,7 @@ describe('Transcode Scenarios', () => {
189189
assert.ok(frameCount > 0, 'Should process at least one frame');
190190
});
191191

192-
it('should handle HW Decode -> HW Filter -> HW Encode', skipInCI, async () => {
192+
it('should handle HW Decode -> HW Filter -> HW Encode', async () => {
193193
using hw = HardwareContext.auto();
194194
if (!hw) {
195195
console.log('No hardware available - skipping test');
@@ -233,7 +233,7 @@ describe('Transcode Scenarios', () => {
233233
assert.ok(frameCount > 0, 'Should process at least one frame');
234234
});
235235

236-
it('should handle HW Decode -> HW Encode (no filter)', skipInCI, async () => {
236+
it('should handle HW Decode -> HW Encode (no filter)', async () => {
237237
using hw = HardwareContext.auto();
238238
if (!hw) {
239239
console.log('No hardware available - skipping test');
@@ -268,7 +268,7 @@ describe('Transcode Scenarios', () => {
268268
assert.ok(frameCount > 0, 'Should process at least one frame');
269269
});
270270

271-
it('should handle SW Decode -> SW/HW Filter -> HW Encode', skipInCI, async () => {
271+
it('should handle SW Decode -> SW/HW Filter -> HW Encode', async () => {
272272
using hw = HardwareContext.auto();
273273
if (!hw) {
274274
console.log('No hardware available - skipping test');
@@ -316,7 +316,7 @@ describe('Transcode Scenarios', () => {
316316
});
317317

318318
describe('Error Cases', () => {
319-
it('should handle software frame with hardware filter gracefully', skipInCI, async () => {
319+
it('should handle software frame with hardware filter gracefully', async () => {
320320
using hw = HardwareContext.auto();
321321
if (!hw) {
322322
console.log('No hardware available, skipping test');

0 commit comments

Comments
 (0)