coveralls
commented
1 week ago Pull Request Test Coverage Report for Build 9673447216
Details
- 53 of 64 (82.81%) changed or added relevant lines in 1 file are covered.
- 135 unchanged lines in 1 file lost coverage.
- Overall coverage decreased (-0.001%) to 73.049%
| Changes Missing Coverage | Covered Lines | Changed/Added Lines | % | ||
|---|---|---|---|---|---|
| contrib/babelfishpg_tsql/src/procedures.c | 53 | 64 | 82.81% | ||
| <!-- | Total: | 53 | 64 | 82.81% | --> |
| Files with Coverage Reduction | New Missed Lines | % | ||
|---|---|---|---|---|
| contrib/babelfishpg_tsql/src/procedures.c | 135 | 88.69% | ||
| <!-- | Total: | 135 | --> |
| Totals | |
|---|---|
| Change from base Build 9646270885: | -0.001% |
| Covered Lines: | 42277 |
| Relevant Lines: | 57875 |
Description
In our previous attempt of fixing sp_describe_undeclared_parameters related crash, we missed some corner cases where few variables will be left NULL if is_sp_describe_undeclared_parameters is marked as false. Then when we try to access those variables, we get seg fault. In this commit, we fix it by adding extra check on is_sp_describe_undeclared_parameters.
Issues Resolved
BABEL-5068 Signed-off-by: Shameem Ahmed shmeeh@amazon.com
Test Scenarios Covered
Use case based -
Boundary conditions -
Arbitrary inputs -
Negative test cases -
Minor version upgrade tests -
Major version upgrade tests -
Performance tests -
Tooling impact -
Client tests -
Check List
By submitting this pull request, I confirm that my contribution is under the terms of the Apache 2.0 and PostgreSQL licenses, and grant any person obtaining a copy of the contribution permission to relicense all or a portion of my contribution to the PostgreSQL License solely to contribute all or a portion of my contribution to the PostgreSQL open source project.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.