Skip to content
This repository has been archived by the owner on Nov 27, 2023. It is now read-only.

fix: Add in-memory parser option (gh-826) #828

Merged
merged 1 commit into from
Aug 4, 2023
Merged

Conversation

mkralik3
Copy link
Contributor

@mkralik3 mkralik3 commented Aug 4, 2023

  • added in-memory parser option (+ config property for disabling in-memory steps if necessary)
  • extended StepCatalogTest tests
  • updated update-resources.sh (+ fix minor issues/typos )

@codecov
Copy link

codecov bot commented Aug 4, 2023

Codecov Report

Merging #828 (adde15a) into main (4be7f6e) will increase coverage by 0.22%.
The diff coverage is 72.72%.

@@             Coverage Diff              @@
##               main     #828      +/-   ##
============================================
+ Coverage     70.34%   70.56%   +0.22%     
  Complexity      199      199              
============================================
  Files            69       69              
  Lines          3332     3340       +8     
  Branches        557      558       +1     
============================================
+ Hits           2344     2357      +13     
+ Misses          771      766       -5     
  Partials        217      217              
Files Changed Coverage Δ
...aoto/backend/api/metadata/catalog/StepCatalog.java 76.66% <62.50%> (+2.12%) ⬆️
...rser/step/camelroute/CamelRestDSLParseCatalog.java 97.52% <100.00%> (+0.02%) ⬆️
...parser/step/camelroute/CamelRouteParseCatalog.java 92.30% <100.00%> (+0.64%) ⬆️
...adata/parser/step/kamelet/KameletParseCatalog.java 90.47% <100.00%> (+0.47%) ⬆️

... and 2 files with indirect coverage changes

@mkralik3 mkralik3 force-pushed the gh-826 branch 3 times, most recently from a67957a to 8db8f17 Compare August 4, 2023 12:09
@sonarcloud
Copy link

sonarcloud bot commented Aug 4, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@mkralik3 mkralik3 marked this pull request as ready for review August 4, 2023 12:34
@mkralik3 mkralik3 requested a review from a team August 4, 2023 12:34
@Delawen Delawen enabled auto-merge (rebase) August 4, 2023 12:37
@Delawen Delawen merged commit b52a5ea into KaotoIO:main Aug 4, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] Camel REST DSL steps are not loaded when application.yaml contains only git repo or no repo
2 participants