Fix/tests idempotent no truncate metrics mt#128
Open
RahulRengeshOfficial wants to merge 198 commits into
Open
Conversation
Referring xconfwebconfig develop to xconfadmin develop and Resyncing
Referencing github path for xconfadmin
Updated Makefile
Reverting older fixes for UT
Update Makefile
Fix build version and update go.mod to use latest version of xconfwebconfig
…d fix response for ip address group delete api
wakeuppool api creation changes
…improve error handling
Refactor imports and enhance wakeup pool API
Refactor SAT Token Validation Logic
Improve MAC list reference validation error handling in firmware rules
…s and percentage beans - baserule_validator.go: validate IP namespaced list existence when operation is IN_LIST and free arg is IP_ADDRESS/EstbIp - baserule_validator.go: validate model existence when operation is IS and free arg is MODEL - percentage_bean_service.go: introduce validatePercentageBeanReferences() to check model, whitelist IP list, and optional conditions - percentage_bean_service.go: call validatePercentageBeanReferences() in both CreatePercentageBean and UpdatePercentageBean
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
- baserule_validator_test.go: add tests for IN_LIST on IP_ADDRESS/EstbIp rejecting missing IP lists - baserule_validator_test.go: add tests for IS on MODEL rejecting unknown models - percentage_bean_service_test.go: add tests for validatePercentageBeanReferences covering invalid/valid model, missing/valid IP list, blank whitelist, and invalid optional conditions - DB-write tests guarded with SkipIfMockDatabase for CI compatibility
…list-references Add model and IP address list reference validation for rule conditions
…ential execution - Add SkipIfMockDatabase() helper to skip tests that require real Cassandra - Use unique IDs (timestamps) to prevent test collisions - Add time.Sleep for Cassandra eventual consistency - Update Makefile to run tests sequentially (-p 1) to avoid DB race conditions - Tests now pass reliably in both mock and real DB modes
Fix flaky tests: add SkipIfMockDatabase pattern, unique IDs, and sequ…
- Adapt DAO calls to include tenantId parameter (multi-tenant API) - Change xhome to rdkcloud (xhome invalid on multi-tenant) - Use mock HTTP server for GroupServiceConnector tests - Fix table name constants (singular to plural) - Skip/fix tests with pre-existing multi-tenant failures
Contributor
- Add contrib/scripts/run_tests_with_summary.sh for JSON test logging and metrics - Update Makefile test target to use the new script - Ported from PR #124 (fix/tests-idempotent-no-truncate-metrics)
Contributor
Contributor
Contributor
|
b'## WARNING: A Blackduck scan failure has been waived A prior failure has been upvoted
|
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.
No description provided.