Skip to content

Add CI test for python API#6636

Merged
lenboo merged 7 commits intoapache:devfrom
zhongjiajie:py-ci
Nov 4, 2021
Merged

Add CI test for python API#6636
lenboo merged 7 commits intoapache:devfrom
zhongjiajie:py-ci

Conversation

@zhongjiajie
Copy link
Copy Markdown
Member

@zhongjiajie zhongjiajie commented Oct 29, 2021 •
edited
Loading

Using pytest as unit test for DolphinScheduler Python API, and integrated to CI

@zhongjiajie zhongjiajie changed the title Add python API unittest to GA [WIP][CI] Add python API unittest to GA Oct 30, 2021
@zhongjiajie zhongjiajie force-pushed the py-ci branch 2 times, most recently from b5e12f7 to 098a061 Compare November 1, 2021 10:03
@zhongjiajie zhongjiajie changed the title [WIP][CI] Add python API unittest to GA [WIP][CI] Add CI for python API Nov 1, 2021
@zhongjiajie zhongjiajie force-pushed the py-ci branch 2 times, most recently from 618d95c to faae7bf Compare November 1, 2021 11:06
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Nov 1, 2021 •
edited
Loading

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 38.47%. Comparing base (f204447) to head (aaa3599).
⚠️ Report is 3542 commits behind head on dev.

Additional details and impacted files 
@@             Coverage Diff              @@
##                dev    #6636      +/-   ##
============================================
- Coverage     38.48%   38.47%   -0.01%     
+ Complexity     3225     3223       -2     
============================================
  Files           646      646              
  Lines         25886    25886              
  Branches       2808     2808              
============================================
- Hits           9961     9959       -2     
  Misses        15013    15013              
- Partials        912      914       +2

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

🚀 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.

@zhongjiajie zhongjiajie changed the title [WIP][CI] Add CI for python API Add CI for python API Nov 1, 2021
@zhongjiajie zhongjiajie mentioned this pull request Nov 1, 2021
Closed
30 tasks
@zhongjiajie
Copy link
Copy Markdown
Member Author

zhongjiajie commented Nov 1, 2021

It's ready to go now, PTAL @lenboo @CalvinKirs

@zhongjiajie zhongjiajie changed the title Add CI for python API Add CI test for python API Nov 1, 2021
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Nov 2, 2021

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
No Duplication information No Duplication information

CalvinKirs
CalvinKirs approved these changes Nov 2, 2021
View reviewed changes
Copy link
Copy Markdown
Member

@CalvinKirs CalvinKirs left a comment

Choose a reason for hiding this comment

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

LGTM

lenboo
lenboo approved these changes Nov 4, 2021
View reviewed changes
Copy link
Copy Markdown
Contributor

@lenboo lenboo left a comment

Choose a reason for hiding this comment

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

+1

@lenboo lenboo merged commit 2e70365 into apache:dev Nov 4, 2021
@zhongjiajie zhongjiajie deleted the py-ci branch November 4, 2021 03:33
@zhongjiajie
Copy link
Copy Markdown
Member Author

zhongjiajie commented Nov 4, 2021

Thanks 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@CalvinKirs CalvinKirs CalvinKirs approved these changes

@lenboo lenboo lenboo approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@zhongjiajie @codecov-commenter @CalvinKirs @lenboo