From 701ab640f1d99f955ba0b14ae5bde49a3b7887f4 Mon Sep 17 00:00:00 2001 From: "Ladue, David (dl3158)" Date: Wed, 22 Aug 2018 17:01:59 -0400 Subject: added test coverage Change-Id: Ie8310bab09ce329e499263f07b3ab2d3a8969727 Issue-ID: DCAEGEN2-630 Signed-off-by: Ladue, David (dl3158) --- snmptrap/mod/trapd_snmpv3.py | 14 - snmptrap/snmptrapd.py | 2 +- spec/snmptrap-collector-component-spec.json | 428 +++++++++++----------------- tests/test_snmptrapd.py | 27 ++ tests/test_trapd_http_session.py | 27 +- tests/test_trapd_snmpv3.py | 60 ++++ 6 files changed, 282 insertions(+), 276 deletions(-) create mode 100644 tests/test_trapd_snmpv3.py diff --git a/snmptrap/mod/trapd_snmpv3.py b/snmptrap/mod/trapd_snmpv3.py index 97c048c..50da980 100644 --- a/snmptrap/mod/trapd_snmpv3.py +++ b/snmptrap/mod/trapd_snmpv3.py @@ -47,20 +47,6 @@ from trapd_io import stdout_logger, ecomp_logger prog_name = os.path.basename(__file__) - -# # # # # # # # # # # # # # # # # # # # # # # # # # # # # # -# module: init -# FMDL: should this re-establish listener, with -# blank V3 config attribs? -# # # # # # # # # # # # # # # # # # # # # # # # # # # # # # - - -def init(): - - global v3_config_dict - v3_config_dict = {} - - # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # module: load_snmpv3_credentials # # # # # # # # # # # # # # # # # # # # # # # # # # # # # # diff --git a/snmptrap/snmptrapd.py b/snmptrap/snmptrapd.py index 66cc25f..70dc668 100644 --- a/snmptrap/snmptrapd.py +++ b/snmptrap/snmptrapd.py @@ -73,7 +73,7 @@ from trapd_runtime_pid import save_pid, rm_pid from trapd_get_cbs_config import get_cbs_config from trapd_exit import cleanup_and_exit from trapd_http_session import init_session_obj, close_session_obj, reset_session_obj -from trapd_snmpv3 import init, load_snmpv3_credentials +from trapd_snmpv3 import load_snmpv3_credentials from trapd_io import roll_all_logs, open_eelf_logs, roll_file, open_file, close_file, ecomp_logger, stdout_logger prog_name = os.path.basename(__file__) diff --git a/spec/snmptrap-collector-component-spec.json b/spec/snmptrap-collector-component-spec.json index e9f827e..e87d800 100644 --- a/spec/snmptrap-collector-component-spec.json +++ b/spec/snmptrap-collector-component-spec.json @@ -1,261 +1,171 @@ { - "self": { - "version": "1.4.0", - "name": "snmptrap-collector", - "description": "Collector for receiving SNMP traps and publishing to DMAAP/MR", - "component_type": "docker" - }, - "streams": { - "subscribes": [ - - ], - "publishes": [ - { - "format": "snmptrap_output", - "version": "1.3.0", - "type": "message router", - "config_key": "snmp_trap" - } - ] - }, - "services": { - "calls": [], - "provides": [] - }, - "parameters": [ - { - "name": "sw_interval_in_seconds", - "value": "60", - "description": "StormWatch sample interval", - "designer_editable" : false, - "policy_editable" : false, - "sourced_at_deployment" : false - }, - { - "name": "StormWatchPolicy", - "description": "Configure storm watch control parameters via Policy", - "sourced_at_deployment": false, - "designer_editable": true, - "policy_editable": true, - "policy_group": "DCAE.Config_StormWatch", - "policy_schema": [ - { - "name": "sw_trap_config", - "description": "List of storm watch control rules", - "type": "list", - "entry_schema": [ - { - "name": "sw_notify_oid", - "description": "Stormwatch notify oid", - "type": "string", - "value": "" - }, - { - "name": "sw_high_water_in_interval", - "description": "Onset number of traps in interval", - "type": "string", - "value": "" - }, - { - "name": "sw_low_water_in_interval", - "description": "Clearing number of traps in interval", - "type": "string", - "value": "" - } - ] - } - ] - }, - { - "name": "Protocols", - "description": "Protocol parameters", - "sourced_at_deployment": false, - "designer_editable": false, - "policy_editable": false, - "value": [ - { - "name": "ipv4_interface", - "description": "ipv4 address trap receiver should listen to (0.0.0.0 -> all)", - "value": "0.0.0.0" - }, - { - "name": "ipv4_port", - "description": "port trap receiver should bind to for ipv4 traps", - "value": "162" - }, - { - "name": "ipv6_interface", - "description": "ipv6 address trap receiver should listen to (::FFFF:0:0 -> all)", - "value": "::FFFF:0:0" - }, - { - "name": "ipv6_port", - "description": "port trap receiver should bind to for ipv6 traps", - "value": "162" - } - ] - }, - { - "name": "Cache", - "description": "Cache parameters", - "sourced_at_deployment": false, - "designer_editable": false, - "policy_editable": false, - "value": [ - { - "name": "dns_cache_ttl_seconds", - "description": "dns cache ttl in seconds", - "value": "60" - } - ] - }, - { - "name": "Files", - "description": "Files parameters", - "sourced_at_deployment": false, - "designer_editable": false, - "policy_editable": false, - "value": [ - { - "name": "arriving_traps_log", - "description": "log for all traps accepted by receiver", - "value": "snmptrapd_arriving_traps.log" - }, - { - "name": "data_dir", - "description": "directory for published traps (json)", - "value": "data" - }, - { - "name": "eelf_audit", - "description": "eelf audit log", - "value": "audit.log" - }, - { - "name": "eelf_base_dir", - "description": "base directory for eelf logs", - "value": "/opt/app/snmptrap/logs" - }, - { - "name": "eelf_debug", - "description": "eelf debug log", - "value": "debug.log" - }, - { - "name": "eelf_error", - "description": "eelf error log", - "value": "error.log" - }, - { - "name": "eelf_metrics", - "description": "eelf metrics log", - "value": "metrics.log" - }, - { - "name": "log_dir", - "description": "logs directory", - "value": "logs" - }, - { - "name": "minimum_severity_to_log", - "description": "minimun severity to log", - "value": "2" - }, - { - "name": "perm_status_file", - "description": "permanent status log", - "value": "snmptrapd_status.log" - }, - { - "name": "pid_dir", - "description": "directory for pid file", - "value": "tmp" - }, - { - "name": "roll_frequency", - "description": "frequency of log file rolls", - "value": "hour" - }, - { - "name": "runtime_base_dir", - "description": "base directory for runtime files", - "value": "hour" - }, - { - "name": "snmptrapd_diag", - "description": "diagnostic program log", - "value": "snmptrapd_prog_diag.log" - }, - { - "name": "traps_stats_log", - "description": "traps statistics log", - "value": "snmptrapd_stats.csv" - } - ] - }, - { - "name": "Publisher", - "description": "Publisher parameters", - "sourced_at_deployment": false, - "designer_editable": false, - "policy_editable": false, - "value": [ - { - "name": "http_milliseconds_between_retries", - "description": "milliseconds between publish retries", - "value": "750" - }, - { - "name": "http_peer_publisher", - "description": "peer publisher", - "value": "unavailable" - }, - { - "name": "http_primary_publisher", - "description": "primary publisher", - "value": "true" - }, - { - "name": "http_retries", - "description": "retries for publish attempts", - "value": "3" - }, - { - "name": "http_timeout_milliseconds", - "description": "milliseconds for publish timeout", - "value": "1500" - }, - { - "name": "max_milliseconds_between_publishes", - "description": "max milliseconds between publishes", - "value": "10000" - }, - { - "name": "max_traps_between_publishes", - "description": "max traps between publishes", - "value": "10" - } - ] - } - ], - "auxilary": { - "healthcheck": { - "type": "docker", - "interval": "300s", - "timeout": "120s", - "script": "/opt/app/snmptrap/bin/healthcheck.sh" - }, - "reconfigs": { - "dti": "/opt/app/snmptrap/bin/snmptrapd.sh reloadCfg" - }, - "ports": [ - "6162:162" - ] - }, - "artifacts": [ - { - "type": "docker image", - "uri": "nexus.onap.org:10001/snapshots/onap/org.onap.dcaegen2.collectors.snmptrap:1.4.0" - } - ] + "self": { + "version": "1.4.0", + "name": "snmptrap-collector", + "description": "Collector for receiving SNMP traps and publishing to DMAAP/MR", + "component_type": "docker" + }, + "streams": { + "subscribes": [], + "publishes": [ + { + "format": "snmptrap_output", + "version": "1.3.0", + "type": "message router", + "config_key": "sec_fault_unsecure" + } + ] + }, + "services": { + "calls": [], + "provides": [] + }, + "parameters": [ + { + "name": "sw_interval_in_seconds", + "value": "60", + "description": "StormWatch sample interval", + "designer_editable": false, + "policy_editable": false, + "sourced_at_deployment": false + }, + { + "name": "StormWatchPolicy", + "value": "", + "description": "Configure storm watch control parameters via Policy", + "sourced_at_deployment": false, + "designer_editable": true, + "policy_editable": true, + "policy_group": "DCAE.Config_StormWatch", + "policy_schema": [ + { + "name": "sw_trap_config", + "description": "List of storm watch control rules", + "type": "list", + "entry_schema": [ + { + "name": "sw_notify_oid", + "description": "Stormwatch notify oid", + "type": "string", + "value": "" + }, + { + "name": "sw_high_water_in_interval", + "description": "Onset number of traps in interval", + "type": "string", + "value": "" + }, + { + "name": "sw_low_water_in_interval", + "description": "Clearing number of traps in interval", + "type": "string", + "value": "" + } + ] + } + ] + }, + { + "name": "cache", + "description": "Cache parameters", + "sourced_at_deployment": true, + "designer_editable": false, + "policy_editable": false, + "value": { + "dns_cache_ttl_seconds": "60" + } + }, + { + "name": "files", + "description": "Files parameters", + "sourced_at_deployment": true, + "designer_editable": false, + "policy_editable": false, + "value": [ + { + "arriving_traps_log": "snmptrapd_arriving_traps.log", + "data_dir": "data", + "eelf_audit": "audit.log", + "eelf_base_dir": "/opt/app/snmptrap/logs", + "eelf_debug": "debug.log", + "eelf_error": "error.log", + "eelf_metrics": "metrics.log", + "log_dir": "logs", + "minimum_severity_to_log": "2", + "perm_status_file": "snmptrapd_status.log", + "pid_dir": "tmp", + "roll_frequency": "day", + "runtime_base_dir": "/opt/app/snmptrap", + "snmptrapd_diag": "snmptrapd_prog_diag.log", + "traps_stats_log": "snmptrapd_stats.csv" + } + ] + }, + { + "name": "publisher", + "description": "Publisher parameters", + "sourced_at_deployment": true, + "designer_editable": false, + "policy_editable": false, + "value": [ + { + "http_milliseconds_between_retries": "750", + "http_peer_publisher": "unavailable", + "http_primary_publisher": "true", + "http_retries": "3", + "http_milliseconds_timeout": "1500", + "max_milliseconds_between_publishes": "10000", + "max_traps_between_publishes": "10" + } + ] + }, + { + "name": "protocols", + "description": "Protocol parameters", + "sourced_at_deployment": true, + "designer_editable": false, + "policy_editable": false, + "value": [ + { + "ipv4_interface": "0.0.0.0", + "ipv4_port": "6162", + "ipv6_interface": "::1", + "ipv6_port": "6162" + } + ] + }, + { + "name": "snmptrapd", + "description": "application basics", + "sourced_at_deployment": true, + "designer_editable": false, + "policy_editable": false, + "value": [ + { + "version": "1.4.0", + "title": "Collector for receiving SNMP traps and publishing to DMAAP/MR" + } + ] + } + ], + "auxilary": { + "healthcheck": { + "type": "docker", + "interval": "300s", + "timeout": "120s", + "script": "/opt/app/snmptrap/bin/snmptrapd.sh status" + }, + "reconfigs": { + "dti": "/opt/app/snmptrap/bin/snmptrapd.sh reloadCfg" + }, + "ports": [ + "6162/udp:162" + ] + }, + "artifacts": [ + { + "type": "docker image", + "uri": "nexus01.research.att.com:18443/dcae-platform/snmptrap:1.4.0" + } + ] } diff --git a/tests/test_snmptrapd.py b/tests/test_snmptrapd.py index 941c0ae..85681ea 100644 --- a/tests/test_snmptrapd.py +++ b/tests/test_snmptrapd.py @@ -10,6 +10,9 @@ import trapd_runtime_pid import trapd_io import trapd_get_cbs_config +from pysnmp.hlapi import * +from pysnmp import debug + class test_snmptrapd(unittest.TestCase): """ Test the save_pid mod @@ -123,5 +126,29 @@ class test_snmptrapd(unittest.TestCase): # open eelf logs trapd_io.open_eelf_logs() + def test_v1_trap_receipt(self): + """ + Test receiving traps + """ + + # init vars + tds.init() + + # request load of CBS data + os.environ.update(CBS_SIM_JSON='/tmp/opt/app/snmptrap/etc/snmptrapd.json') + trapd_get_cbs_config.get_cbs_config() + + errorIndication, errorStatus, errorIndex, varbinds = next(sendNotification(SnmpEngine(), + CommunityData('not_public'), + UdpTransportTarget(('localhost', 6162)), + ContextData(), + 'trap', + [ObjectType(ObjectIdentity('.1.3.6.1.4.1.999.1'), OctetString('test trap - ignore')), + ObjectType(ObjectIdentity('.1.3.6.1.4.1.999.2'), OctetString('ONAP pytest trap'))]) + ) + + result = errorIndication + self.assertEqual(result, None) + if __name__ == '__main__': unittest.main() diff --git a/tests/test_trapd_http_session.py b/tests/test_trapd_http_session.py index 8f61d08..a423be6 100644 --- a/tests/test_trapd_http_session.py +++ b/tests/test_trapd_http_session.py @@ -7,14 +7,37 @@ class test_init_session_obj(unittest.TestCase): Test the init_session_obj mod """ - def test_correct_usage(self): + def init_session(self): """ - Test that attempt to create http session object works + test creation of http session """ result = trapd_http_session.init_session_obj() compare = str(result).startswith("