Skip to content

fix: re-enable snippet integration test#5161

Closed
999axel999 wants to merge 3 commits into
objectionary:masterfrom
999axel999:zerocracy/5090-reenable-snippet-it
Closed

fix: re-enable snippet integration test#5161
999axel999 wants to merge 3 commits into
objectionary:masterfrom
999axel999:zerocracy/5090-reenable-snippet-it

Conversation

@999axel999
Copy link
Copy Markdown
Contributor

Summary

  • Re-enable SnippetIT.runsAllSnippets by removing its @Disabled annotation.
  • Remove the resolved PDD puzzle text for #4707.

Closes #5090

Tests

  • Verified SnippetIT.java no longer imports or uses Disabled
  • Verified the #4707 PDD text was removed from SnippetIT.java

Prepared with local automation assistance and reviewed before submission.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 23, 2026

🚀 Performance Analysis

All benchmarks are within the acceptable range. No critical degradation detected (threshold is 100%). Please refer to the detailed report for more information.

Click to see the detailed report
Test Base Score PR Score Change % Change Unit Mode
benchmarks.XmirBench.xmirToEO 9214.158 9336.438 122.279 1.33% ms/op Average Time

⚠️ Performance loss: benchmarks.XmirBench.xmirToEO is slower by 122.279 ms/op (1.33%)

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Jun 5, 2026

@yegor256
Copy link
Copy Markdown
Member

yegor256 commented Jun 5, 2026

@999axel999 is it some sort of slop?

@yegor256 yegor256 closed this Jun 5, 2026
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.

SnippetIT.runsAllSnippets function is disabled, causing improper functionality

2 participants