test: add automatic table cleanup after go sdk tests#3799
Merged
tobegit3hub merged 1 commit into4paradigm:mainfrom Mar 12, 2024
Merged
test: add automatic table cleanup after go sdk tests#3799tobegit3hub merged 1 commit into4paradigm:mainfrom
tobegit3hub merged 1 commit into4paradigm:mainfrom
Conversation
Contributor
SDK Test Report101 files ±0 101 suites ±0 2m 5s ⏱️ -8s Results for commit eb83f3d. ± Comparison against base commit 42f9b89. This pull request removes 30 and adds 9 tests. Note that renamed tests count towards both. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #3799 +/- ##
============================================
+ Coverage 40.28% 40.31% +0.02%
Complexity 658 658
============================================
Files 194 194
Lines 11436 11436
Branches 1380 1380
============================================
+ Hits 4607 4610 +3
+ Misses 6525 6522 -3
Partials 304 304 ☔ View full report in Codecov by Sentry. |
Contributor
Linux Test Report 53 files ±0 60 suites ±0 27m 58s ⏱️ -3s For more details on these failures, see this check. Results for commit eb83f3d. ± Comparison against base commit 42f9b89. |
aceforeverd
approved these changes
Mar 12, 2024
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What kind of change does this PR introduce?
This PR introduces a test enhancement in the Go SDK. It adds automatic cleanup logic for database table created during tests, ensuring that tests can be rerun without the need for manual table deletion.
What is the current behavior? (You can also link to an open issue here)
Currently, the tests in the Go SDK create demo table in the database to verify various functionalities. However, it's not automatically cleaned up after the tests are completed. This behavior requires manual intervention to delete the table before tests can be rerun, leading to inefficiencies in the development process.
What is the new behavior (if this is a feature change)?
With this change, the Go SDK tests now include cleanup logic that automatically deletes demo table created during the testing process. This ensures that the database is restored to its initial state after the tests complete, allowing for tests to be rerun multiple times without manual cleanup. This improvement streamlines the testing process, making it more efficient and less error-prone.