From eada7872f2cf935d9a6e8e7cd410a895bee979e7 Mon Sep 17 00:00:00 2001 From: Julien Danjou Date: Mon, 17 Dec 2012 11:30:22 +0100 Subject: [PATCH] Fix dbsync API change This binary didn't get modified, so add test to be sure we don't miss it next time. Change-Id: I62692e7ee2fd8dd0f4845d130d6b193c7849c5f9 Signed-off-by: Julien Danjou --- bin/ceilometer-dbsync | 2 +- tests/test_bin.py | 28 ++++++++++++++++++++++++++++ 2 files changed, 29 insertions(+), 1 deletion(-) create mode 100644 tests/test_bin.py diff --git a/bin/ceilometer-dbsync b/bin/ceilometer-dbsync index 935166c3..0eaf6ae7 100755 --- a/bin/ceilometer-dbsync +++ b/bin/ceilometer-dbsync @@ -25,4 +25,4 @@ from ceilometer.openstack.common import cfg if __name__ == '__main__': service.prepare_service(sys.argv) - storage.get_connection(cfg.CONF).db_sync() + storage.get_connection(cfg.CONF).upgrade() diff --git a/tests/test_bin.py b/tests/test_bin.py new file mode 100644 index 00000000..19c60721 --- /dev/null +++ b/tests/test_bin.py @@ -0,0 +1,28 @@ +#!/usr/bin/env python +# -*- encoding: utf-8 -*- +# +# Copyright © 2012 eNovance +# +# Author: Julien Danjou +# +# 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. + +import subprocess +import unittest + + +class BinDbsyncTestCase(unittest.TestCase): + def test_dbsync_run(self): + subp = subprocess.Popen(["../bin/ceilometer-dbsync", + "--database_connection=log://localhost"]) + self.assertEqual(subp.wait(), 0)