From c85f2a56f0af7ebe5fb61107acea3b931739470b Mon Sep 17 00:00:00 2001 From: Uggla Date: Wed, 6 Apr 2016 19:25:55 +0200 Subject: [PATCH] Remove not working tests --- redfish/tests/__init__.py | 0 redfish/tests/base.py | 23 ------- redfish/tests/test_redfish.py | 122 ---------------------------------- 3 files changed, 145 deletions(-) delete mode 100644 redfish/tests/__init__.py delete mode 100644 redfish/tests/base.py delete mode 100644 redfish/tests/test_redfish.py diff --git a/redfish/tests/__init__.py b/redfish/tests/__init__.py deleted file mode 100644 index e69de29..0000000 diff --git a/redfish/tests/base.py b/redfish/tests/base.py deleted file mode 100644 index 1c30cdb..0000000 --- a/redfish/tests/base.py +++ /dev/null @@ -1,23 +0,0 @@ -# -*- coding: utf-8 -*- - -# Copyright 2010-2011 OpenStack Foundation -# Copyright (c) 2013 Hewlett-Packard Development Company, L.P. -# -# Licensed under the Apache License, Version 2.0 (the "License"); you may -# not use this file except in compliance with the License. You may obtain -# a copy of the License at -# -# http://www.apache.org/licenses/LICENSE-2.0 -# -# Unless required by applicable law or agreed to in writing, software -# distributed under the License is distributed on an "AS IS" BASIS, WITHOUT -# WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the -# License for the specific language governing permissions and limitations -# under the License. - -from oslotest import base - - -class TestCase(base.BaseTestCase): - - """Test case base class for all unit tests.""" diff --git a/redfish/tests/test_redfish.py b/redfish/tests/test_redfish.py deleted file mode 100644 index b70b43b..0000000 --- a/redfish/tests/test_redfish.py +++ /dev/null @@ -1,122 +0,0 @@ -# -*- coding: utf-8 -*- - -# Licensed under the Apache License, Version 2.0 (the "License"); you may -# not use this file except in compliance with the License. You may obtain -# a copy of the License at -# -# http://www.apache.org/licenses/LICENSE-2.0 -# -# Unless required by applicable law or agreed to in writing, software -# distributed under the License is distributed on an "AS IS" BASIS, WITHOUT -# WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the -# License for the specific language governing permissions and limitations -# under the License. - -""" -test_redfish ----------------------------------- - -Tests for `redfish` module. -""" - -import fixtures -import httplib -import json -import mock -import ssl - -from redfish.tests import base -from redfish import server -from redfish import types - - -def get_fake_params(host=None, user=None, pword=None): - if not host: - host = 'https://127.0.0.1' - if not user: - user = 'admin' - if not pword: - pword = 'password' - return (host, user, pword) - - -def get_response(): - class _response(object): - status = 200 - def read(self): - return '{"foo": "bar"}' - def getheaders(self): - return [('Fake-Header', 'fake value')] - return _response() - - -class TestException(Exception): - pass - - -class TestRedfishConnection(base.TestCase): - - def setUp(self): - super(TestRedfishConnection, self).setUp() - self.log_fixture = self.useFixture(fixtures.FakeLogger()) - self.con_mock = mock.MagicMock() - self.con_mock.getresponse = get_response - - self.http_mock = mock.patch.object(httplib, 'HTTPConnection').start() - self.http_mock.return_value = self.con_mock - self.https_mock = mock.patch.object(httplib, 'HTTPSConnection').start() - self.https_mock.return_value = self.con_mock - self.addCleanup(self.http_mock.stop) - self.addCleanup(self.https_mock.stop) - - def test_create_ok(self): - con = server.RedfishConnection(*get_fake_params()) - self.assertEqual(1, self.https_mock.call_count) - self.assertEqual(0, self.http_mock.call_count) - - def test_create_calls_https_connect(self): - self.https_mock.side_effect = TestException() - self.assertRaises(TestException, - server.RedfishConnection, - *get_fake_params(host='https://fake')) - - def test_create_calls_http_connect(self): - self.http_mock.side_effect = TestException() - self.assertRaises(TestException, - server.RedfishConnection, - *get_fake_params(host='http://fake')) - - # TODO: add test for unknown connection schema (eg, ftp://) - -# FIXME: ssl module has no attribute 'SSLContext' -# NOTE: skip this test if sys.version_info (major, minor) != (2, 7) and micro < 9 -# @mock.patch.object(ssl, 'SSLContext') -# def test_insecure_ssl(self, ssl_mock): -# ssl_mock.return_value = mock.Mock() -# con = connection.RedfishConnection(*get_fake_params) -# ssl_mock.assert_called_once_with(ssl.PROTOCOL_TLSv1) - - def test_get_ok(self): - con = server.RedfishConnection(*get_fake_params()) - res = con.rest_get('/v1/test/', '') - self.assertEqual(200, con.status) - # Headers ae lower cased when returned - self.assertIn('fake-header', con.headers.keys()) - self.assertIn('foo', res.keys()) - self.con_mock.request.assert_called_with( - 'GET', '/v1/test/', body='null', headers=mock.ANY) - - # TODO: add test for redirects - - # TODO: add test for collections - - # TODO: add test for gzip'd body - - def test_post_ok(self): - body = '{"fake": "body"}' - json_body = json.dumps(body) - con = server.RedfishConnection(*get_fake_params()) - res = con.rest_post('/v1/test/', '', body) - self.assertEqual(200, con.status) - self.con_mock.request.assert_called_with( - 'POST', '/v1/test/', body=json_body, headers=mock.ANY)