SWI-5429 Update Test Suite and Workflows (#30)

* SWI-5429 Update Test Suite and Workflows

* update template

* remove matrix from smoke tests

.github/workflows/deploy.yml

@@ -51,8 +51,8 @@ jobs:
 
       - name: Install Packages and Test
         run: |
-          npm i
-          npm test
+          yarn
+          yarn test-unit
 
       - name: Publish npm Package
         run: |

.github/workflows/test-nightly.yml

@@ -1,4 +1,4 @@
-name: Test Main Branch Nightly
+name: Nightly Smoke Tests
 
 on:
   schedule:
@@ -26,13 +26,8 @@ env:
 
 jobs:
   test_main:
-    name: Test Main Branch Nightly
-    runs-on: ${{ matrix.os }}
-    strategy:
-      matrix:
-        os: [windows-2022, windows-2019, ubuntu-20.04, ubuntu-22.04]
-        node-version: [16, 18, 20, 22]
-      fail-fast: false
+    name: Nightly Smoke Test
+    runs-on: ubuntu-latest
     steps:
       - name: Checkout
         uses: actions/checkout@v4
@@ -42,14 +37,14 @@ jobs:
       - name: Setup Node
         uses: actions/setup-node@v4
         with:
-          node-version: ${{ matrix.node-version }}
+          node-version: 20
 
       - name: Install Packages and Test
         run: |
           yarn
-          yarn test
+          yarn test-smoke
         env:
-          OPERATING_SYSTEM: ${{ matrix.os }}
+          OPERATING_SYSTEM: "ubuntu"
 
   notify_for_failures:
       name: Notify for Failures

.github/workflows/test-pr.yml

@@ -46,6 +46,6 @@ jobs:
       - name: Install Packages and Test
         run: |
           yarn
-          yarn test
+          yarn test-unit
         env:
           OPERATING_SYSTEM: ${{ matrix.os }}

custom_templates/package.mustache

@@ -24,6 +24,8 @@
   "scripts": {
     "build": "tsc {{#supportsES6}}&& tsc -p tsconfig.esm.json{{/supportsES6}}",
     "prepare": "npm run build",
+    "test-unit": "jest --coverage --detectOpenHandles --testPathPattern=tests/unit",
+    "test-smoke": "jest --coverage --detectOpenHandles --testPathPattern=tests/smoke",
     "test": "jest --coverage --detectOpenHandles"
   },
   "dependencies": {

package.json

@@ -20,6 +20,8 @@
   "scripts": {
     "build": "tsc ",
     "prepare": "npm run build",
+    "test-unit": "jest --coverage --detectOpenHandles --testPathPattern=tests/unit",
+    "test-smoke": "jest --coverage --detectOpenHandles --testPathPattern=tests/smoke",
     "test": "jest --coverage --detectOpenHandles"
   },
   "dependencies": {

tests/unit/bxml/Bxml.test.ts → tests/unit/models/bxml/Bxml.test.ts

@@ -1,6 +1,6 @@
-import { Bxml } from '../../../models/bxml/Bxml';
-import { PauseRecording } from '../../../models/bxml/verbs/PauseRecording';
-import { Root } from '../../../models/bxml/Root';
+import { Bxml } from '../../../../models/bxml/Bxml';
+import { PauseRecording } from '../../../../models/bxml/verbs/PauseRecording';
+import { Root } from '../../../../models/bxml/Root';
 
 describe('Bxml', () => {
     const pauseRecording = new PauseRecording();

tests/unit/bxml/NestableVerb.test.ts → tests/unit/models/bxml/NestableVerb.test.ts

@@ -1,5 +1,5 @@
-import { Verb } from '../../../models/bxml/Verb';
-import { NestableVerb } from '../../../models/bxml/NestableVerb';
+import { Verb } from '../../../../models/bxml/Verb';
+import { NestableVerb } from '../../../../models/bxml/NestableVerb';
 
 describe('NestableVerb', () => {
     test('should create a nestable verb', () => {

tests/unit/bxml/Response.test.ts → tests/unit/models/bxml/Response.test.ts

@@ -1,6 +1,6 @@
-import { Response } from '../../../models/bxml/Response';
-import { PauseRecording } from '../../../models/bxml/verbs/PauseRecording';
-import { Root } from '../../../models/bxml/Root';
+import { Response } from '../../../../models/bxml/Response';
+import { PauseRecording } from '../../../../models/bxml/verbs/PauseRecording';
+import { Root } from '../../../../models/bxml/Root';
 
 describe('Response', () => {
     const pauseRecording = new PauseRecording();

tests/unit/bxml/Verb.test.ts → tests/unit/models/bxml/Verb.test.ts

@@ -1,4 +1,4 @@
-import { Verb } from '../../../models/bxml/Verb';
+import { Verb } from '../../../../models/bxml/Verb';
 
 describe('Verb', () => {
     test('should create a verb', () => {

tests/unit/bxml/verbs/Bridge.test.ts → tests/unit/models/bxml/verbs/Bridge.test.ts

@@ -1,5 +1,5 @@
-import { Verb } from '../../../../models/bxml/Verb';
-import { Bridge, BridgeAttributes } from '../../../../models/bxml/verbs/Bridge';
+import { Verb } from '../../../../../models/bxml/Verb';
+import { Bridge, BridgeAttributes } from '../../../../../models/bxml/verbs/Bridge';
 
 describe('Bridge', () => {
     const attributes: BridgeAttributes = {

tests/unit/bxml/verbs/Conference.test.ts → tests/unit/models/bxml/verbs/Conference.test.ts

@@ -1,5 +1,5 @@
-import { Verb } from '../../../../models/bxml/Verb';
-import { Conference, ConferenceAttributes } from '../../../../models/bxml/verbs/Conference';
+import { Verb } from '../../../../../models/bxml/Verb';
+import { Conference, ConferenceAttributes } from '../../../../../models/bxml/verbs/Conference';
 
 describe('Conference', () => {
     const attributes: ConferenceAttributes = {

tests/unit/bxml/verbs/CustomParam.test.ts → tests/unit/models/bxml/verbs/CustomParam.test.ts

@@ -1,5 +1,5 @@
-import { Verb } from '../../../../models/bxml/Verb';
-import { CustomParam, CustomParamAttributes } from '../../../../models/bxml/verbs/CustomParam';
+import { Verb } from '../../../../../models/bxml/Verb';
+import { CustomParam, CustomParamAttributes } from '../../../../../models/bxml/verbs/CustomParam';
 
 describe('CustomParam', () => {
     const attributes: CustomParamAttributes = {

tests/unit/bxml/verbs/Forward.test.ts → tests/unit/models/bxml/verbs/Forward.test.ts

@@ -1,5 +1,5 @@
-import { Verb } from '../../../../models/bxml/Verb';
-import { Forward, ForwardAttributes} from '../../../../models/bxml/verbs/Forward';
+import { Verb } from '../../../../../models/bxml/Verb';
+import { Forward, ForwardAttributes} from '../../../../../models/bxml/verbs/Forward';
 
 describe('Forward', () => {
     const attributes: ForwardAttributes = {

tests/unit/bxml/verbs/Gather.test.ts → tests/unit/models/bxml/verbs/Gather.test.ts

@@ -1,7 +1,7 @@
-import { Verb } from '../../../../models/bxml/Verb';
-import { Gather, GatherAttributes } from '../../../../models/bxml/verbs/Gather';
-import { PlayAudio } from '../../../../models/bxml/verbs/PlayAudio';
-import { SpeakSentence } from '../../../../models/bxml/verbs/SpeakSentence';
+import { Verb } from '../../../../../models/bxml/Verb';
+import { Gather, GatherAttributes } from '../../../../../models/bxml/verbs/Gather';
+import { PlayAudio } from '../../../../../models/bxml/verbs/PlayAudio';
+import { SpeakSentence } from '../../../../../models/bxml/verbs/SpeakSentence';
 
 describe('Gather', () => {
     const attributes: GatherAttributes = {

tests/unit/bxml/verbs/Hangup.test.ts → tests/unit/models/bxml/verbs/Hangup.test.ts

@@ -1,5 +1,5 @@
-import { Verb } from '../../../../models/bxml/Verb';
-import { Hangup } from '../../../../models/bxml/verbs/Hangup';
+import { Verb } from '../../../../../models/bxml/Verb';
+import { Hangup } from '../../../../../models/bxml/verbs/Hangup';
 
 describe('Hangup', () => {
     const expected = '<Hangup/>';

tests/unit/bxml/verbs/Pause.test.ts → tests/unit/models/bxml/verbs/Pause.test.ts

@@ -1,5 +1,5 @@
-import { Verb } from '../../../../models/bxml/Verb';
-import { Pause, PauseAttributes } from '../../../../models/bxml/verbs/Pause';
+import { Verb } from '../../../../../models/bxml/Verb';
+import { Pause, PauseAttributes } from '../../../../../models/bxml/verbs/Pause';
 
 describe('Pause', () => {
     const attributes: PauseAttributes = {

tests/unit/bxml/verbs/PauseRecording.test.ts → tests/unit/models/bxml/verbs/PauseRecording.test.ts

@@ -1,5 +1,5 @@
-import { Verb } from '../../../../models/bxml/Verb';
-import { PauseRecording } from '../../../../models/bxml/verbs/PauseRecording';
+import { Verb } from '../../../../../models/bxml/Verb';
+import { PauseRecording } from '../../../../../models/bxml/verbs/PauseRecording';
 
 describe('PauseRecording', () => {
     const expected = '<PauseRecording/>';

tests/unit/bxml/verbs/PhoneNumber.test.ts → tests/unit/models/bxml/verbs/PhoneNumber.test.ts

@@ -1,5 +1,5 @@
-import { Verb } from '../../../../models/bxml/Verb';
-import { PhoneNumber, PhoneNumberAttributes } from '../../../../models/bxml/verbs/PhoneNumber';
+import { Verb } from '../../../../../models/bxml/Verb';
+import { PhoneNumber, PhoneNumberAttributes } from '../../../../../models/bxml/verbs/PhoneNumber';
 
 describe('PhoneNumber', () => {
     const attributes: PhoneNumberAttributes = {

tests/unit/bxml/verbs/PlayAudio.test.ts → tests/unit/models/bxml/verbs/PlayAudio.test.ts

@@ -1,5 +1,5 @@
-import { Verb } from '../../../../models/bxml/Verb';
-import { PlayAudio, PlayAudioAttributes } from '../../../../models/bxml/verbs/PlayAudio';
+import { Verb } from '../../../../../models/bxml/Verb';
+import { PlayAudio, PlayAudioAttributes } from '../../../../../models/bxml/verbs/PlayAudio';
 
 describe('PlayAudio', () => {
     const attributes: PlayAudioAttributes = {

tests/unit/bxml/verbs/Record.test.ts → tests/unit/models/bxml/verbs/Record.test.ts

@@ -1,5 +1,5 @@
-import { Verb } from '../../../../models/bxml/Verb';
-import { Record, RecordAttributes } from '../../../../models/bxml/verbs/Record';
+import { Verb } from '../../../../../models/bxml/Verb';
+import { Record, RecordAttributes } from '../../../../../models/bxml/verbs/Record';
 
 describe('Record', () => {
     const attributes: RecordAttributes = {

tests/unit/bxml/verbs/Redirect.test.ts → tests/unit/models/bxml/verbs/Redirect.test.ts

@@ -1,5 +1,5 @@
-import { Verb } from '../../../../models/bxml/Verb';
-import { Redirect, RedirectAttributes } from '../../../../models/bxml/verbs/Redirect';
+import { Verb } from '../../../../../models/bxml/Verb';
+import { Redirect, RedirectAttributes } from '../../../../../models/bxml/verbs/Redirect';
 
 describe('Redirect', () => {
     const attributes: RedirectAttributes = {

tests/unit/bxml/verbs/ResumeRecording.test.ts → tests/unit/models/bxml/verbs/ResumeRecording.test.ts

@@ -1,5 +1,5 @@
-import { Verb } from '../../../../models/bxml/Verb';
-import { ResumeRecording } from '../../../../models/bxml/verbs/ResumeRecording';
+import { Verb } from '../../../../../models/bxml/Verb';
+import { ResumeRecording } from '../../../../../models/bxml/verbs/ResumeRecording';
 
 describe('ResumeRecording', () => {
     const expected = '<ResumeRecording/>';

tests/unit/bxml/verbs/Ring.test.ts → tests/unit/models/bxml/verbs/Ring.test.ts

@@ -1,5 +1,5 @@
-import { Verb } from '../../../../models/bxml/Verb';
-import { Ring, RingAttributes } from '../../../../models/bxml/verbs/Ring';
+import { Verb } from '../../../../../models/bxml/Verb';
+import { Ring, RingAttributes } from '../../../../../models/bxml/verbs/Ring';
 
 describe('Ring', () => {
     const attributes: RingAttributes = {

tests/unit/bxml/verbs/SendDtmf.test.ts → tests/unit/models/bxml/verbs/SendDtmf.test.ts

@@ -1,5 +1,5 @@
-import { Verb } from '../../../../models/bxml/Verb';
-import { SendDtmf, SendDtmfAttributes } from '../../../../models/bxml/verbs/SendDtmf';
+import { Verb } from '../../../../../models/bxml/Verb';
+import { SendDtmf, SendDtmfAttributes } from '../../../../../models/bxml/verbs/SendDtmf';
 
 describe('SendDtmf', () => {
     const attributes: SendDtmfAttributes = {

tests/unit/bxml/verbs/SipUri.test.ts → tests/unit/models/bxml/verbs/SipUri.test.ts

@@ -1,5 +1,5 @@
-import { Verb } from '../../../../models/bxml/Verb';
-import { SipUri, SipUriAttributes } from '../../../../models/bxml/verbs/SipUri';
+import { Verb } from '../../../../../models/bxml/Verb';
+import { SipUri, SipUriAttributes } from '../../../../../models/bxml/verbs/SipUri';
 
 describe('SipUri', () => {
     const attributes: SipUriAttributes = {

tests/unit/bxml/verbs/SpeakSentence.test.ts → tests/unit/models/bxml/verbs/SpeakSentence.test.ts

@@ -1,5 +1,5 @@
-import { Verb } from '../../../../models/bxml/Verb';
-import { SpeakSentence, SpeakSentenceAttributes } from '../../../../models/bxml/verbs/SpeakSentence';
+import { Verb } from '../../../../../models/bxml/Verb';
+import { SpeakSentence, SpeakSentenceAttributes } from '../../../../../models/bxml/verbs/SpeakSentence';
 
 describe('SpeakSentence', () => {
     const attributes: SpeakSentenceAttributes = {

tests/unit/bxml/verbs/StartGather.test.ts → tests/unit/models/bxml/verbs/StartGather.test.ts

@@ -1,5 +1,5 @@
-import { Verb } from '../../../../models/bxml/Verb';
-import { StartGather, StartGatherAttributes } from '../../../../models/bxml/verbs/StartGather';
+import { Verb } from '../../../../../models/bxml/Verb';
+import { StartGather, StartGatherAttributes } from '../../../../../models/bxml/verbs/StartGather';
 
 describe('CustomParam', () => {
     const attributes: StartGatherAttributes = {

tests/unit/bxml/verbs/StartRecording.test.ts → tests/unit/models/bxml/verbs/StartRecording.test.ts

@@ -1,5 +1,5 @@
-import { Verb } from '../../../../models/bxml/Verb';
-import { StartRecording, StartRecordingAttributes } from '../../../../models/bxml/verbs/StartRecording';
+import { Verb } from '../../../../../models/bxml/Verb';
+import { StartRecording, StartRecordingAttributes } from '../../../../../models/bxml/verbs/StartRecording';
 
 describe('StartRecording', () => {
     const attributes: StartRecordingAttributes = {

tests/unit/bxml/verbs/StartStream.test.ts → tests/unit/models/bxml/verbs/StartStream.test.ts

@@ -1,6 +1,6 @@
-import { Verb } from '../../../../models/bxml/Verb';
-import { StartStream, StartStreamAttributes } from '../../../../models/bxml/verbs/StartStream';
-import { StreamParam } from '../../../../models/bxml/verbs/StreamParam';
+import { Verb } from '../../../../../models/bxml/Verb';
+import { StartStream, StartStreamAttributes } from '../../../../../models/bxml/verbs/StartStream';
+import { StreamParam } from '../../../../../models/bxml/verbs/StreamParam';
 
 describe('StartStream', () => {
     const attributes: StartStreamAttributes = {

tests/unit/bxml/verbs/StartTranscription.test.ts → tests/unit/models/bxml/verbs/StartTranscription.test.ts

@@ -1,6 +1,6 @@
-import { Verb } from '../../../../models/bxml/Verb';
-import { StartTranscription, StartTranscriptionAttributes } from '../../../../models/bxml/verbs/StartTranscription';
-import { CustomParam } from '../../../../models/bxml/verbs/CustomParam';
+import { Verb } from '../../../../../models/bxml/Verb';
+import { StartTranscription, StartTranscriptionAttributes } from '../../../../../models/bxml/verbs/StartTranscription';
+import { CustomParam } from '../../../../../models/bxml/verbs/CustomParam';
 
 describe('StartTranscription', () => {
     const attributes: StartTranscriptionAttributes = {

tests/unit/bxml/verbs/StopGather.test.ts → tests/unit/models/bxml/verbs/StopGather.test.ts

@@ -1,5 +1,5 @@
-import { Verb } from '../../../../models/bxml/Verb';
-import { StopGather } from '../../../../models/bxml/verbs/StopGather';
+import { Verb } from '../../../../../models/bxml/Verb';
+import { StopGather } from '../../../../../models/bxml/verbs/StopGather';
 
 describe('StopGather', () => {
     const expected = '<StopGather/>';

tests/unit/bxml/verbs/StopRecording.test.ts → tests/unit/models/bxml/verbs/StopRecording.test.ts

@@ -1,5 +1,5 @@
-import { Verb } from '../../../../models/bxml/Verb';
-import { StopRecording } from '../../../../models/bxml/verbs/StopRecording';
+import { Verb } from '../../../../../models/bxml/Verb';
+import { StopRecording } from '../../../../../models/bxml/verbs/StopRecording';
 
 describe('StopRecording', () => {
     const expected = '<StopRecording/>';

tests/unit/bxml/verbs/StopStream.test.ts → tests/unit/models/bxml/verbs/StopStream.test.ts

@@ -1,5 +1,5 @@
-import { Verb } from '../../../../models/bxml/Verb';
-import { StopStream, StopStreamAttributes } from '../../../../models/bxml/verbs/StopStream';
+import { Verb } from '../../../../../models/bxml/Verb';
+import { StopStream, StopStreamAttributes } from '../../../../../models/bxml/verbs/StopStream';
 
 describe('StopStream', () => {
     const attributes: StopStreamAttributes = {

tests/unit/bxml/verbs/StopTranscription.test.ts → tests/unit/models/bxml/verbs/StopTranscription.test.ts

@@ -1,5 +1,5 @@
-import { Verb } from '../../../../models/bxml/Verb';
-import { StopTranscription, StopTranscriptionAttributes } from '../../../../models/bxml/verbs/StopTranscription';
+import { Verb } from '../../../../../models/bxml/Verb';
+import { StopTranscription, StopTranscriptionAttributes } from '../../../../../models/bxml/verbs/StopTranscription';
 
 describe('StopTranscription', () => {
     const attributes: StopTranscriptionAttributes = {

tests/unit/bxml/verbs/StreamParam.test.ts → tests/unit/models/bxml/verbs/StreamParam.test.ts

@@ -1,5 +1,5 @@
-import { Verb } from '../../../../models/bxml/Verb';
-import { StreamParam, StreamParamAttributes } from '../../../../models/bxml/verbs/StreamParam';
+import { Verb } from '../../../../../models/bxml/Verb';
+import { StreamParam, StreamParamAttributes } from '../../../../../models/bxml/verbs/StreamParam';
 
 describe('StreamParam', () => {
     const attributes: StreamParamAttributes = {

tests/unit/bxml/verbs/Tag.test.ts → tests/unit/models/bxml/verbs/Tag.test.ts

@@ -1,5 +1,5 @@
-import { Verb } from '../../../../models/bxml/Verb';
-import { Tag } from '../../../../models/bxml/verbs/Tag';
+import { Verb } from '../../../../../models/bxml/Verb';
+import { Tag } from '../../../../../models/bxml/verbs/Tag';
 
 describe('Tag', () => {
     const expected = '<Tag>testTag</Tag>';

tests/unit/bxml/verbs/Transfer.test.ts → tests/unit/models/bxml/verbs/Transfer.test.ts

@@ -1,7 +1,7 @@
-import { Verb } from '../../../../models/bxml/Verb';
-import { PhoneNumber } from '../../../../models/bxml/verbs/PhoneNumber';
-import { SipUri } from '../../../../models/bxml/verbs/SipUri';
-import { Transfer, TransferAttributes } from '../../../../models/bxml/verbs/Transfer';
+import { Verb } from '../../../../../models/bxml/Verb';
+import { PhoneNumber } from '../../../../../models/bxml/verbs/PhoneNumber';
+import { SipUri } from '../../../../../models/bxml/verbs/SipUri';
+import { Transfer, TransferAttributes } from '../../../../../models/bxml/verbs/Transfer';
 
 describe('Transfer', () => {
     const attributes: TransferAttributes = {

tests/unit/call-direction-enum.test.ts → tests/unit/models/call-direction-enum.test.ts

@@ -1,4 +1,4 @@
-import { CallDirectionEnum } from '../../models/call-direction-enum';
+import { CallDirectionEnum } from '../../../models/call-direction-enum';
 
 describe('CallDirectionEnum', () => {
     test('should define the expected values', () => {

tests/unit/call-state-enum.test.ts → tests/unit/models/call-state-enum.test.ts

@@ -1,4 +1,4 @@
-import { CallStateEnum } from '../../models/call-state-enum';
+import { CallStateEnum } from '../../../models/call-state-enum';
 
 describe('CallStateEnum', () => {
     test('should define the expected values', () => {

tests/unit/callback-method-enum.test.ts → tests/unit/models/callback-method-enum.test.ts

@@ -1,4 +1,4 @@
-import { CallbackMethodEnum } from '../../models/callback-method-enum';
+import { CallbackMethodEnum } from '../../../models/callback-method-enum';
 
 describe('CallbackMethodEnum', () => {
     test('should define the expected values', () => {

tests/unit/conference-state-enum.test.ts → tests/unit/models/conference-state-enum.test.ts

@@ -1,4 +1,4 @@
-import { ConferenceStateEnum } from '../../models/conference-state-enum';
+import { ConferenceStateEnum } from '../../../models/conference-state-enum';
 
 describe('ConferenceStateEnum', () => {
     test('should define the expected values', () => {

tests/unit/file-format-enum.test.ts → tests/unit/models/file-format-enum.test.ts

@@ -1,4 +1,4 @@
-import { FileFormatEnum } from '../../models/file-format-enum';
+import { FileFormatEnum } from '../../../models/file-format-enum';
 
 describe('FileFormatEnum', () => {
     test('should define the expected values', () => {

tests/unit/list-message-direction-enum.test.ts → tests/unit/models/list-message-direction-enum.test.ts

@@ -1,4 +1,4 @@
-import { ListMessageDirectionEnum } from '../../models/list-message-direction-enum';
+import { ListMessageDirectionEnum } from '../../../models/list-message-direction-enum';
 
 describe('ListMessageDirectionEnum', () => {
     test('should define the expected values', () => {

tests/unit/lookup-status-enum.test.ts → tests/unit/models/lookup-status-enum.test.ts

@@ -1,4 +1,4 @@
-import { LookupStatusEnum } from '../../models/lookup-status-enum';
+import { LookupStatusEnum } from '../../../models/lookup-status-enum';
 
 describe('LookupStatusEnum', () => {
     test('should define the expected values', () => {

tests/unit/machine-detection-mode-enum.test.ts → tests/unit/models/machine-detection-mode-enum.test.ts

@@ -1,4 +1,4 @@
-import { MachineDetectionModeEnum } from '../../models/machine-detection-mode-enum';
+import { MachineDetectionModeEnum } from '../../../models/machine-detection-mode-enum';
 
 describe('MachineDetectionModeEnum', () => {
     test('should define the expected values', () => {