[OpenNMS/opennms] 98c279: NMS-19096: Replace scan-stamp unit test with a beh...
Marshall Massengill via opennms-cvs <[email protected]> Fri, 24 Jul 2026 10:22:38 -0700
| Newsgroups | gmane.network.opennms.cvs |
|---|---|
| Message-ID | <OpenNMS/opennms/push/refs/heads/mm/NMS-19096-smoke/[email protected]> |
Branch: refs/heads/mm/NMS-19096-smoke
Home: https://github.com/OpenNMS/opennms
Commit: 98c2793aca24e4c036aaf2ecbd7816ce4cf14a0f
https://github.com/OpenNMS/opennms/commit/98c2793aca24e4c036aaf2ecbd7816ce4cf14a0f
Author: Marshall Massengill <[email protected]>
Date: 2026-07-24 (Fri, 24 Jul 2026)
Changed paths:
R opennms-provision/opennms-provisiond/src/test/java/org/opennms/netmgt/provision/service/NodeScanTest.java
M opennms-provision/opennms-provisiond/src/test/java/org/opennms/netmgt/provision/service/ProvisionerIT.java
Log Message:
-----------
NMS-19096: Replace scan-stamp unit test with a behavioral IT
The NodeScanTest only reflected reset() and asserted it reassigned
m_scanStamp, which restates the implementation rather than testing
behavior. Replace it with ProvisionerIT#testReusedNodeScanReapsVanishedInterfaceNms19096,
which reuses a single NodeScan instance across two runs (as the scheduler
does) and asserts that an SNMP-discovered interface which disappears
between runs is reaped. This fails against the frozen-stamp behavior and
passes with the fix.
To unsubscribe from these emails, change your notification settings at https://github.com/OpenNMS/opennms/settings/notifications
_______________________________________________
Please read the OpenNMS Mailing List FAQ:
http://www.opennms.org/wiki/index.php?page=MailingListFaq
opennms-cvs mailing list
To *unsubscribe* or change your subscription options, see the bottom of this page:
https://lists.sourceforge.net/lists/listinfo/opennms-cvs