Skip to content

test: add unit tests for LinuxFoundationModal#458

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

test: add unit tests for LinuxFoundationModal#458
andoriyaprashant merged 1 commit into
andoriyaprashant:mainfrom
adityashirsatrao007:test/linux-foundation-modal

Conversation

@adityashirsatrao007

Copy link
Copy Markdown
Contributor

Description

Adds comprehensive unit tests for the LinuxFoundationModal class.

Changes

  • Added test/linux_foundation_modal_test.dart with 9 test cases covering:
    • Constructor field initialization
    • copyWith() with partial updates and no-op copy
    • toMap() serialization
    • fromMap() deserialization
    • Equality operator (==) for identical objects
    • Inequality when fields differ
    • hashCode consistency for equal objects
    • toJson() JSON encoding
    • 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 LinuxFoundationModal covering:
- Constructor field initialization
- copyWith() with partial updates
- toMap() serialization
- fromMap() deserialization
- Equality operator (==)
- Inequality when fields differ
- hashCode consistency for equal objects
- toJson() JSON encoding
- 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 f9f7f9f
🔍 Latest deploy log https://app.netlify.com/projects/aquamarine-kheer-83feda/deploys/6a1ac227fcfac00008bf3d1f

@andoriyaprashant andoriyaprashant merged commit 52792c1 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