Skip to content

Various fixes/tweaks for unit tests.#5275

Merged
out-of-phaze merged 1 commit intoNebulaSS13:devfrom
MistakeNot4892:organ_validation_test
Feb 1, 2026
Merged

Various fixes/tweaks for unit tests.#5275
out-of-phaze merged 1 commit intoNebulaSS13:devfrom
MistakeNot4892:organ_validation_test

Conversation

@MistakeNot4892
Copy link
Copy Markdown
Contributor

  • Allows bodytypes to skip organ validation (needed for bodytypes that don't implement a full set of organs, like specific prosthetic models).
  • Makes category generation more explicit about failures in fusion reaction info.
  • Fixes a copypaste error.

@MistakeNot4892 MistakeNot4892 added the ready for review This PR is ready for review and merge. label Jan 15, 2026
Comment thread code/modules/bodytype/_bodytype.dm Outdated

var/list/tail_data = has_limbs[BP_TAIL]
if(tail_data)
if(tail_data && !TYPE_IS_ABSTRACT(src))
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

i don't think abstract bodytypes can even be instantiated or validated, can they? did this fix a real practical issue? because afaict it should never be triggered.

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.

Oh, no, I just chucked it in there in case.

@out-of-phaze out-of-phaze merged commit c3b0622 into NebulaSS13:dev Feb 1, 2026
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready for review This PR is ready for review and merge.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants