fix(elasticache-alpha): the default engine for NoPasswordUser contradict in the docs

[mazyu36]

Issue # (if applicable)

Closes #35847

Reason for this change

NoPasswordUser only supports the Redis engine, but the documentation incorrectly states that the default engine is VALKEY, creating a contradiction between the implementation and documentation. Since NoPasswordUser only works with Redis, the engine property should not be configurable for this user type.

Description of changes

• Fixed the contradiction by removing the engine configuration option from NoPasswordUser.
• The engine property was moved from the base user class to only the user types that actually support multiple engines (IamUserProps and PasswordUserProps).

Describe any new or updated permissions being added

No new or updated IAM permissions are being added with these changes.

Description of how you validated changes

• Verified that all existing unit tests pass after the changes
• Updated test cases to remove references to the now-removed engine property in NoPasswordUser tests

Checklist

• My code adheres to the CONTRIBUTING GUIDE and DESIGN GUIDELINES

BREAKING CHANGE: The engine property in NoPasswordUserProps has been removed.

---

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

[aws-cdk-automation]

(This review is outdated)

[mazyu36]

Exemption request: Only changes to the setting method of the property, sufficient with unit tests.

[github-actions]

	Tests	Passed ❌️	Skipped	Failed
Security Guardian Results

Test	Result
No test annotations available

• View Security Guardian Results

[github-actions]

	Tests	Passed ❌️	Skipped	Failed
Security Guardian Results with resolved templates

Test	Result
No test annotations available

• View Security Guardian Results with resolved templates

✅ Updated pull request passes all PRLinter validations. Dismissing previous PRLinter review.

[mergify]

Thank you for contributing! Your pull request will be updated from main and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork).

[mergify]

Merge Queue Status

✅ The pull request has been merged at bc466a0

This pull request spent 6 seconds in the queue, with no time running CI.
The checks were run in-place.

Required conditions to merge

• #approved-reviews-by >= 1 [🛡 GitHub branch protection]
  • fix(elasticache-alpha): the default engine for NoPasswordUser contradict in the docs #35920
• #changes-requested-reviews-by = 0 [🛡 GitHub branch protection]
  • fix(elasticache-alpha): the default engine for NoPasswordUser contradict in the docs #35920
• any of [🛡 GitHub branch protection]:
  • check-success = validate-pr
  • check-neutral = validate-pr
  • check-skipped = validate-pr
• any of [🛡 GitHub branch protection]:
  • check-success = build
  • check-neutral = build
  • check-skipped = build

[github-actions]

Comments on closed issues and PRs are hard for our team to see.
If you need help, please open a new issue that references this one.