Fix dart tests

[Fellmonkey]

What does this PR do?

1. Enhanced test data generation for response models to use generic data structures when multiple models are present, while maintaining specific type expectations for single-model cases.
2. Removed unnecessary server test intended for the client

This ensures that generated tests use valid enum values for properties with enums, preventing runtime errors during test execution.

Test Plan

regenerate sdk and run dart test

also push changes and ensure the GitHub Actions workflow for Dart SDK completes successfully with all tests passing. #1247

before:


after:

Have you read the Contributing Guidelines on issues?

Yes

Summary by CodeRabbit

• Tests
  • Enhanced test data generation for response models to use generic data structures when multiple models are present, while maintaining specific type expectations for single-model cases.

[coderabbitai]

📝 Walkthrough

Walkthrough

A template file for generating Dart service tests has been updated to handle response model types conditionally. When multiple response models exist, the generated test expects a Map<String, dynamic> instead of a specific model type. When only one response model exists, the specific model type is still expected.

Changes

Cohort / File(s)	Summary
Dart Test Template templates/dart/test/services/service_test.dart.twig	Modified test data generation logic to expect Map<String, dynamic> for multiple response models, while preserving specific model type expectations for single response model scenarios.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Poem

🐰 Four lines of code, so neat and small,
A template tweak that handles them all,
When models multiply, a Map takes flight,
But one stays loyal, specific and tight! ✨

🚥 Pre-merge checks | ✅ 3 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Title check	❓ Inconclusive	The title 'Fix dart tests' is vague and generic, failing to specify which aspect of Dart tests is being fixed or what the actual change is.	Use a more descriptive title that specifies the fix, such as 'Fix Dart tests by using valid enum values for string properties' or 'Ensure Dart tests use valid enum values in mock data'.

✅ Passed checks (3 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Merge Conflict Detection	✅ Passed	✅ No merge conflicts detected when merging into master
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[Fellmonkey]

[Fellmonkey]

commit -> failed test