Simplify test setup

[benjeffery]

No description provided.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.23%. Comparing base (a4d801c) to head (3eb78d2).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #2420   +/-   ##
=======================================
  Coverage   91.23%   91.23%           
=======================================
  Files          20       20           
  Lines       11831    11831           
  Branches     2296     2296           
=======================================
  Hits        10794    10794           
  Misses        568      568           
  Partials      469      469           

Flag	Coverage Δ
C	91.23% <ø> (ø)
python	98.68% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[jeromekelleher]

Note we should be able to get rid of MSP_CONDA_PREFIX etc from setup.py now too, given this?

[benjeffery]

MSP_CONDA_PREFIX

I'm not sure - isn't this used in the feedstock?

[jeromekelleher]

Oh yes, maybe it is! Oops.

[jeromekelleher]

Some gnarly powershell needed here - what's the basic gist of what's going on there? Is it a problem with the wheel or the locally build dll? If it's the wheel, does this need to be done for the pypi artefact also?

[benjeffery]

Some gnarly powershell needed here

The basic problem is that building then running the tests was not finding the needed DLLs. I tried putting what was needed on the PATH but had no success. Building the wheel is a workaround as delvewheel finds the DLLs and puts them in the wheel. We then run the tests with the wheel.

[jeromekelleher]

Might be worth writing some of this up if it's going to stay long term - won't be obvious to most devs