Develop by RahulRengeshOfficial · Pull Request #126 · rdkcentral/xconfwebconfig

RahulRengeshOfficial · 2026-03-11T10:19:16Z

No description provided.

- Increase coverage for estb_firmware_handler.go functions - GetCheckMinFirmwareHandler: 71.9% → 100% (+28.1%) - GetEstbFirmwareVersionInfoPath: 69.7% → 100% (+30.3%) - Added comprehensive edge case tests for: * IP parsing paths (query param vs body) * Query parameter filtering (clientProtocol, cert expiry) * Body parameter parsing * Security token manager code path * Cert expiry handling * Multi-value query parameters * Application type extraction * MAC normalization Impact: - Main dataapi package: 51.1% → 52.2% (+1.1%) - Overall coverage: 38.2% → 38.9% (+0.7%) - 20 new test functions, all passing - Proper mocking to avoid DB/service dependencies Files: - dataapi/estb_firmware_handler_test.go: +20 tests - PROGRESS_SUMMARY_ESTB_FIRMWARE_HANDLER.md: detailed analysis

…ndling

new flag to determine whether to add security token if no new fw offered

…ctions

Patch 1

Removed added debug logs

Copilot

Copilot wasn't able to review this pull request because it exceeds the maximum number of lines (20,000). Try reducing the number of changed lines and requesting a review from Copilot again.

…te function

Current behavior is consistent with existing logic where unknown-account detection uses the same utility check.. Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

…tions and tests

…tion

…ance error handling for partner route requests

Add condition enable_fw_penetration_metrics flag

Copilot

Copilot wasn't able to review this pull request because it exceeds the maximum number of lines (20,000). Try reducing the number of changed lines and requesting a review from Copilot again.

…into feature/foxtelMigration

…sage of ContextMap

Feature/foxtel migration

Copilot

Copilot wasn't able to review this pull request because it exceeds the maximum number of lines (20,000). Try reducing the number of changed lines and requesting a review from Copilot again.

…eature/foxtelMigration

Feature/foxtel migration

Copilot

Copilot wasn't able to review this pull request because it exceeds the maximum number of lines (20,000). Try reducing the number of changed lines and requesting a review from Copilot again.

RahulRengeshOfficial and others added 30 commits October 28, 2025 19:10

Create comparing_evaluator_test.go

6831cc3

Update condition_test.go

bdfff25

Update fixed_arg_test.go

14ef6ed

Update free_arg_test.go

ab1fa79

Create ipaddress_evaluator_test.go

a7f5587

Create parser_test.go

3685786

Create rule_processor_factory_test.go

f01065d

Update rule_processor_test.go

ef0af2c

Update rule_test.go

836e115

Update rule_utils_test.go

b9be669

Remove unwanted coverage files

6cce6b7

test: add comprehensive tests for feature control and RFC response ha…

8a6f7d4

…ndling

test: add unit tests for evaluation result and download location filters

ddeef33

new flag to determine whether to add security token if no new fw offered

b1b9b61

Merge pull request #61 from rdkcentral/kloder

e925862

new flag to determine whether to add security token if no new fw offered

test: add unit tests for GetSettingRulesBySettingType and related fun…

ee4042f

…ctions

Merge pull request #62 from RahulRengeshOfficial/patch-1

ee24c5b

Patch 1

Update feature_rule_test.go

6f21a8a

Update feature_test.go

82e2dc8

Update config_change_logs_test.go

bf79343

Create download_location_filter_test.go

2e5d54b

Update estb_converters_func_test.go

6aa1c88

Update estb_fimware_context_test.go

52d5abb

Create firmware_config_test.go

2770a47

Create firmware_config_facade_test.go

23bc15b

Create ip_filter_test.go

5691c5b

Create percent_filter_test.go

fcc5356

Create reboot_immediately_filter_test.go

ff5d443

Create singleton_filter_test.go

d9404e9

gravi21 added 2 commits July 9, 2026 18:57

Update estb_firmware_context.go

80291f0

Merge pull request #190 from rdkcentral/revert_tz_debug_log

77bef6c

Removed added debug logs

Copilot AI review requested due to automatic review settings July 9, 2026 13:38

Copilot started reviewing on behalf of gravi21 July 9, 2026 13:38 View session

Copilot AI reviewed Jul 9, 2026

gravi21 and others added 8 commits July 10, 2026 19:04

Add condition enable_fw_penetration_metrics flag

905448d

Implement APAC routing support and enhance SAT service integration

a200817

Enhance APAC routing logic and add comprehensive tests for useApacRou…

6c3e099

…te function

Potential fix for pull request finding

fcf7731

Current behavior is consistent with existing logic where unknown-account detection uses the same utility check.. Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Refactor APAC routing to partner routing and update related configura…

cc6d076

…tions and tests

Update test to use 'unknown' partner ID for feature response verifica…

04c0b25

…tion

Refactor SAT token manager to support multiple partner tokens and enh…

5044aed

…ance error handling for partner route requests

Merge pull request #193 from rdkcentral/add_metrics_flag

3404129

Add condition enable_fw_penetration_metrics flag

Copilot AI review requested due to automatic review settings July 10, 2026 14:19

Copilot started reviewing on behalf of gravi21 July 10, 2026 14:20 View session

Copilot AI reviewed Jul 10, 2026

yeswanth2420 added 4 commits July 13, 2026 12:36

Merge branch 'develop' of https://github.com/rdkcentral/xconfwebconfig …

cd896a7

…into feature/foxtelMigration

Update Go version to 1.26.5

a301913

Rename partner_ids to partner_list in configuration and removed the u…

c89cb9e

…sage of ContextMap

Merge pull request #189 from rdkcentral/feature/foxtelMigration

b9a685d

Feature/foxtel migration

Copilot AI review requested due to automatic review settings July 13, 2026 15:10

Copilot started reviewing on behalf of yeswanth2420 July 13, 2026 15:10 View session

Copilot AI reviewed Jul 13, 2026

yeswanth2420 added 4 commits July 14, 2026 10:52

Enhance partner services configuration validation

4c52458

Merge branch 'develop' of github.com:rdkcentral/xconfwebconfig into f…

5be4f5a

…eature/foxtelMigration

Update README and configuration to include partner service variables

5fb0877

Merge pull request #197 from rdkcentral/feature/foxtelMigration

c160092

Feature/foxtel migration

Copilot AI review requested due to automatic review settings July 14, 2026 15:25

Copilot started reviewing on behalf of yeswanth2420 July 14, 2026 15:25 View session

Copilot AI reviewed Jul 14, 2026

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Develop#126

Develop#126
RahulRengeshOfficial wants to merge 330 commits into
mainfrom
develop

RahulRengeshOfficial commented Mar 11, 2026

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI left a comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

14 participants

Uh oh!

Conversation

RahulRengeshOfficial commented Mar 11, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

14 participants