Skip to content

Add comprehensive tests for Monify console wrappers#24

Merged
MooVC merged 4 commits intohotfix/19-pm-arrayequalityfrom
codex/create-comprehensive-tests-for-monify-classes
Oct 27, 2025
Merged

Add comprehensive tests for Monify console wrappers#24
MooVC merged 4 commits intohotfix/19-pm-arrayequalityfrom
codex/create-comprehensive-tests-for-monify-classes

Conversation

@MooVC
Copy link
Copy Markdown
Owner

@MooVC MooVC commented Oct 26, 2025

Summary

  • add tests for simple Monify wrappers in classes, records, and structs covering constructors, equality, conversions, and formatting
  • add tests for every nested Monify wrapper scenario across class, record, struct, interface, and record-struct containers

Testing

  • dotnet test src/Monify.Console.Tests/Monify.Console.Tests.csproj

https://chatgpt.com/codex/tasks/task_b_68fe951bfe608323aae750f9e019f32f

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@codecov
Copy link
Copy Markdown

codecov Bot commented Oct 26, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.46%. Comparing base (c15e6f3) to head (25de740).
⚠️ Report is 1 commits behind head on hotfix/19-pm-arrayequality.

Additional details and impacted files
@@                      Coverage Diff                       @@
##           hotfix/19-pm-arrayequality      #24      +/-   ##
==============================================================
+ Coverage                       88.24%   89.46%   +1.21%     
==============================================================
  Files                              43       44       +1     
  Lines                             851      864      +13     
  Branches                           86       89       +3     
==============================================================
+ Hits                              751      773      +22     
+ Misses                             60       50      -10     
- Partials                           40       41       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@MooVC MooVC merged commit 4b40314 into hotfix/19-pm-arrayequality Oct 27, 2025
3 checks passed
@MooVC MooVC deleted the codex/create-comprehensive-tests-for-monify-classes branch October 27, 2025 21:46
MooVC added a commit that referenced this pull request Oct 31, 2025
* Added SequenceEqualityComparerGenerator and HashCodeGenerator to mimic the equality behavior supported by Valuify (#19).
Added Console and Console.Tests to support tests on generated code (rather than just tests for generated code).

* Mirror declaration snippets in console project (#20)

* Mirror declaration snippets in console project

* Corrected Simple class definitions - some minor whitespace cleanup

* Supressed annoying warning

* Addressed some warnings

* docs: add xml comments to console samples (#21)

* Add comprehensive tests for SimpleForInt Monify output (#23)

* Add comprehensive tests for SimpleForInt Monify output

* Corrected tests for SimpleForInt

* Add comprehensive tests for Monify console wrappers (#24)

* Add Monify console wrapper tests

* Minor format corrections

* - `ToString` no longer throws a `FormatException`.

* - Equality checks involing an encapsulated array containing identical values now yield true (#19).

* Add equivalent string operator tests across ForString suites (#25)

* Add ForString operator tests for equivalent strings

* Applied suggestions

* Flatten nested console test InnerTests folders (#26)

* Flatten nested console test InnerTests directories

* Remove trailing newlines from nested console tests

* Use collection expressions in immutable array tests (#27)

* - MONFY03 is no longer raised if the encapsulated type cannot be determined (#22).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant