Skip to content
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.

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

Frame Pallets: Clean a lot of test setups #4642

Merged
merged 16 commits into from
Jun 7, 2024
Merged

Conversation

kianenigma
Copy link
Contributor

Screenshot 2024-05-30 at 10 30 41

@kianenigma kianenigma added R0-silent Changes should not be mentioned in any release notes T1-FRAME This PR/Issue is related to core FRAME, the framework. labels May 30, 2024
@kianenigma kianenigma requested a review from a team as a code owner May 30, 2024 02:31
@paritytech-review-bot paritytech-review-bot bot requested a review from a team May 30, 2024 02:31
Copy link
Contributor

@seadanda seadanda left a comment

Choose a reason for hiding this comment

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

Missed a few opportunities to remove things ;) already happy to approve as is though

substrate/frame/system/benches/bench.rs Outdated Show resolved Hide resolved
substrate/frame/indices/src/mock.rs Outdated Show resolved Hide resolved
type BlockLength = ();
type DbWeight = ();
type RuntimeOrigin = RuntimeOrigin;
type RuntimeCall = RuntimeCall;
type Nonce = u64;
Copy link
Contributor

Choose a reason for hiding this comment

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

Elsewhere you've removed this and converted the tests to use u32 - is it worth just doing this everywhere?

substrate/frame/tx-pause/src/mock.rs Outdated Show resolved Hide resolved
substrate/frame/transaction-payment/src/mock.rs Outdated Show resolved Hide resolved
substrate/frame/system/benchmarking/src/mock.rs Outdated Show resolved Hide resolved
substrate/frame/im-online/src/mock.rs Show resolved Hide resolved
@kianenigma kianenigma enabled auto-merge May 31, 2024 02:24
@paritytech-cicd-pr
Copy link

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: cargo-clippy
Logs: https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/6417534

@kianenigma kianenigma disabled auto-merge June 7, 2024 07:40
@kianenigma kianenigma enabled auto-merge June 7, 2024 07:40
@kianenigma kianenigma added this pull request to the merge queue Jun 7, 2024
Merged via the queue into master with commit 9bb1f3f Jun 7, 2024
157 checks passed
@kianenigma kianenigma deleted the kiz-clean-test-setups branch June 7, 2024 11:59
ordian added a commit that referenced this pull request Jun 13, 2024
* master: (29 commits)
  Append overlay optimization. (#1223)
  finalization: Skip tree route calculation if no forks present (#4721)
  Remove unncessary call remove_from_peers_set (#4742)
  add pov-recovery unit tests and support for elastic scaling (#4733)
  approval-voting: Add no shows debug information (#4726)
  Revamp the Readme of the parachain template (#4713)
  Update README.md to move the PSVM link under a "Tooling" section under the "Releases" section (#4734)
  frame/proc-macro: Refactor code for better readability (#4712)
  Contracts:  update wasmi to 0.32 (#3679)
  Backport style changes from P<>K bridge to R<>W bridge (#4732)
  New reference doc for Custom RPC V2 (#4654)
  Frame Pallets: Clean a lot of test setups (#4642)
  Fix occupied core handling (#4691)
  statement-distribution: Fix false warning (#4727)
  Update the README to include a link to the Polkadot SDK Version Manager (#4718)
  Cleanup PVF artifact by cache limit and stale time (#4662)
  Update link to a latest polkadot release (#4711)
  [CI] Delete cargo-deny config (#4677)
  fix build on MacOS: bump secp256k1 and secp256k1-sys to patched versions (#4709)
  Unify dependency aliases (#4633)
  ...
TarekkMA pushed a commit to moonbeam-foundation/polkadot-sdk that referenced this pull request Aug 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
R0-silent Changes should not be mentioned in any release notes T1-FRAME This PR/Issue is related to core FRAME, the framework.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants