Ftr:integrate test #535
Ftr:integrate test #535
Conversation
Codecov Report
📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more @@ Coverage Diff @@
## master #535 +/- ##
==========================================
- Coverage 38.64% 36.65% -1.99%
==========================================
Files 152 160 +8
Lines 10250 11133 +883
==========================================
+ Hits 3961 4081 +120
- Misses 5963 6705 +742
- Partials 326 347 +21 see 66 files with indirect coverage changes Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
.github/workflows/integrate-test.yml
Outdated
| strategy: | ||
| matrix: | ||
| golang: | ||
| - 1.19 |
There was a problem hiding this comment.
项目中用的 1.18，最好统一一下
已改
.github/workflows/integrate-test.yml
Outdated
| - name: "set up go" | ||
| uses: actions/setup-go@v3 | ||
| with: | ||
| go-version: 1.19 |
integrate_test.sh
Outdated
| echo "github pull request head branch -> ${GITHUB_HEAD_REF}" | ||
|
|
||
| echo "use seata-go-samples $3 branch for integration testing" | ||
| git clone -b ftr/integrate_test https://github.com/seata/seata-go-samples samples && cd samples |
There was a problem hiding this comment.
这里的 ftr/integrate_test 分支未来是否还会存在？
There was a problem hiding this comment.
这个搞错了，已经删掉了 直接master就可以了
|
samples那边已经合并了，但是这个项目跑好像还有点问题： |
It seems that there is still a problem: |
这个命令是直接直接的shell吗 少了两个参数。把这个action重新触发下吧，看看能不能run起来 |
Is this order a direct shell? Two parameters are missing. Re-trigger this action to see if it can run |
feat add integrate_test
What this PR does:
集成测试的脚本，自动加载sample下面的integrate_test，检测当前代码是否有问题
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?: