Skip to content

Add pallet testing #1168

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 9 commits into
base: staging/product-ia
Choose a base branch
from
Open

Add pallet testing #1168

wants to merge 9 commits into from

Conversation

@brunopgalvao
Copy link
Contributor

@brunopgalvao brunopgalvao commented Nov 3, 2025

📝 Description

Resolves #1048

🔍 Review Preference

Choose one:

  • ✅ I have time to handle formatting/style feedback myself
  • ⚡ Docs team handles formatting (check "Allow edits from maintainers")

🤖 AI-Ready Docs

If content changed, regenerate AI files:

  • ✅ I ran python3 scripts/generate_llms.py
  • ⚡ Docs team will regenerate (check "Allow edits from maintainers")

✅ Checklist

@brunopgalvao brunopgalvao self-assigned this Nov 3, 2025
@brunopgalvao brunopgalvao added B0 - Needs Review Pull request is ready for review C1 - Medium Medium priority task A0 - New Content Pull request contains new content pages labels Nov 3, 2025
@brunopgalvao brunopgalvao marked this pull request as ready for review November 3, 2025 14:49
@brunopgalvao brunopgalvao requested a review from a team as a code owner November 3, 2025 14:49
nhussein11
nhussein11 approved these changes Nov 11, 2025
View reviewed changes
Copy link
Collaborator

@nhussein11 nhussein11 left a comment

Choose a reason for hiding this comment

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

Thanks, left some minor comments

parachains/customize-runtime/pallet-development/pallet-testing.md
Comment on lines +314 to +317
```
running 15 tests
test mock::__construct_runtime_integrity_test::runtime_integrity_tests ... ok
test mock::test_genesis_config_builds ... ok
Copy link
Collaborator

@nhussein11 nhussein11 Nov 11, 2025

Choose a reason for hiding this comment

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

This should be a termynal element

@brunopgalvao
Adding a pallet is included in the create a pallet page
c7a2f4e
@brunopgalvao
Resolve PR review comments
aef21f1 
- Add periods to bullet points for consistency
- Change Events and Block Number admonition from warning to info
- Add title to lib.rs code block
@brunopgalvao
Resolve PR review comments and add pallet code reference
14e698b 
- Add periods to all bullet points for consistency
- Change Events and Block Number admonition from warning to info
- Add title to lib.rs code block
- Add collapsible complete pallet code reference using snippet include
- Add test descriptions as sentences under section headers
@brunopgalvao
Merge branch 'staging/product-ia' into bruno/pallet-unit-testing
d8fb1a4
@brunopgalvao
llms
aea766c
@brunopgalvao
Merge remote-tracking branch 'origin/staging/product-ia' into bruno/p…
10bb664 
…allet-unit-testing
@brunopgalvao
resolve llms
a127e47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@nhussein11 nhussein11 nhussein11 approved these changes

At least 1 approving review is required to merge this pull request.

Assignees

@brunopgalvao brunopgalvao

@Telucero Telucero

Labels

A0 - New Content Pull request contains new content pages B0 - Needs Review Pull request is ready for review C1 - Medium Medium priority task

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@brunopgalvao @nhussein11 @Telucero