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

feat: presentation_submission validation logic #1332

Merged
merged 7 commits into from
Sep 12, 2024
Merged

Conversation

patlo-iog
Copy link
Contributor

@patlo-iog patlo-iog commented Sep 9, 2024

Description:

ATL-7645

  • add PresentationSubmission model
  • add PresentationSubmissionVerification and tets

The PR looks big, but half of the LOC are just test 😄

Copy link
Contributor

github-actions bot commented Sep 9, 2024

Integration Test Results

20 files  ±0  20 suites  ±0   2s ⏱️ ±0s
45 tests ±0  45 ✅ ±0  0 💤 ±0  0 ❌ ±0 
71 runs  ±0  71 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit 4cd5ac1. ± Comparison against base commit d29eebb.

♻️ This comment has been updated with latest results.

Copy link
Contributor

github-actions bot commented Sep 9, 2024

Unit Test Results

100 files  + 2  100 suites  +2   19m 29s ⏱️ +18s
857 tests +19  849 ✅ +19  8 💤 ±0  0 ❌ ±0 
864 runs  +19  856 ✅ +19  8 💤 ±0  0 ❌ ±0 

Results for commit 4cd5ac1. ± Comparison against base commit d29eebb.

♻️ This comment has been updated with latest results.

Signed-off-by: Pat Losoponkul <[email protected]>

chore: only use claim format value defined in contract

Signed-off-by: Pat Losoponkul <[email protected]>

fix: adapt verification interface

Signed-off-by: Pat Losoponkul <[email protected]>

wip

Signed-off-by: Pat Losoponkul <[email protected]>

feat: ps submission entry processing

Signed-off-by: Pat Losoponkul <[email protected]>

test: sanity check

Signed-off-by: Pat Losoponkul <[email protected]>

wip: verify input constraint

test: matching descriptor and submission

Signed-off-by: Pat Losoponkul <[email protected]>

test: basic matching descriptor

Signed-off-by: Pat Losoponkul <[email protected]>

test: submission id not match

Signed-off-by: Pat Losoponkul <[email protected]>

test: pd unique id test

Signed-off-by: Pat Losoponkul <[email protected]>

test: moar test!!

style: fix and fmt

Signed-off-by: Pat Losoponkul <[email protected]>

wip

test: cleanup testing

tests: nested_path test and multiple descriptors

Signed-off-by: Pat Losoponkul <[email protected]>

test: optional field constraints test

Signed-off-by: Pat Losoponkul <[email protected]>
Pat Losoponkul added 2 commits September 10, 2024 15:21
Signed-off-by: Pat Losoponkul <[email protected]>
Signed-off-by: Pat Losoponkul <[email protected]>
@patlo-iog patlo-iog force-pushed the feat/prex-ps-1 branch 3 times, most recently from e1861a3 to 0b9193a Compare September 10, 2024 08:54
Signed-off-by: Pat Losoponkul <[email protected]>
@coveralls
Copy link

coveralls commented Sep 11, 2024

Coverage Status

coverage: 48.799% (+0.3%) from 48.488%
when pulling 4cd5ac1 on feat/prex-ps-1
into d29eebb on main.

@patlo-iog patlo-iog marked this pull request as ready for review September 11, 2024 08:31
@patlo-iog patlo-iog requested a review from a team as a code owner September 11, 2024 08:31
@patlo-iog patlo-iog changed the title feat: presentation_submission validation logic [WIP] feat: presentation_submission validation logic Sep 11, 2024
Copy link
Member

@yshyn-iohk yshyn-iohk left a comment

Choose a reason for hiding this comment

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

🔥

@patlo-iog patlo-iog enabled auto-merge (squash) September 12, 2024 05:21
@patlo-iog patlo-iog merged commit f80b3c3 into main Sep 12, 2024
13 checks passed
@patlo-iog patlo-iog deleted the feat/prex-ps-1 branch September 12, 2024 05:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants