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

Gno code coverage and plan improve it. #56

Open
AnhVAR opened this issue Jan 18, 2024 · 4 comments
Open

Gno code coverage and plan improve it. #56

AnhVAR opened this issue Jan 18, 2024 · 4 comments

Comments

@AnhVAR
Copy link

AnhVAR commented Jan 18, 2024

Gno code quality status

We have analyzed and executed all the existing test cases for the Gno repository to evaluate the current status of the project. We observed that the average code coverage for the project is around 60%. There are several directories that either have not been tested or lack test code.
Screenshot 2024-01-17 at 22 18 29

Source code have many issues detected by static analysis tool.
Screenshot 2024-01-17 at 22 17 40

Based on our analysis, we believe that to enhance the quality of the project's source code, we can contribute the following actions:

  1. Perform static testing for any code changes.
  2. Fix bugs identified by static code analysis.
  3. Write integration and unit test scripts to improve code coverage.
  4. Set up continuous integration and continuous delivery (CI/CD) for the repository to ensure that test cases pass with each new code update.
  5. Create Regression testing plan
@Nhatran94
Copy link

Nhatran94 commented Jan 18, 2024

@dongwon8247 @moul @michelleellen pls take a look

@notJoon
Copy link
Member

notJoon commented Jan 18, 2024

I'm gradually adding tests while working on the codebase. I definitely agree that a major improvement is needed.

@zivkovicmilos
Copy link
Member

Hey @AnhVAR,

Thank you for opening up the issue 🙏

We are well aware of the coverage problem, and have internally dedicated a significant chunk of our time to tackling this problem, especially for critical code segments. Additionally, we've started setting up procedures and tooling with our security engineer @kristovatlas, so there are no show-stopping bugs.

I suggest we keep this issue open as an ongoing thread on the coverage progress

@thehowl
Copy link
Member

thehowl commented Jan 31, 2024

See-also gnolang/gno#1215

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

No branches or pull requests

5 participants