Skip to content

test: add unit tests for SobProjectModal#460

Merged
andoriyaprashant merged 1 commit into
andoriyaprashant:mainfrom
adityashirsatrao007:test/sob-project-modal
Jun 3, 2026
Merged

test: add unit tests for SobProjectModal#460
andoriyaprashant merged 1 commit into
andoriyaprashant:mainfrom
adityashirsatrao007:test/sob-project-modal

Conversation

@adityashirsatrao007

Copy link
Copy Markdown
Contributor

Description

Adds comprehensive unit tests for the SobProjectModal class.

Changes

  • Added test/sob_project_modal_test.dart with 9 test cases covering:
    • Constructor field initialization (8 fields including projects list)
    • copyWith() with partial updates
    • toMap() serialization (uses project_links key)
    • fromMap() deserialization with project_links list
    • fromMap() graceful handling of missing project_links
    • fromMap() with all null/empty values (uses defaults)
    • Equality operator (==)
    • Inequality when fields differ
    • toString() output

Related Issue

Contributes to #417 (improving test coverage and code quality)

Type of Change

  • New feature (adding tests)

Checklist

  • Tests pass locally
  • Code follows project style guidelines

Comprehensive unit tests for SobProjectModal covering:
- Constructor field initialization (8 fields including projects list)
- copyWith() with partial updates
- toMap() serialization with project_links key
- fromMap() deserialization
- fromMap() graceful handling of missing project_links
- fromMap() with all null/empty values
- Equality operator (==)
- Inequality when fields differ
- toString() output
Copilot AI review requested due to automatic review settings May 30, 2026 10:55
@github-actions

Copy link
Copy Markdown

Thank you for submitting your pull request! We'll review it as soon as possible.

@netlify

netlify Bot commented May 30, 2026

Copy link
Copy Markdown

Deploy Preview for aquamarine-kheer-83feda failed. Why did it fail? →

Name Link
🔨 Latest commit 0db802a
🔍 Latest deploy log https://app.netlify.com/projects/aquamarine-kheer-83feda/deploys/6a1ac23923e88e0007f8626d

@andoriyaprashant andoriyaprashant merged commit d19d881 into andoriyaprashant:main Jun 3, 2026
1 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants