Skip to content

Initial implementation of IMP mockup#805

Merged
Kailai-Wang merged 25 commits intodevfrom
765-mock-IMP
Sep 9, 2022
Merged

Initial implementation of IMP mockup#805
Kailai-Wang merged 25 commits intodevfrom
765-mock-IMP

Conversation

@Kailai-Wang
Copy link
Copy Markdown
Collaborator

@Kailai-Wang Kailai-Wang commented Sep 1, 2022 •
edited
Loading

resolves #765

This PR includes:

  • a mock-up of shielding keys
  • a replay of decryption -> decoding -> processing of call_worker bytes
  • a mock-up for the linking/unliking/verification of identities and hiding the logic in TEE. It should provide a template for F/E to interact with the parachain (= extrinsics and events)
  • small improvements to the CI

TODO:

  • unittests (mock of mock)
  • verification message data structure within verify_identity
  • maybe e2e tests with F/E

@Kailai-Wang Kailai-Wang requested review from a team and ziming-zung September 1, 2022 07:55
@Kailai-Wang
Copy link
Copy Markdown
Collaborator Author

Kailai-Wang commented Sep 1, 2022

Will check later for compilation error

@Kailai-Wang Kailai-Wang marked this pull request as draft September 3, 2022 19:04
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Sep 3, 2022 •
edited
Loading

Codecov Report

Merging #805 (c541046) into dev (41c984e) will decrease coverage by 2.38%.
The diff coverage is 15.81%.

@@            Coverage Diff             @@
##              dev     #805      +/-   ##
==========================================
- Coverage   93.01%   90.63%   -2.39%     
==========================================
  Files          21       25       +4     
  Lines        6744     6959     +215     
==========================================
+ Hits         6273     6307      +34     
- Misses        471      652     +181
Impacted Files Coverage Δ
...s/identity-management-mock/src/identity_context.rs 0.00% <0.00%> (ø)
pallets/identity-management-mock/src/lib.rs 0.00% <0.00%> (ø)
...ets/identity-management-mock/src/stf_primitives.rs 0.00% <0.00%> (ø)
pallets/identity-management-mock/src/key.rs 87.17% <87.17%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@Kailai-Wang Kailai-Wang self-assigned this Sep 4, 2022
@Kailai-Wang Kailai-Wang marked this pull request as ready for review September 4, 2022 14:37
@Kailai-Wang Kailai-Wang mentioned this pull request Sep 4, 2022
Closed
@Kailai-Wang Kailai-Wang requested review from BillyWooo and jingleizhang and removed request for a team September 9, 2022 08:43
@Kailai-Wang Kailai-Wang merged commit 687ad15 into dev Sep 9, 2022
@Kailai-Wang Kailai-Wang deleted the 765-mock-IMP branch September 9, 2022 18:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jingleizhang jingleizhang Awaiting requested review from jingleizhang

@BillyWooo BillyWooo Awaiting requested review from BillyWooo

1 more reviewer

@ziming-zung ziming-zung ziming-zung approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@Kailai-Wang Kailai-Wang

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Create mock extrinsic/functions within the Parachain IMP

3 participants

@Kailai-Wang @codecov-commenter @ziming-zung