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.

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

Refactor Identity to benchmark v2 #1838

Merged
merged 12 commits into from
Oct 11, 2023
Merged

Refactor Identity to benchmark v2 #1838

Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
55791d3
refactor to benchmark v2
Oct 10, 2023
000d5d6
Update substrate/frame/identity/src/benchmarking.rs
0xmovses Oct 10, 2023
a04d14e
bot fmt
0xmovses Oct 11, 2023
c05e883
removed expanded output
0xmovses Oct 11, 2023
09c6630
".git/.scripts/commands/fmt/fmt.sh"
Oct 11, 2023
3555c26
Merge branch 'master' into 0xmovses-identity-benchmark-v2
franciscoaguirre Oct 11, 2023
ed8c24b
remove v1 import
0xmovses Oct 11, 2023
34c3d65
".git/.scripts/commands/fmt/fmt.sh"
Oct 11, 2023
fa26b04
Merge branch 'master' into 0xmovses-identity-benchmark-v2
0xmovses Oct 11, 2023
9f766f5
Update substrate/frame/identity/src/benchmarking.rs
0xmovses Oct 11, 2023
02f2fd1
use ensure
0xmovses Oct 11, 2023
8ba5ebc
".git/.scripts/commands/fmt/fmt.sh"
Oct 11, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
".git/.scripts/commands/fmt/fmt.sh"
  • Loading branch information
command-bot committed Oct 11, 2023
commit 09c6630390c4a1569106116586dff4914be36f0c
36 changes: 16 additions & 20 deletions substrate/frame/identity/src/benchmarking.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -48,16 +48,14 @@ fn add_registrars<T: Config>(r: u32) -> Result<(), &'static str> {
.expect("RegistrarOrigin has no successful origin required for the benchmark");
Identity::<T>::add_registrar(registrar_origin, registrar_lookup)?;
Identity::<T>::set_fee(RawOrigin::Signed(registrar.clone()).into(), i, 10u32.into())?;
let fields = IdentityFields(
IdentityField::Display
| IdentityField::Legal
| IdentityField::Web
| IdentityField::Riot
| IdentityField::Email
| IdentityField::PgpFingerprint
| IdentityField::Image
| IdentityField::Twitter,
);
let fields =
IdentityFields(
IdentityField::Display |
IdentityField::Legal | IdentityField::Web |
IdentityField::Riot | IdentityField::Email |
IdentityField::PgpFingerprint |
IdentityField::Image | IdentityField::Twitter,
);
Identity::<T>::set_fields(RawOrigin::Signed(registrar.clone()).into(), i, fields)?;
}

Expand Down Expand Up @@ -388,16 +386,14 @@ mod benchmarks {
.expect("RegistrarOrigin has no successful origin required for the benchmark");
Identity::<T>::add_registrar(registrar_origin, caller_lookup)?;

let fields = IdentityFields(
IdentityField::Display
| IdentityField::Legal
| IdentityField::Web
| IdentityField::Riot
| IdentityField::Email
| IdentityField::PgpFingerprint
| IdentityField::Image
| IdentityField::Twitter,
);
let fields =
IdentityFields(
IdentityField::Display |
IdentityField::Legal | IdentityField::Web |
IdentityField::Riot | IdentityField::Email |
IdentityField::PgpFingerprint |
IdentityField::Image | IdentityField::Twitter,
);

let registrars = Registrars::<T>::get();
assert!(
Expand Down