Rename this field "totalCount" to match the regular expression ^[_a-z][_a-z0-9]*$

[anirudhprmar]

Describe the bug
So in backend/tests/apps/github/management/commands/github_sync_user_test.py. There is field named as totalCount, it will be good to use camelCase expression if the backend was in javascript but its in python and according to python's naming convention using snake_case which will support the required regular expression ^[_a-z][_a-z0-9]*$ & will be more efficient. The camelCased totalCount violates this rule.

To Reproduce
Steps to reproduce the behavior:

1. Go to 'backend/tests/apps/github/management/commands/github_sync_user_test.py'
2. rename totalCount to total_count

Expected behavior
A clear and concise description of what you expected to happen.

Are you going to work on fixing this?

• Yes
• No

Additional Context:
https://sonarcloud.io/project/issues?impactSeverities=LOW&issueStatuses=OPEN%2CCONFIRMED&id=OWASP_Nest&open=AZp-i-mzHhyenAko7tNZ

[anirudhprmar]

Hi @arkid15r , @kasya I would love to solve this issue. Can i get assigned to fix it.

[anirudhprmar]

here's the link for this finding

[arkid15r]

here's the link for this finding

@anirudhprmar this one is a false positive -- see https://github.com/PyGithub/PyGithub/blob/61dcf49d30d6854849ab745c501cab8856267bdb/github/PaginatedList.py#L235

Please add # NOSONAR for that line to make Sonar happy

[Raibipasha-24]

@arkid15r @kasya Can I fix this?

[arkid15r]

The reporter wanted to work on it so it's theirs.

@Raibipasha-24 you can go over Sonar's findings and create similar issue(s) if you want.

[Raibipasha-24]

ok