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

generate report for api test #9787

Merged
merged 1 commit into from
Aug 13, 2024
Merged

generate report for api test #9787

merged 1 commit into from
Aug 13, 2024

Conversation

ScharfViktor
Copy link
Contributor

@ScharfViktor ScharfViktor commented Aug 12, 2024

related #9730

report for api test:
how to generate: MONTH=08 YEAR=2024 go run tests/qa-activity-report/generate-qa-activity-report-finish.go
how to generate using MAKE: make generate-qa-activity-report MONTH=08 YEAR=2024

generated report:

Test-Type,Date,Tests Added,Tests Changed,Tests Deleted,commit-ID
API Test,2024-08-07,2,0,0,610731f4937446dc9bb07c216e7f06e3a75c52b3
API Test,2024-08-06,2,0,0,8113997a84e5676d4b110fd9f9b813f2bd6d379b
API Test,2024-06-24,3,1,1,b1e55a2853a6ef7c41ac909dda38402a02cbdee0
API Test,2024-08-05,13,0,5,558bbcc6aefa894ff60b3b6899570586c97bdee3
API Test,2024-08-05,0,3,0,62c024bc667b7ebb1b568b45cc6745f39806ae8e
API Test,2024-08-05,0,2,0,903131f76782a3e0251c16cbc75bcc67e3f96ca7
API Test,2024-07-30,0,3,0,f976386ee2e7c9ce8aee06817d655ad565876d37
API Test,2024-07-29,7,0,0,d8c74a30a2ea6c3c8ea2ed7e2f9e60479302d257

git log --since='2024-08-01' --until='2024-08-30 23:59:59' --format="%H" -- tests/acceptance/features/ | grep -v 'tests/acceptance/features/bootstrap/'

output:

610731f4937446dc9bb07c216e7f06e3a75c52b3
8113997a84e5676d4b110fd9f9b813f2bd6d379b
b1e55a2853a6ef7c41ac909dda38402a02cbdee0
558bbcc6aefa894ff60b3b6899570586c97bdee3
62c024bc667b7ebb1b568b45cc6745f39806ae8e
903131f76782a3e0251c16cbc75bcc67e3f96ca7
f976386ee2e7c9ce8aee06817d655ad565876d37
d8c74a30a2ea6c3c8ea2ed7e2f9e60479302d257

and compare:
git show commitId

Copy link

update-docs bot commented Aug 12, 2024

Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes.

Copy link
Member

@saw-jan saw-jan left a comment

Choose a reason for hiding this comment

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

LGTM (I'm not a go guy though 😅 )

Copy link
Collaborator

@kobergj kobergj left a comment

Choose a reason for hiding this comment

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

couldn't resist reviewing. sorry 🙈

tests/qa-activity-report/generate-qa-activity-report.go Outdated Show resolved Hide resolved
tests/qa-activity-report/generate-qa-activity-report.go Outdated Show resolved Hide resolved
tests/qa-activity-report/generate-qa-activity-report.go Outdated Show resolved Hide resolved
tests/qa-activity-report/generate-qa-activity-report.go Outdated Show resolved Hide resolved
tests/qa-activity-report/generate-qa-activity-report.go Outdated Show resolved Hide resolved
@ScharfViktor ScharfViktor requested a review from kobergj August 13, 2024 13:28
@ScharfViktor
Copy link
Contributor Author

@kobergj thank you for review 👍
I fixed it, can you pleaes have look it again?

Copy link
Collaborator

@kobergj kobergj left a comment

Choose a reason for hiding this comment

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

Looks almost perfect. But you need to explain to QA why we don't have unit tests 😉

tests/qa-activity-report/generate-qa-activity-report.go Outdated Show resolved Hide resolved
tests/qa-activity-report/generate-qa-activity-report.go Outdated Show resolved Hide resolved
@ScharfViktor
Copy link
Contributor Author

Looks almost perfect. But you need to explain to QA why we don't have unit tests 😉

let make it in different PR. because I don't know how to do it yet 😆

Copy link
Collaborator

@kobergj kobergj left a comment

Choose a reason for hiding this comment

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

Nice 👍 When can you start as golang developer? I have some more tasks for you 😉

Copy link

@ScharfViktor
Copy link
Contributor Author

ScharfViktor commented Aug 13, 2024

Nice 👍 When can you start as golang developer? I have some more tasks for you 😉

I’m all in for if you will review my noodle code 😉
thanks Julian 😍

@ScharfViktor ScharfViktor merged commit 99b301e into master Aug 13, 2024
4 checks passed
@ScharfViktor ScharfViktor deleted the QA_report_API_tests branch August 13, 2024 14:46
ownclouders pushed a commit that referenced this pull request Aug 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants