[prev in list] [next in list] [prev in thread] [next in thread] 

List:       spacewalk-devel
Subject:    [Spacewalk-devel] [PATCH 2/4] python oracle tests: moved connection settings to configuration file
From:       Flavio Castelli <fcastelli () suse ! com>
Date:       2013-10-18 13:20:47
Message-ID: 1382102449-8023-3-git-send-email-fcastelli () suse ! com
[Download RAW message or body]

Moved all the Oracle connection settings to the db_settings.ini file
(like we already do with the PostgreSQL's values).
---
 backend/test/db_settings.ini    |  4 ++++
 backend/test/runtests-oracle.py | 10 +++++++---
 2 files changed, 11 insertions(+), 3 deletions(-)

diff --git a/backend/test/db_settings.ini b/backend/test/db_settings.ini
index b93502e..9f5f1fb 100644
--- a/backend/test/db_settings.ini
+++ b/backend/test/db_settings.ini
@@ -3,3 +3,7 @@ host: localhost
 user: dgoodwin
 password: spacewalk
 database: spacewalk
+[oracle]
+user: spacewalk
+password: spacewalk
+database: xe
diff --git a/backend/test/runtests-oracle.py b/backend/test/runtests-oracle.py
index c13f70f..75fada2 100755
--- a/backend/test/runtests-oracle.py
+++ b/backend/test/runtests-oracle.py
@@ -23,6 +23,7 @@ import os
 import sys
 import unittest
 
+from ConfigParser import ConfigParser
 from spacewalk.server import rhnSQL
 
 # Import all test modules here:
@@ -32,9 +33,12 @@ sys.path.insert(
 )
 import dbtests
 
-USER = "spacewalk"
-PASSWORD = "spacewalk"
-DATABASE = "xe"
+config = ConfigParser()
+config.read(os.path.dirname(os.path.abspath(__file__)) + "/db_settings.ini")
+
+USER     = config.get('oracle', 'user')
+PASSWORD = config.get('oracle', 'password')
+DATABASE = config.get('oracle', 'database')
 
 rhnSQL.initDB(backend="oracle", username=USER,
         password=PASSWORD, database=DATABASE)
-- 
1.8.1.4

_______________________________________________
Spacewalk-devel mailing list
Spacewalk-devel@redhat.com
https://www.redhat.com/mailman/listinfo/spacewalk-devel
[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic