Ensure ddev env test skips environments when disabled by e2e-env

[AAraKKe]

What does this PR do?

Modifis the ddev env test command to ensure that when run on an integration that has the e2e-env option set to False, end-to-end test do not run.

This fix also supersedes the previous quick fix we did (#20967) to ensure we were on time for release 7.70.0.

Motivation

This variable was respected when running with environment set to all but, if an environment was provided, end-to-end test would be launched. The dd_environment fixture is not available and it would break. Some integrations might have the fixture defined and this won't break but since it is not a requirement when no end-to-end tests are defined, we need to make sure the command does not break.

This became an issue once we started parallelizing tests per environment and we now run a lot of them passing the target environment. Even for integrations that do not have end-to-end tests.

Validation

When running before this comment for infinibad that has e2e set to False and no fixture defined we would get the following error:

ddev env test --base --dev infiniband py3.12
────────────────────────────────────────────────────────────────────── Infiniband ───────────────────────────────────────────────────────────────────────
─────────────────────────────────────────────────────────────────── Starting: py3.12 ────────────────────────────────────────────────────────────────────

!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! _pytest.outcomes.Exit: NO E2E FIXTURE AVAILABLE !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!

After the fix

ddev env test --base --dev infiniband py3.12
Selected target 'infiniband' does not have E2E tests to run. Skipping.

Review checklist (to be filled by reviewers)

• Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
• Add the qa/skip-qa label if the PR doesn't need to be tested during QA.
• If you need to backport this PR to another branch, you can add the backport/<branch-name> label to the PR and it will automatically open a backport PR once this one is merged

[codecov]

Codecov Report

❌ Patch coverage is 96.74419% with 7 lines in your changes missing coverage. Please review.
✅ Project coverage is 90.84%. Comparing base (c00de20) to head (08d20b0).
⚠️ Report is 11 commits behind head on master.

Additional details and impacted files

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[NouemanKHAL]

Suggestion for the PR title: Ensure ddev env test skips environments with disabled e2e-env

[NouemanKHAL]

Otherwise, LGTM! Nice!

Review from NouemanKHAL is dismissed. Related teams and files:

• agent-integrations
  • ddev/src/ddev/cli/env/test.py
  • ddev/tests/cli/env/test_test.py

Review from steveny91 is dismissed. Related teams and files:

• agent-integrations
  • ddev/src/ddev/cli/env/test.py
  • ddev/tests/cli/env/test_test.py