Skip to content

Yaml: Add yaml syntax UT#2872

Merged
Abacn merged 11 commits intoGoogleCloudPlatform:mainfrom
derrickaw:yamlSyntaxUT
Oct 24, 2025
Merged

Yaml: Add yaml syntax UT#2872
Abacn merged 11 commits intoGoogleCloudPlatform:mainfrom
derrickaw:yamlSyntaxUT

Conversation

@derrickaw
Copy link
Contributor

@derrickaw derrickaw commented Oct 10, 2025

  1. Adding yaml syntax UT to verify that the yaml blueprints placed in the yaml folder are syntactically correct, since there will be occasions where we do not have 1:1 mappings with yaml templates.
  2. mvn clean test -pl yaml -am

@derrickaw derrickaw added addition New feature or request unit testing java Pull requests that update Java code yaml Pull requests that update Yaml code improvement Making existing code better labels Oct 10, 2025
@codecov
Copy link

codecov bot commented Oct 10, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 50.04%. Comparing base (6381f56) to head (ed510be).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff              @@
##               main    #2872      +/-   ##
============================================
- Coverage     50.06%   50.04%   -0.02%     
- Complexity     4972     5348     +376     
============================================
  Files           967      967              
  Lines         59329    59329              
  Branches       6448     6448              
============================================
- Hits          29702    29691      -11     
- Misses        27522    27533      +11     
  Partials       2105     2105              
Components Coverage Δ
spanner-templates 70.44% <ø> (-0.05%) ⬇️
spanner-import-export 68.94% <ø> (-0.13%) ⬇️
spanner-live-forward-migration 79.69% <ø> (ø)
spanner-live-reverse-replication 77.42% <ø> (ø)
spanner-bulk-migration 88.21% <ø> (ø)
see 6 files with indirect coverage changes
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@derrickaw
Copy link
Contributor Author

Elastic search IT failure is unrelated

@derrickaw derrickaw marked this pull request as ready for review October 15, 2025 14:20
@derrickaw derrickaw requested a review from Abacn October 24, 2025 13:10
@Abacn Abacn merged commit fd1e0ff into GoogleCloudPlatform:main Oct 24, 2025
42 checks passed
@derrickaw derrickaw deleted the yamlSyntaxUT branch October 24, 2025 17:04
aasthabharill pushed a commit that referenced this pull request Dec 10, 2025
* add jinjava library

* add dependency for ut

* clean up formatting

* add yaml syntaxt ut

* fix pom conflict

* add python unit test for yaml to replace java based

* remove java yaml ut

* remove old dependencies

* remove jinjava

* remove comment

* fix comment about test case generation
MnkyGns pushed a commit to MnkyGns/DataflowTemplates that referenced this pull request Feb 12, 2026
* add jinjava library

* add dependency for ut

* clean up formatting

* add yaml syntaxt ut

* fix pom conflict

* add python unit test for yaml to replace java based

* remove java yaml ut

* remove old dependencies

* remove jinjava

* remove comment

* fix comment about test case generation
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

addition New feature or request improvement Making existing code better java Pull requests that update Java code size/L unit testing yaml Pull requests that update Yaml code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants