From 6bf206eb810f3f72115e2194f615fce88fb3b0f2 Mon Sep 17 00:00:00 2001 From: Thomas Haller Date: Sun, 3 Nov 2019 22:26:04 +0100 Subject: libnm/tests: drop test_activate_failed() test With this test the stub service simulates a failure to add-and-activate the connection. However the implementation of the stub service was not simulating the real behavior of NetworkManager service. libnm will add the possibility to assert against invalid server behavior by setting LIBNM_CLIENT_DEBUG=error. With that change, libnm will complain that the stub service behaves invalid, and rightly so. Instead of fixing the test, just drop it. --- tools/test-networkmanager-service.py | 8 -------- 1 file changed, 8 deletions(-) (limited to 'tools/test-networkmanager-service.py') diff --git a/tools/test-networkmanager-service.py b/tools/test-networkmanager-service.py index e91b45233e..1a8e2e6af5 100755 --- a/tools/test-networkmanager-service.py +++ b/tools/test-networkmanager-service.py @@ -1306,14 +1306,6 @@ class NetworkManager(ExportedObj): ac = ActiveConnection(device, con_inst, None) self.active_connection_add(ac) - - if NmUtil.con_hash_get_id(con_hash) == 'object-creation-failed-test': - # FIXME: this is not the right test, to delete the active-connection - # before returning it. It's the wrong order of what NetworkManager - # would do. - self.active_connection_remove(ac) - return ExportedObj.to_path(ac) - return ExportedObj.to_path(ac) def active_connection_add(self, ac): -- cgit v1.2.1