Fixing tests related to repo restructuring

justbuchanan commented 1 year ago

This PR is just a small addition to https://github.com/CadQuery/cq-cli/pull/23. I'm opening it to get some feedback from GitHub actions and see if test fixes are working.

@jmwright if this PR is helpful, feel free to just merge this into #23 and close it.

Notes:

I recently discovered act, which was pretty helpful in testing this locally rather than pushing a ton of commits to github
I updated everything in tests/ to point to src/cq_cli/cq_cli.py rather than the old path cq-cli.py
Every test was failing with an error about being unable to find "libGL.so.1", so I added an apt-get install libgl1-mesa-glx to tests.yml. This seems to have fixed things for linux, but we'll have to do something different for macOS and windows. Update: I was able to change this to only run apt-get install on linux

justbuchanan commented 1 year ago

Ok, everything passes now on windows and linux, but not macos.

justbuchanan commented 1 year ago

The tests are passing on linux and windows, but macos gives us:

import cadquery as cq
ImportError: No module named cadquery

jmwright commented 1 year ago

The MacOS runner has been a pain on this repo for some reason. You can just take it out of the matrix for now and I'll fix it later. Thanks

jmwright commented 1 year ago

Actually, I'll just merge this and disable MaxOS. Thanks for fixing the tests.

jmwright commented 1 year ago

All green now on the restructure branch. Thanks @justbuchanan !

CadQuery / cq-cli

Fixing tests related to repo restructuring #24