Skip to content

Commit eca3889

Browse files
authored
fix: Ensure analytics desktop dll is copied to the CI pipeline and add new test to verify (#1803)
* fix: add in test for the DLL * Add in the DLL copy to the workflow * Fix formatting * Consolidate the internal namespace on the analytics_desktop file * Revert "fix: add in test for the DLL" This reverts commit ea27540. * Revert the last of the changes to test dll if is there
1 parent 52db94f commit eca3889

File tree

1 file changed

+5
-0
lines changed

1 file changed

+5
-0
lines changed

.github/workflows/integration_tests.yml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -356,6 +356,11 @@ jobs:
356356
--gha_build \
357357
--arch ${{ matrix.arch }} \
358358
${additional_flags[*]}
359+
- name: Copy Analytics DLL to test folders (Windows)
360+
if: startsWith(matrix.os, 'windows')
361+
shell: bash
362+
run: |
363+
find "testapps-desktop-${{ matrix.os }}-${{ matrix.arch }}-${{ matrix.ssl_variant }}" -name integration_test.exe -exec dirname {} \; | while read dir; do cp analytics/windows/analytics_win.dll "$dir/"; done
359364
- name: Upload Desktop Cmake
360365
uses: actions/upload-artifact@v4
361366
if: ${{ !cancelled() }}

0 commit comments

Comments
 (0)