Skip to content

Fix CheatSheet Tests#219

Closed
maennchen wants to merge 1 commit intoash-project:mainfrom
maennchen:jm/cheat_sheet_test_fix
Closed

Fix CheatSheet Tests#219
maennchen wants to merge 1 commit intoash-project:mainfrom
maennchen:jm/cheat_sheet_test_fix

Conversation

@maennchen
Copy link
Copy Markdown
Contributor

Contributor checklist

Leave anything that you believe does not apply unchecked.

  • I accept the AI Policy, or AI was not used in the creation of this PR.
  • Bug fixes include regression tests
  • Chores
  • Documentation changes
  • Features include unit/acceptance tests
  • Refactoring
  • Update dependencies

A keyword list of nested entities.
"""
@type entities :: keyword(t)
@type entities :: keyword(t | [t])
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This seems strange, entities should be a keyword list no?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

You tell me, in the entity nested definitions must be lists of entities or a single entity test, we have this:
entities: [child: @child, other_children: [@different_child]]

So we for sure have a keyword list outside, but the value is apparently either one or more entities.

Copy link
Copy Markdown
Contributor

@marot marot Sep 25, 2025

Choose a reason for hiding this comment

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

This is something I changed in #218. It should also be fixed by #221
I'm not sure why tests failed after merging, but not in the pull request...

#221 should make the List.wrap calls unnecessary

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@marot I’ll close this PR then and leave it in your hands 😄

I’ll rebase #220 as well once your PR is merged since it contained related / wrong code.

@maennchen maennchen closed this Sep 25, 2025
@maennchen maennchen deleted the jm/cheat_sheet_test_fix branch September 25, 2025 20:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants