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

Use legacy intake pipeline by default #10120

Merged
merged 4 commits into from Nov 28, 2023
Merged

Conversation

alittley
Copy link
Contributor

Signed-off-by: Austin Littley <austin@swirldslabs.com>
Copy link

github-actions bot commented Nov 27, 2023

Node: HAPI Test (Token) Results

190 tests  ±0   188 ✔️ ±0   19m 39s ⏱️ + 1m 15s
  13 suites ±0       2 💤 ±0 
  13 files   ±0       0 ±0 

Results for commit 68e0574. ± Comparison against base commit 4b955d3.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Nov 27, 2023

Node: HAPI Test (Crypto) Results

211 tests  ±0   201 ✔️ +7   18m 40s ⏱️ + 1m 50s
  22 suites ±0     10 💤 ±0 
  22 files   ±0       0  - 7 

Results for commit 68e0574. ± Comparison against base commit 4b955d3.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Nov 27, 2023

Node: HAPI Test (Misc) Results

418 tests  ±0   286 ✔️ +5   20m 50s ⏱️ - 1m 42s
  73 suites ±0   132 💤  - 5 
  73 files   ±0       0 ±0 

Results for commit 68e0574. ± Comparison against base commit 4b955d3.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Nov 27, 2023

Node: Unit Test Results

    2 286 files  +  1      2 286 suites  +1   46m 25s ⏱️ - 6m 37s
118 389 tests +21  118 355 ✔️ +21  34 💤 ±0  0 ±0 
126 761 runs  +77  126 727 ✔️ +77  34 💤 ±0  0 ±0 

Results for commit 68e0574. ± Comparison against base commit 4b955d3.

♻️ This comment has been updated with latest results.

Copy link

codecov bot commented Nov 27, 2023

Codecov Report

Attention: 5 lines in your changes are missing coverage. Please review.

Comparison is base (4b955d3) 62.32% compared to head (1f8687d) 62.41%.
Report is 12 commits behind head on develop.

❗ Current head 1f8687d differs from pull request most recent head 68e0574. Consider uploading reports for the commit 68e0574 to get more accurate results

Files Patch % Lines
...va/com/swirlds/platform/wiring/PlatformWiring.java 16.66% 5 Missing ⚠️
Additional details and impacted files
@@              Coverage Diff              @@
##             develop   #10120      +/-   ##
=============================================
+ Coverage      62.32%   62.41%   +0.08%     
- Complexity     30392    30452      +60     
=============================================
  Files           3344     3344              
  Lines         134387   134486      +99     
  Branches       13896    13919      +23     
=============================================
+ Hits           83763    83939     +176     
+ Misses         47328    47222     -106     
- Partials        3296     3325      +29     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link

github-actions bot commented Nov 27, 2023

Node: HAPI Test (Time Consuming) Results

21 tests  ±0     9 ✔️ ±0   25m 22s ⏱️ +14s
  2 suites ±0   12 💤 ±0 
  2 files   ±0     0 ±0 

Results for commit 68e0574. ± Comparison against base commit 4b955d3.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Nov 27, 2023

Node: E2E Test Results

    1 files  ±    0      1 suites  ±0   21m 16s ⏱️ + 21m 16s
310 tests +309  310 ✔️ +310  0 💤 ±0  0  - 1 
332 runs  +331  332 ✔️ +332  0 💤 ±0  0  - 1 

Results for commit 68e0574. ± Comparison against base commit 4b955d3.

This pull request removes 1 and adds 310 tests. Note that renamed tests count towards both.
EndToEndTests ‑ initializationError
EndToEndTests ‑ ADDRESS_BOOK_CONTROLCanUpdateADDRESS_BOOK
EndToEndTests ‑ ADDRESS_BOOK_CONTROLCanUpdateNODE_DETAILS
EndToEndTests ‑ AccountsGetPayerRecordsIfSoConfigured
EndToEndTests ‑ Acct57CanMakeSmallChanges
EndToEndTests ‑ Acct57CantMakeLargeChanges
EndToEndTests ‑ AddingSignaturesToExecutedTxFails
EndToEndTests ‑ AddingSignaturesToExecutedTxFailsWithLongTermEnabled
EndToEndTests ‑ AddingSignaturesToNonExistingTxFails
EndToEndTests ‑ AddingSignaturesToNonExistingTxFailsWithLongTermEnabled
EndToEndTests ‑ AddressAliasIdFuzzing
…

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Nov 27, 2023

Node: Integration Test Results

279 tests  ±0   279 ✔️ ±0   28m 12s ⏱️ -1s
    5 suites ±0       0 💤 ±0 
    5 files   ±0       0 ±0 

Results for commit 68e0574. ± Comparison against base commit 4b955d3.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Nov 27, 2023

Node: HAPI Test (Smart Contract) Results

411 tests  +1   318 ✔️ +5   42m 47s ⏱️ +50s
  56 suites ±0     93 💤 +1 
  56 files   ±0       0  - 5 

Results for commit 68e0574. ± Comparison against base commit 4b955d3.

♻️ This comment has been updated with latest results.

Signed-off-by: Austin Littley <austin@swirldslabs.com>
lpetrovic05
lpetrovic05 previously approved these changes Nov 28, 2023
Signed-off-by: Austin Littley <austin@swirldslabs.com>
@alittley alittley merged commit 56073a2 into develop Nov 28, 2023
22 of 25 checks passed
@alittley alittley deleted the 10119-use-legacy-intake branch November 28, 2023 19:04
agadzhalov pushed a commit that referenced this pull request Dec 1, 2023
Signed-off-by: Austin Littley <austin@swirldslabs.com>
agadzhalov pushed a commit that referenced this pull request Dec 1, 2023
Signed-off-by: Austin Littley <austin@swirldslabs.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Use legacy intake pipeline by default
3 participants