Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Test: print the diff list when validation failed #46

Merged
merged 1 commit into from
Jul 21, 2020

Conversation

kezhenxu94
Copy link
Member

This patch makes it more convenient for the developers when running tests locally and the tests failed

image

@kezhenxu94 kezhenxu94 added enhancement New feature or request test test labels Jul 20, 2020
@kezhenxu94 kezhenxu94 added this to the 0.2.0 milestone Jul 20, 2020
@kezhenxu94 kezhenxu94 requested a review from wu-sheng July 20, 2020 15:54
@kezhenxu94
Copy link
Member Author

kezhenxu94 commented Jul 20, 2020

@alonelaval if you're interested in, feel free to review

@alonelaval
Copy link
Member

This patch makes it more convenient for the developers when running tests locally and the tests failed

image

👌

@wu-sheng
Copy link
Member

Should I wait for @alonelaval feedback.

Copy link
Member

@alonelaval alonelaval left a comment

Choose a reason for hiding this comment

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

LGTM,sava my time @kezhenxu94

@kezhenxu94 kezhenxu94 merged commit d3c13bf into master Jul 21, 2020
@kezhenxu94 kezhenxu94 deleted the test/diff-when-failed branch July 21, 2020 13:38
kezhenxu94 pushed a commit that referenced this pull request Aug 3, 2020
author zkscpqm <qmcpzks@naver.com> 1595084266 +0300
committer kezhenxu94 <kezhenxu94@163.com> 1596458512 +0800
gpgsig -----BEGIN PGP SIGNATURE-----

 iQIzBAABCAAdFiEE/Eo3KEa/qP7Oqvu/Ng+pfz8mM0wFAl8oBhAACgkQNg+pfz8m
 M0y/bBAAoBBwE54/GJ00JQ8tgn5xqNo1bfqlBcNa6obvXJbkvFwt/knVEU+fuKxm
 p/jGahr2EyHzhXTh4XUo5xxlK+xgTid5JHRopjgTY3MgbEG/397tcS2+1P8NQUCB
 Lh+OxX5V4Gm0GqvFP02zrsHjBiJYL1TVz4R8PomuE+w7fuNyuqPtNMUt5ILsZz/3
 X3X7JJnbJqjKK5fEcerPKPUL3Ee+muazoCuR5KFGJ6PbxrVu/YBG3UTTeOhVonOb
 r7VKicGJSKQZD3nCH8cE56Qwt3uoQE1W0lQ/L1RwE++ixa+vk19FpPlwoup/eYac
 vYTGjMSgloCpiksGmhLhPU5bOlMxoArqJQg2zHayPfrdFNfgu1qfu+fMciU+8A11
 OxhLEjLxAnlEO7wtjcUuVD+siVlmkKeRBxffsoiZTXWFQVbbNnM9Sgrc70ql9KwK
 uCi3Bcfr2gJctC+vrHEBO2AJX4HtSuhYtpGzKAPc0SHb2niKqqFFZHWa+t5db7zg
 ixxUrT5j9OSAi4TzDi4/QLEjCt0ECOxJJpeUFMXn4SLHYiSaU5lgV+H5VOxEKotj
 w3dpHU1J1/mylZ9TauFX703vIc/il8iqKeqc0JFgSa25F0ZjlyfI8MIkwfHA8CNS
 MTMjnwmcDpOgc/AADPqAz+kwLYLARlvBUkz5uhwqI8slXkpwnrY=
 =4v88
 -----END PGP SIGNATURE-----

Created venv builders for linux/windows and req flashers + use documentation (#38)

* Created venv builders for linux/windows and req flashers + use documentation

* aadd license headers to scripts

* moved scripts, updated paths, changed how reqs are installed, updated linux venv bin path

* update license checker to accomodate for shebangs and batchfile comments

Add ability to collect SQL Parameters in PyMsql plugin (#39)

Add plugin for Django Agent (#37)

Add ignore_suffix Config (#40)

Add Django in document (#42)

Add Flask Http Params (#43)

* Add Flask Http Params

* Add Flask Http Params

Co-authored-by: huawei <huawei@bit-s.cn>

Add Redis Plugin (#44)

Update Redis plugin Component Id (#45)

Test: print the diff list when validation failed (#46)

feature: add Tornado Plugin (#48)

feature: add Kafka Plugin (#50)

Enable Django collect http parameters (#49)

Add changelog for 0.2.0 (#51)

Make plugin compatible with Django < 2.2 (#52)

Add Rabbitmq Plugin (#53)

chore: add make goal to package release tar ball (#54)

Support propagating correlation context (#55)

Sync

Add changelog for 0.2.0 (#51)

Make plugin compatible with Django < 2.2 (#52)

Add Rabbitmq Plugin (#53)

chore: add make goal to package release tar ball (#54)

Support propagating correlation context (#55)

Sync
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request test test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants