feat(devkit): improving error handeling read target options

[wout-junius]

Catching a wrong target given in the project.json for component testing.

closed #20251

Current Behavior

When running a Cypress component test with a wrong target it will error with an unclear error:

TypeError: Cannot read properties of undefined (reading 'executor')
    at readTargetOptions (/[PATH TO]/node_modules/@nx/devkit/src/executors/read-target-options.js:24:60)
    etc...

Expected Behavior

With this fix, it should catch when the executor is not found and throw a clear error saying that the target is not found on the project. This makes it clearer what the error is about

Related Issue(s)

Fixes #20251

(This is a re-opening of This PR since something went wring formatting and it closed it)

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment

Name	Status	Preview	Updated (UTC)
nx-dev	⬜️ Ignored (Inspect)	Visit Preview	Jan 30, 2024 5:46pm

[nx-cloud]

☁️ Nx Cloud Report

CI is running/has finished running commands for commit 7c92c10. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this CI Pipeline Execution

---

⌛ The following target is in progress

• nx affected --target=e2e --base=8f9ea6a8c55423f28d6a2159a572d0d90d0a4e13 --head=57cef5d9795e3fa3f7539b71d976176c8a35014c --parallel=1

🟥 Failed Commands
nx affected --targets=lint,test,build --base=57cef5d9795e3fa3f7539b71d976176c8a35014c --head=7c92c10a2491987a5cb0e90f7392fa3a246ffeca --parallel=3

✅ Successfully ran 4 targets

• nx run-many -t check-imports check-commit check-lock-files check-codeowners documentation --parallel=1 --no-dte
• nx-cloud record -- nx format:check --base=57cef5d9795e3fa3f7539b71d976176c8a35014c --head=7c92c10a2491987a5cb0e90f7392fa3a246ffeca
• nx documentation --no-dte
• nx affected -t e2e-macos --parallel=1 --base=57cef5d9795e3fa3f7539b71d976176c8a35014c --head=7c92c10a2491987a5cb0e90f7392fa3a246ffeca

---

Sent with 💌 from NxCloud.

[wout-junius]

Sorry @AgentEnder I accidentally pushed 700+ file changes in my previous PR and reverted that commit concluding to it closing my PR

[nx-cloud]

☁️ Nx Cloud Report

CI is running/has finished running commands for commit e39b1f1. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this CI Pipeline Execution

---

✅ Successfully ran 6 targets

• nx affected --target=e2e --base=77a5c9eaa85118b25b53f1e9df2eeffb76d6d6eb --head=e39b1f1c26b7fdb1f4a745d180013aa83f43ec7a --parallel=1
• nx affected --targets=lint,test,build --base=77a5c9eaa85118b25b53f1e9df2eeffb76d6d6eb --head=e39b1f1c26b7fdb1f4a745d180013aa83f43ec7a --parallel=3
• nx run-many -t check-imports check-commit check-lock-files check-codeowners documentation --parallel=1 --no-dte
• nx-cloud record -- nx format:check --base=77a5c9eaa85118b25b53f1e9df2eeffb76d6d6eb --head=e39b1f1c26b7fdb1f4a745d180013aa83f43ec7a
• nx documentation --no-dte
• nx affected -t e2e-macos --parallel=1 --base=77a5c9eaa85118b25b53f1e9df2eeffb76d6d6eb --head=e39b1f1c26b7fdb1f4a745d180013aa83f43ec7a

---

Sent with 💌 from NxCloud.

[github-actions]

This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request.