Skip to content

Fix FieldInfo.equals() issue#234

Merged
Splatcrafter merged 1 commit intoaether-framework:developfrom
darya-vv:fix/issues-172-field-info-equals
Mar 26, 2026
Merged

Fix FieldInfo.equals() issue#234
Splatcrafter merged 1 commit intoaether-framework:developfrom
darya-vv:fix/issues-172-field-info-equals

Conversation

@darya-vv
Copy link
Copy Markdown

@darya-vv darya-vv commented Mar 25, 2026

Summary

Fixed bug with equals method and added tests for same case

Type of Change

  • Bug fix

Related Issues

Closes issue

Changes

  • Updated equals method in FieldInfo class
  • Added tests in FieldInfoTest.EqualsAndHashCode

Verification

  • Unit tests added/updated
  • Existing tests pass
  • Manual verification performed (if applicable)

Checklist

  • Code follows project conventions
  • Public APIs are documented
  • Tests cover new behavior
  • No unnecessary dependencies added

1) same Type.ref and Type.describe but different Type.codec -> objects not equal
2) same Type.codec but different Type.ref and Type.describe -> objects not equal
@darya-vv darya-vv requested a review from a team as a code owner March 25, 2026 18:51
@Splatcrafter
Copy link
Copy Markdown
Member

I will look over it tomorrow morning. Thanks for your PR!

@Splatcrafter Splatcrafter self-assigned this Mar 26, 2026
@Splatcrafter Splatcrafter added this to the v1.0.0 milestone Mar 26, 2026
@Splatcrafter Splatcrafter added bug Something isn't working good first issue Good for newcomers labels Mar 26, 2026
@Splatcrafter Splatcrafter self-requested a review March 26, 2026 08:22
@Splatcrafter Splatcrafter changed the base branch from main to develop March 26, 2026 08:38
@Splatcrafter Splatcrafter merged commit 70155fd into aether-framework:develop Mar 26, 2026
10 of 11 checks passed
@Splatcrafter
Copy link
Copy Markdown
Member

Thank you for your contribution to this project! Your name will be recognized in the contributors list as part of the next release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working good first issue Good for newcomers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: FieldInfo.equals() compares only type reference, not type structure

2 participants