surface conflicting env var names as warnings and errors

[christianvogt]

https://issues.redhat.com/browse/RHOAIENG-11538

Description

Provides user with feedback when creating fields which have conflicting env var values.
Note that microcopy is not finalized.

cc @simrandhaliw

How Has This Been Tested?

• Go to Settings -> Connection types as an admin with the connection type feature flag enabled.
• Create a new connection type.
• Add two fields with the same env var name.
• Observe the error in the table and fields modal and that the form save button is disabled
• Edit a conflicting field to have a unique env var
• Observe the form save button is no longer disabled (providing the rest of the form is valid)

Test Impact

Added unit tests.

Request review criteria:

Self checklist (all need to be checked):

• The developer has manually tested the changes and verified that the changes work
• Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
• The developer has added tests or explained why testing cannot be added (unit or cypress tests for related changes)

If you have UI changes:

• Included any necessary screenshots or gifs if it was a UI change.
• Included tags to the UX team if it was a UI/UX change.

After the PR is posted & before it merges:

• The developer has tested their solution on a cluster by using the image produced by the PR to main

[emilys314]

I tested this on my cluster and it looks like it's all working to me

[christianvogt]

@emilys314 updated with useMemo where env vars array is computed

[emilys314]

/lgtm

[christianvogt]

rebased to fix all the conflicts with main

[codecov]

Codecov Report

Attention: Patch coverage is 92.85714% with 2 lines in your changes missing coverage. Please review.

Project coverage is 80.85%. Comparing base (4a0626e) to head (bf5e8dd).
Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
...ctionTypes/manage/ConnectionTypeDataFieldModal.tsx	90.00%	1 Missing ⚠️
...onnectionTypes/manage/ManageConnectionTypePage.tsx	83.33%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3124      +/-   ##
==========================================
+ Coverage   80.84%   80.85%   +0.01%     
==========================================
  Files        1243     1244       +1     
  Lines       27099    27117      +18     
  Branches     7144     7152       +8     
==========================================
+ Hits        21907    21926      +19     
+ Misses       5192     5191       -1     

Files with missing lines	Coverage Δ
frontend/src/concepts/connectionTypes/types.ts	100.00% <100.00%> (ø)
...onnectionTypes/manage/ConnectionTypeFieldModal.tsx	25.00% <ø> (ø)
...onTypes/manage/ManageConnectionTypeFieldsTable.tsx	38.88% <ø> (-1.12%)	⬇️
...ypes/manage/ManageConnectionTypeFieldsTableRow.tsx	82.22% <100.00%> (+3.27%)	⬆️
.../pages/connectionTypes/manage/fieldTableColumns.ts	100.00% <100.00%> (ø)
...ctionTypes/manage/ConnectionTypeDataFieldModal.tsx	81.08% <90.00%> (+0.79%)	⬆️
...onnectionTypes/manage/ManageConnectionTypePage.tsx	77.77% <83.33%> (+0.27%)	⬆️

... and 2 files with indirect coverage changes

---

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4a0626e...bf5e8dd. Read the comment docs.

[christianvogt]

rebased

[jeff-phillips-18]

/lgtm

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

Approval requirements bypassed by manually added approval.

This pull-request has been approved by:

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• frontend/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment