BUG: test:build constraints not respected when using selectors

[h-vetinari]

Hi! 👋

I'm starting to work on emscripten-forge a bit these days, and ran into the following problem, where @DerThorsten asked me to raise an issue.

In our pyjs recipe, there's the following:

tests:
  - script: |
       [...]
    requirements:
      build:      
          - pytester
          - if: linux
            then:
              - pyjs_code_runner >=3.0.0
              - microsoft::playwright <1.42
            else:
              - pyjs_code_runner >=3.0.0
              - microsoft::playwright

however, despite being on linux, my test environment mis-detects the condition:

 │ │ Resolving environment for:
 │ │   Platform: linux-64
 │ │   Channels:
 │ │    - file:///tmp/.tmp4JVVSW
 │ │    - file:///home/[...]/emf/output
 │ │    - https://repo.mamba.pm/emscripten-forge
 │ │    - microsoft
 │ │    - conda-forge
 │ │   Specs:
 │ │    - pytester
 │ │    - pyjs_code_runner >=3.0.0
 │ │    - microsoft::playwright                 # <-- no constraint!

which unsurprisingly installs:

 │ │ │ playwright          ┆ 1.44.0   ┆ py312_0            ┆ microsoft   ┆ 32.58 MiB  │

removing the if: linux portion makes the constraints work again.

[wolfv]

I'll take a look!

[wolfv]

OK, actually there is no bug because the if: linux is for the target_platform (emscripten) and not the build_platform.

You should be able to fix the issue by using build_platform == "linux-64"

[wolfv]

It's a little confusing, but it's also consistent with the rest of the recipe, so not sure what to do here. Ideas welcome!

It also makes me realize that it's not easy to check wether the build_platform is unix, or linux. Maybe we should add is_linux(build_platform), or is_unix(build_platform) functions ...

[wolfv]

We could also add a starts_with filter to minijinja :)

[h-vetinari]

OK, actually there is no bug because the if: linux is for the target_platform (emscripten) and not the build_platform.

You should be able to fix the issue by using build_platform == "linux-64"

OK, that makes sense of course! Clearly I'm not yet used to the test section having a build section as well! 😅

+1 to the other jinja enhancements too!

[mitsuhiko]

We could also add a starts_with filter to minijinja :)

There is a startingwith test in minijinja:

build_platform is startingwith('linux')