Tests for device config updates on org variable changes

[ssmadhavan006]

Checklist

• I have read the OpenWISP Contributing Guidelines.
• I have manually tested the changes proposed in this pull request.
• I have written new test cases for new code and/or updated existing tests for changes to existing code.
• I have updated the documentation.

Reference to Existing Issue

Closes #1070.

Description of Changes

This PR adds missing automated tests to ensure device configuration status updates correctly when organization configuration variables (context) change. The existing implementation already handled this logic correctly (via checksum comparison) but there was no automated test coverage verifying the behavior.

What’s Added

A new test class TestOrganizationConfigUpdates in openwisp_controller/config/tests/test_device.py. It includes three scenarios:

1. Affected device becomes modified

   • Template uses {{ ssid }}
   • Changing org context updates config status

2. Unaffected device remains applied

   • Template has no variables
   • Context change does not affect checksum

3. No status change when checksum does not change

   • Unused variable changes
   • Config remains unchanged

Notes:

• No production code was modified
• Tests run synchronously (CELERY_TASK_ALWAYS_EAGER=True)
• Ensures future regressions are prevented

Screenshot

Creating a Organization

Creating a Template-1

Creating a Template-2

Device-1 Configuration

Device-2 Configuration

Before Change

Change Org Variable

Device Status Change

[nemesifier]

@coderabbitai review

Sorry @ssmadhavan006 we've been flooded with PRs lately and it's taking time to respond to everyone.

Instead of sharing screenshots in this case a video recording demonstrating the solution would have been better.

[coderabbitai]

@nemesifier: I'll review the changes in this PR. This looks like a focused test-only PR adding coverage for device configuration status updates when organization variables change.

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

[coderabbitai]

Caution

Review failed

The head commit changed during the review from 8a4e0e7 to 3388fb1.

Walkthrough

This pull request adds a new test class TestOrganizationConfigUpdates with three test methods to validate behavior when organization-level configuration variables are modified. The tests verify that: (1) changing organization variables causes related device configs to transition to "modified" status, (2) unrelated devices remain unaffected when organization variables change, and (3) config status remains "applied" when variable changes do not affect the resulting checksum.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

🚥 Pre-merge checks | ✅ 4

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately describes the main change: adding tests for device config updates when organization variables change.
Description check	✅ Passed	The description is comprehensive with linked issue, detailed changes explanation, test scenarios, and supporting screenshots. All required sections are well-documented.
Linked Issues check	✅ Passed	The PR directly addresses issue #1070 by adding test coverage for the scenario where changing organization variables should update device config status to 'modified'.
Out of Scope Changes check	✅ Passed	All changes are scoped to test code only in test_device.py; no production code modifications, staying within the objective of adding test coverage for issue #1070.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

Important

Action Needed: IP Allowlist Update

If your organization protects your Git platform with IP whitelisting, please add the new CodeRabbit IP address to your allowlist:

• ✨ 136.113.208.247/32 (new)
• 34.170.211.100/32
• 35.222.179.152/32

Failure to add the new IP will result in interrupted reviews.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[nemesifier]

I only see a test here, but no solution, is this on purpose? Waiting for the CI build, I assume you worked on replicating the bug in the tests and I expect this test to fail, let's see.

[stktyagi]

Apologies for the noise