Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Run tests in qemu in GitHub action #79

Draft
wants to merge 56 commits into
base: main
Choose a base branch
from

Commits on Aug 18, 2024

  1. ci: Use pre-built DJGPP image in GitHub Actions Workflows, so DJGPP d…

    …oesn't have to be downloaded and installed in each pipeline
    volkertb committed Aug 18, 2024
    Configuration menu
    Copy the full SHA
    0f849f1 View commit details
    Browse the repository at this point in the history
  2. ci: Update temporarily allowlisted branch, to test the build-and-rele…

    …ase workflow (best to remove `branches` block before merging, to have this workflow triggered only on pushed tags)
    volkertb committed Aug 18, 2024
    Configuration menu
    Copy the full SHA
    7bbae20 View commit details
    Browse the repository at this point in the history
  3. ci: Update .github/workflows/02-pr-checks.yml

    Co-authored-by: Thomas Perl <[email protected]>
    volkertb and thp committed Aug 18, 2024
    Configuration menu
    Copy the full SHA
    301d8e4 View commit details
    Browse the repository at this point in the history
  4. ci: Update .github/workflows/01-build-and-release.yml

    Co-authored-by: Thomas Perl <[email protected]>
    volkertb and thp committed Aug 18, 2024
    Configuration menu
    Copy the full SHA
    7700fae View commit details
    Browse the repository at this point in the history
  5. Configuration menu
    Copy the full SHA
    d4eea95 View commit details
    Browse the repository at this point in the history
  6. ci: Move shellcheck of artifacts build script to separate lint job …

    …in PR Checks workflow, so the build container image does not have to include it
    volkertb committed Aug 18, 2024
    Configuration menu
    Copy the full SHA
    5a786c1 View commit details
    Browse the repository at this point in the history
  7. Configuration menu
    Copy the full SHA
    3b06b6d View commit details
    Browse the repository at this point in the history
  8. test: Try KVM test assuming root already, sudo isn't even present i…

    …n `ubuntu-latest` by default, apparently
    volkertb committed Aug 18, 2024
    Configuration menu
    Copy the full SHA
    9db52b4 View commit details
    Browse the repository at this point in the history
  9. Configuration menu
    Copy the full SHA
    7da81ec View commit details
    Browse the repository at this point in the history
  10. ci: Use Docker image again in GitHub Actions Workflow, and provide ac…

    …cess to /dev/kvm for KVM support
    volkertb committed Aug 18, 2024
    Configuration menu
    Copy the full SHA
    9b14604 View commit details
    Browse the repository at this point in the history
  11. Configuration menu
    Copy the full SHA
    743931d View commit details
    Browse the repository at this point in the history
  12. Configuration menu
    Copy the full SHA
    5a79db8 View commit details
    Browse the repository at this point in the history
  13. test: Trying something that should deliberately fail, to troubleshoot…

    … (verify that args are actually being passed on when running docker)
    volkertb committed Aug 18, 2024
    Configuration menu
    Copy the full SHA
    547403a View commit details
    Browse the repository at this point in the history
  14. ci: Try passing on Docker options in another way (although this way, …

    …I can't prevent the entrypoint from being overridden by the steps, apparently)
    volkertb committed Aug 18, 2024
    Configuration menu
    Copy the full SHA
    1405361 View commit details
    Browse the repository at this point in the history
  15. ci: In the step, run the same QEMU launch script that the default ent…

    …rypoint of the Docker image would run
    volkertb committed Aug 18, 2024
    Configuration menu
    Copy the full SHA
    ea902c8 View commit details
    Browse the repository at this point in the history
  16. test: Try running SBEMU in a DOS VM under KVM-accelerated QEMU in Git…

    …Hub Actions Workflow, as a sanity test (will extend this test further later)
    volkertb committed Aug 18, 2024
    Configuration menu
    Copy the full SHA
    6353642 View commit details
    Browse the repository at this point in the history
  17. ci: Check out repository before running shellcheck on build script, m…

    …ake test job dependent on build job, since build job needs to complete and upload SBEMU.EXE as artifact before test job can download and run it
    volkertb committed Aug 18, 2024
    Configuration menu
    Copy the full SHA
    179d604 View commit details
    Browse the repository at this point in the history
  18. Configuration menu
    Copy the full SHA
    b90a5a6 View commit details
    Browse the repository at this point in the history
  19. test: Add AIL/32 files from http://www.thegleam.com/ke5fx/misc/AIL2.ZIP

    …, DOS/32A DOS extender and DTMF tone test WAV file to test directory for testing actual sound output
    volkertb committed Aug 18, 2024
    Configuration menu
    Copy the full SHA
    e7dd2b5 View commit details
    Browse the repository at this point in the history
  20. test: Copy test dependencies along with built EXE as test artifacts, …

    …so the test job has access to all the necessary files for running the tests in the DOS VM
    volkertb committed Aug 18, 2024
    Configuration menu
    Copy the full SHA
    5b647fd View commit details
    Browse the repository at this point in the history
  21. Configuration menu
    Copy the full SHA
    863cf9b View commit details
    Browse the repository at this point in the history
  22. ci: Apparently, actions/upload-artifact@v4 and actions/download-artif…

    …act@v4 don't strip directory prefixes when files from multiple directories are uploaded.
    volkertb committed Aug 18, 2024
    Configuration menu
    Copy the full SHA
    56ab093 View commit details
    Browse the repository at this point in the history
  23. Configuration menu
    Copy the full SHA
    d0c2e26 View commit details
    Browse the repository at this point in the history
  24. Configuration menu
    Copy the full SHA
    2703fc2 View commit details
    Browse the repository at this point in the history
  25. ci: Update to newer version of cicd-qemu-dos-docker image, to fix `…

    …CICD_DOS.BAT not found` error and shut down DOS VM faster (without unnecessary disk spindown) at end of test
    volkertb committed Aug 18, 2024
    Configuration menu
    Copy the full SHA
    532e6f5 View commit details
    Browse the repository at this point in the history
  26. test: Replace default QEMU launch script of cicd-qemu-dos-docker im…

    …age with a project-specific one (with different emulated sound devices configured)
    volkertb committed Aug 18, 2024
    Configuration menu
    Copy the full SHA
    1612432 View commit details
    Browse the repository at this point in the history
  27. Configuration menu
    Copy the full SHA
    ca8d087 View commit details
    Browse the repository at this point in the history
  28. test: Show BLASTER environment variable (after having loaded SBEMU.EX…

    …E), for troubleshooting, since Sound Blaster Pro AIL/32 driver is not detecting SBEMU's emulated Sound Blaster Pro
    volkertb committed Aug 18, 2024
    Configuration menu
    Copy the full SHA
    9a4123a View commit details
    Browse the repository at this point in the history
  29. test: Override BLASTER setting to troubleshoot Sound Blaster Pro dete…

    …ction problem in AIL/32 Sound Blaster Pro driver
    volkertb committed Aug 18, 2024
    Configuration menu
    Copy the full SHA
    0c4fc59 View commit details
    Browse the repository at this point in the history
  30. test: Remove temporary BLASTER environment variable override, since t…

    …hat didn't resolve Sound Blaster Pro detection problem in AIL/32 Sound Blaster Pro driver
    volkertb committed Aug 18, 2024
    Configuration menu
    Copy the full SHA
    33f7ec8 View commit details
    Browse the repository at this point in the history
  31. test: Try the AIL/32 regular Sound Blster drivers instead of the AIL/…

    …32 Sound Blaster Pro driver, which isn't detected the emulated Sound Blaster Pro of SBEMU for some reason
    volkertb committed Aug 18, 2024
    Configuration menu
    Copy the full SHA
    ba3f9da View commit details
    Browse the repository at this point in the history
  32. test: Emulate older intel-hda (ich6) variant instead of ich9-intel-hd…

    …a, to see if that makes any difference, and list directory contents after completing DOS VM session
    volkertb committed Aug 18, 2024
    Configuration menu
    Copy the full SHA
    451c06b View commit details
    Browse the repository at this point in the history
  33. Configuration menu
    Copy the full SHA
    ba4a911 View commit details
    Browse the repository at this point in the history
  34. Configuration menu
    Copy the full SHA
    39d0e9f View commit details
    Browse the repository at this point in the history
  35. Configuration menu
    Copy the full SHA
    b18882d View commit details
    Browse the repository at this point in the history
  36. test: try capturing exit code of test command (`stp32 dtmf_src.wav a3…

    …2sbdg.dll`) in file and print it to console
    volkertb committed Aug 18, 2024
    Configuration menu
    Copy the full SHA
    9aa3703 View commit details
    Browse the repository at this point in the history
  37. Configuration menu
    Copy the full SHA
    ca3c0be View commit details
    Browse the repository at this point in the history
  38. test: Load Jemm with QPIEMU module to allow SBEMU to be tested in Rea…

    …l Mode as well as in Protected Mode
    volkertb committed Aug 18, 2024
    Configuration menu
    Copy the full SHA
    5384490 View commit details
    Browse the repository at this point in the history
  39. Configuration menu
    Copy the full SHA
    726fbd2 View commit details
    Browse the repository at this point in the history
  40. ci: try Intel HD Audio (HDA) instead of AC'97/ICHx, since the latter …

    …is apparently not working with SBEMU in the QEMU VM, at least not for the stp32 Protected Mode audio player
    volkertb committed Aug 18, 2024
    Configuration menu
    Copy the full SHA
    0e0000c View commit details
    Browse the repository at this point in the history
  41. Configuration menu
    Copy the full SHA
    4abdad9 View commit details
    Browse the repository at this point in the history
  42. Revert "test: Add workarounds for possible issue with DOS/32A DOS ext…

    …ender, see crazii#88"
    
    This reverts commit 4abdad9.
    volkertb committed Aug 18, 2024
    Configuration menu
    Copy the full SHA
    92bd299 View commit details
    Browse the repository at this point in the history
  43. Configuration menu
    Copy the full SHA
    f074ba2 View commit details
    Browse the repository at this point in the history
  44. Configuration menu
    Copy the full SHA
    64d0274 View commit details
    Browse the repository at this point in the history

Commits on Aug 19, 2024

  1. test: Download dtmf2num source and compile and install it before usin…

    …g it, since we are testing on an Alpine image, not an Ubuntu or Debian image
    volkertb committed Aug 19, 2024
    Configuration menu
    Copy the full SHA
    b273180 View commit details
    Browse the repository at this point in the history
  2. test: Try decoding both the source WAV file and the output dump WAV f…

    …ile, since the output file isn't yielding any DTMF tones, according to the dtmf2num tool
    volkertb committed Aug 19, 2024
    Configuration menu
    Copy the full SHA
    618b05a View commit details
    Browse the repository at this point in the history
  3. Revert "ci: try Intel HD Audio (HDA) instead of AC'97/ICHx, since the…

    … latter is apparently not working with SBEMU in the QEMU VM, at least not for the stp32 Protected Mode audio player"
    
    This reverts commit 0e0000c.
    volkertb committed Aug 19, 2024
    Configuration menu
    Copy the full SHA
    b46cca4 View commit details
    Browse the repository at this point in the history
  4. test: fix audio output dump file name, after the revert from HD Audio…

    … to AC'97/ICHx in the previous commit
    volkertb committed Aug 19, 2024
    Configuration menu
    Copy the full SHA
    6b70a0b View commit details
    Browse the repository at this point in the history
  5. ci: upload WAV output dumps from emulated QEMU sound devices as artif…

    …acts, for manual analysis
    volkertb committed Aug 19, 2024
    Configuration menu
    Copy the full SHA
    f805995 View commit details
    Browse the repository at this point in the history
  6. test: Add VOCPLAY.EXE and Sound Blaster AILv2 Real Mode driver to tes…

    …t emulation in Real Mode (or rather V8086 mode) and temporarily comment out Protected Mode playback with STP32.EXE
    volkertb committed Aug 19, 2024
    Configuration menu
    Copy the full SHA
    1420eb9 View commit details
    Browse the repository at this point in the history
  7. ci: restore DOS test error code propagation after having successfully…

    … tested it in a Docker container manually
    volkertb committed Aug 19, 2024
    Configuration menu
    Copy the full SHA
    50fbb51 View commit details
    Browse the repository at this point in the history
  8. ci: fix DOS error code propagation bug by exporting environment varia…

    …ble, so the if-statement further below has it in its context
    volkertb committed Aug 19, 2024
    Configuration menu
    Copy the full SHA
    b4d6c42 View commit details
    Browse the repository at this point in the history
  9. ci: fix DOS error code propagation bug by escaping brackets, since th…

    …ose are reserved characters in YAML
    volkertb committed Aug 19, 2024
    Configuration menu
    Copy the full SHA
    e278310 View commit details
    Browse the repository at this point in the history
  10. ci: debug command parsing error by adding history command to see ho…

    …w the previous if-statement was interpeted
    volkertb committed Aug 19, 2024
    Configuration menu
    Copy the full SHA
    8fc2828 View commit details
    Browse the repository at this point in the history
  11. Configuration menu
    Copy the full SHA
    b22309d View commit details
    Browse the repository at this point in the history
  12. ci: Clean up script after previous commit finally fixed the DOS error…

    … code propagation (i.e. the line with the if-statement)
    volkertb committed Aug 19, 2024
    Configuration menu
    Copy the full SHA
    d40a958 View commit details
    Browse the repository at this point in the history