Skip to content

tests(): enable tests + flaky#679

Merged
Malaydewangan09 merged 6 commits intomainfrom
feat/tests
Jan 16, 2026
Merged

tests(): enable tests + flaky#679
Malaydewangan09 merged 6 commits intomainfrom
feat/tests

Conversation

@Malaydewangan09
Copy link
Member

@Malaydewangan09 Malaydewangan09 commented Jan 15, 2026 •
edited
Loading

@github-project-automation github-project-automation bot moved this to To review in Pull Requests Jan 15, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Jan 15, 2026 •
edited
Loading

🧪 Java Unit Tests

TestsPassed ✅Skipped ⚠️FailedTime ⏱
Java Tests Report94 ran93 ✅1 ⚠️0 ❌3m 45s 549ms

📦 Artifacts

Name Size Updated Expiration
jar 60.22 MB Jan 16, 26, 7:30:31 AM UTC Jan 23, 26, 7:30:27 AM UTC

🛡 Trivy

Vulnerability in: Java

Vulnerability Severity Package Installed Version Fixed Version
CVE-2025-67735 MEDIUM io.netty:netty-codec-http 4.2.5.Final 4.2.8.Final, 4.1.129.Final

🔁 Unreleased Commits

✅ No unreleased commits found.

@Malaydewangan09 Malaydewangan09 changed the title tests(): enable Tests tests(): enable tests Jan 15, 2026
@Malaydewangan09 Malaydewangan09 changed the title tests(): enable tests tests(): enable tests + flaky Jan 16, 2026
@Malaydewangan09 Malaydewangan09 self-assigned this Jan 16, 2026
@fdelbrayelle fdelbrayelle mentioned this pull request Jan 16, 2026
Merged
28 tasks
fdelbrayelle
fdelbrayelle reviewed Jan 16, 2026
View reviewed changes
fdelbrayelle
fdelbrayelle reviewed Jan 16, 2026
View reviewed changes
fdelbrayelle
fdelbrayelle reviewed Jan 16, 2026
View reviewed changes
Copy link
Member

@fdelbrayelle fdelbrayelle left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Why did you delete GetJobRunIntegrationTest.java‎ and others?

Malaydewangan09 and others added 3 commits January 16, 2026 12:53
@Malaydewangan09 @fdelbrayelle
Update src/main/java/io/kestra/plugin/aws/emr/SubmitSteps.java
76ac2f5 
Co-authored-by: François Delbrayelle <fdelbrayelle@gmail.com>
@Malaydewangan09 @fdelbrayelle
Update src/main/java/io/kestra/plugin/aws/emr/DeleteCluster.java
442bc8c 
Co-authored-by: François Delbrayelle <fdelbrayelle@gmail.com>
@Malaydewangan09 @fdelbrayelle
Update src/main/java/io/kestra/plugin/aws/emr/CreateClusterAndSubmitS…
87d0c09 
…teps.java

Co-authored-by: François Delbrayelle <fdelbrayelle@gmail.com>
@Malaydewangan09
Copy link
Member Author

Malaydewangan09 commented Jan 16, 2026

Replaced them with mock tests, else we needed to setup glue environment in aws!

@Malaydewangan09 Malaydewangan09 merged commit b3e5442 into main Jan 16, 2026
7 checks passed
@Malaydewangan09 Malaydewangan09 deleted the feat/tests branch January 16, 2026 07:40
@github-project-automation github-project-automation bot moved this from To review to Done in Pull Requests Jan 16, 2026
@github-actions github-actions bot mentioned this pull request Jan 19, 2026
Merged
@shivamwayal37 shivamwayal37 mentioned this pull request Jan 24, 2026
Merged
1 task
This was referenced Jan 26, 2026
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@fdelbrayelle fdelbrayelle fdelbrayelle approved these changes

Assignees

@Malaydewangan09 Malaydewangan09

Labels

None yet

Projects

Pull Requests
Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Malaydewangan09 @fdelbrayelle