Modification Plugin-test.md

[rainbend]

#4624 (comment)

[wu-sheng]

I think more changes should be done, ref #4628

Could you go through this doc, and fix in this PR too?

[codecov-io]

Codecov Report

Merging #4706 into master will decrease coverage by 11.43%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             master    #4706       +/-   ##
=============================================
- Coverage     51.10%   39.67%   -11.44%     
+ Complexity     2690     1484     -1206     
=============================================
  Files          1284     1280        -4     
  Lines         27926    27833       -93     
  Branches       3018     3022        +4     
=============================================
- Hits          14272    11042     -3230     
- Misses        13011    16129     +3118     
- Partials        643      662       +19     

Impacted Files	Coverage Δ	Complexity Δ
...kywalking/oap/server/core/storage/AbstractDAO.java	0.00% <0.00%> (-100.00%)	0.00% <0.00%> (-2.00%)
...ywalking/apm/agent/core/plugin/EnhanceContext.java	0.00% <0.00%> (-100.00%)	0.00% <0.00%> (-5.00%)
...walking/apm/agent/core/plugin/match/NameMatch.java	0.00% <0.00%> (-100.00%)	0.00% <0.00%> (-3.00%)
...ng/oap/server/core/analysis/config/NoneStream.java	0.00% <0.00%> (-100.00%)	0.00% <0.00%> (-1.00%)
.../apm/agent/core/remote/StandardChannelBuilder.java	0.00% <0.00%> (-100.00%)	0.00% <0.00%> (-2.00%)
...server/storage/plugin/influxdb/base/RecordDAO.java	0.00% <0.00%> (-100.00%)	0.00% <0.00%> (-4.00%)
...hance/ClassInstanceMethodsEnhancePluginDefine.java	0.00% <0.00%> (-100.00%)	0.00% <0.00%> (-2.00%)
...er/storage/plugin/influxdb/base/NoneStreamDAO.java	0.00% <0.00%> (-100.00%)	0.00% <0.00%> (-4.00%)
...erver/storage/plugin/elasticsearch/base/EsDAO.java	0.00% <0.00%> (-100.00%)	0.00% <0.00%> (-4.00%)
...storage/plugin/influxdb/base/InfluxStorageDAO.java	0.00% <0.00%> (-100.00%)	0.00% <0.00%> (-4.00%)
... and 230 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4acf0eb...5b2b598. Read the comment docs.

[rainbend]

I think more changes should be done, ref #4628

Could you go through this doc, and fix in this PR too?

Updated.

[wu-sheng]

@BFergerson I can't choose you as a reviewer in the repo, could you have the 3 steps all checked on this page? https://gitbox.apache.org/setup/

[BFergerson]

@wu-sheng, these changes work too. All 3 steps are checked now

[wu-sheng]

@BFergerson Strange, I can't choose you as reviewers, and you are not a part of SkyWalking committer team. https://github.com/orgs/apache/teams/skywalking-committers/members

Could you provide a screenshot about the content at the bottom of the page, about which repos you have write access after 3 steps checked?

[BFergerson]

lgtm

[dmsolr]

LGTM

[rainbend]

@BFergerson If you think good, should submit the Approve.

[dmsolr]

Has some plugin test failed, I am sure it no matter about this update.