[OE-core] [PATCH 1/1] oe-selftest: use spaces to indent python code, not tabs

Paul Eggleton paul.eggleton at linux.intel.com
Fri Mar 20 12:15:02 UTC 2015


Indenting-only change.

Signed-off-by: Paul Eggleton <paul.eggleton at linux.intel.com>
---
 meta/lib/oeqa/selftest/_toaster.py | 556 ++++++++++++++++++-------------------
 meta/lib/oeqa/selftest/bbtests.py  |  72 ++---
 2 files changed, 314 insertions(+), 314 deletions(-)

diff --git a/meta/lib/oeqa/selftest/_toaster.py b/meta/lib/oeqa/selftest/_toaster.py
index 1cf28a0..2ed08d2 100644
--- a/meta/lib/oeqa/selftest/_toaster.py
+++ b/meta/lib/oeqa/selftest/_toaster.py
@@ -19,89 +19,89 @@ from oeqa.utils.decorators import testcase
 class ToasterSetup(oeSelfTest):
 
     def recipe_parse(self, file_path, var):
-	for line in open(file_path,'r'):
-	    if line.find(var) > -1:
-		val = line.split(" = ")[1].replace("\"", "").strip()
-		return val
+        for line in open(file_path,'r'):
+            if line.find(var) > -1:
+                val = line.split(" = ")[1].replace("\"", "").strip()
+                return val
 
     def fix_file_path(self, file_path):
-	if ":" in file_path:
-	    file_path=file_path.split(":")[2]
-	return file_path
+        if ":" in file_path:
+            file_path=file_path.split(":")[2]
+        return file_path
 
 class Toaster_DB_Tests(ToasterSetup):
 
     # Check if build name is unique - tc_id=795
     @testcase(795)
     def test_Build_Unique_Name(self):
-	all_builds = Build.objects.all().count()
-	distinct_builds = Build.objects.values('id').distinct().count()
-	self.assertEqual(distinct_builds, all_builds, msg = 'Build name is not unique')
+        all_builds = Build.objects.all().count()
+        distinct_builds = Build.objects.values('id').distinct().count()
+        self.assertEqual(distinct_builds, all_builds, msg = 'Build name is not unique')
 
     # Check if build coocker log path is unique - tc_id=819
     @testcase(819)
     def test_Build_Unique_Cooker_Log_Path(self):
-	distinct_path = Build.objects.values('cooker_log_path').distinct().count()
-	total_builds = Build.objects.values('id').count()
-	self.assertEqual(distinct_path, total_builds, msg = 'Build coocker log path is not unique')
+        distinct_path = Build.objects.values('cooker_log_path').distinct().count()
+        total_builds = Build.objects.values('id').count()
+        self.assertEqual(distinct_path, total_builds, msg = 'Build coocker log path is not unique')
 
     # Check if the number of errors matches the number of orm_logmessage.level entries with value 2 - tc_id=820
     @testcase(820)
     def test_Build_Errors_No(self):
-	builds = Build.objects.values('id', 'errors_no')
-	cnt_err = []
-	for build in builds:
-	    log_mess_err_no = LogMessage.objects.filter(build = build['id'], level = 2).count()
-	    if (build['errors_no'] != log_mess_err_no):
-		cnt_err.append(build['id'])
-	self.assertEqual(len(cnt_err), 0, msg = 'Errors for build id: %s' % cnt_err)
+        builds = Build.objects.values('id', 'errors_no')
+        cnt_err = []
+        for build in builds:
+            log_mess_err_no = LogMessage.objects.filter(build = build['id'], level = 2).count()
+            if (build['errors_no'] != log_mess_err_no):
+                cnt_err.append(build['id'])
+        self.assertEqual(len(cnt_err), 0, msg = 'Errors for build id: %s' % cnt_err)
 
     # Check if the number of warnings matches the number of orm_logmessage.level entries with value 1 - tc=821
     @testcase(821)
     def test_Build_Warnings_No(self):
-	builds = Build.objects.values('id', 'warnings_no')
-	cnt_err = []
-	for build in builds:
-	    log_mess_warn_no = LogMessage.objects.filter(build = build['id'], level = 1).count()
-	    if (build['warnings_no'] != log_mess_warn_no):
-		cnt_err.append(build['id'])
-	self.assertEqual(len(cnt_err), 0, msg = 'Errors for build id: %s' % cnt_err)
+        builds = Build.objects.values('id', 'warnings_no')
+        cnt_err = []
+        for build in builds:
+            log_mess_warn_no = LogMessage.objects.filter(build = build['id'], level = 1).count()
+            if (build['warnings_no'] != log_mess_warn_no):
+                cnt_err.append(build['id'])
+        self.assertEqual(len(cnt_err), 0, msg = 'Errors for build id: %s' % cnt_err)
 
     # Check if the build succeeded then the errors_no is 0 - tc_id=822
     @testcase(822)
     def test_Build_Suceeded_Errors_No(self):
-	builds = Build.objects.filter(outcome = 0).values('id', 'errors_no')
-	cnt_err = []
-	for build in builds:
-	    if (build['errors_no'] != 0):
-		cnt_err.append(build['id'])
-	self.assertEqual(len(cnt_err), 0, msg = 'Errors for build id: %s' % cnt_err)
+        builds = Build.objects.filter(outcome = 0).values('id', 'errors_no')
+        cnt_err = []
+        for build in builds:
+            if (build['errors_no'] != 0):
+                cnt_err.append(build['id'])
+        self.assertEqual(len(cnt_err), 0, msg = 'Errors for build id: %s' % cnt_err)
 
     # Check if task order is unique for one build - tc=824
-    @testcase(824)	
+    @testcase(824)
     def test_Task_Unique_Order(self):
-	builds = Build.objects.values('id')
-	cnt_err = []
-	for build in builds:
-	    total_task_order = Task.objects.filter(build = build['id']).values('order').count()
-	    distinct_task_order = Task.objects.filter(build = build['id']).values('order').distinct().count()
-	    if (total_task_order != distinct_task_order):
-		cnt_err.append(build['id'])
-	self.assertEqual(len(cnt_err), 0, msg = 'Errors for build id: %s' % cnt_err)
+        builds = Build.objects.values('id')
+        cnt_err = []
+        for build in builds:
+            total_task_order = Task.objects.filter(build = build['id']).values('order').count()
+            distinct_task_order = Task.objects.filter(build = build['id']).values('order').distinct().count()
+            if (total_task_order != distinct_task_order):
+                cnt_err.append(build['id'])
+        self.assertEqual(len(cnt_err), 0, msg = 'Errors for build id: %s' % cnt_err)
 
     # Check task order sequence for one build - tc=825
     @testcase(825)
     def test_Task_Order_Sequence(self):
-	builds = builds = Build.objects.values('id')
-	cnt_err = []
-	for build in builds:
-	    tasks = Task.objects.filter(Q(build = build['id']), ~Q(order = None), ~Q(task_name__contains = '_setscene')).values('id', 'order').order_by("order")
-	    cnt_tasks = 0
-	    for task in tasks:
-		cnt_tasks += 1
-		if (task['order'] != cnt_tasks):
-		    cnt_err.append(task['id'])
-	self.assertEqual(len(cnt_err), 0, msg = 'Errors for task id: %s' % cnt_err)
+        builds = builds = Build.objects.values('id')
+        cnt_err = []
+        for build in builds:
+            tasks = Task.objects.filter(Q(build = build['id']), ~Q(order = None), ~Q(task_name__contains = '_setscene')).values('id', 'order').order_by("order")
+            cnt_tasks = 0
+            for task in tasks:
+                cnt_tasks += 1
+                if (task['order'] != cnt_tasks):
+                    cnt_err.append(task['id'])
+        self.assertEqual(len(cnt_err), 0, msg = 'Errors for task id: %s' % cnt_err)
 
     # Check if disk_io matches the difference between EndTimeIO and StartTimeIO in build stats - tc=828
     ### this needs to be updated ###
@@ -110,336 +110,336 @@ class Toaster_DB_Tests(ToasterSetup):
     # Check if outcome = 2 (SSTATE) then sstate_result must be 3 (RESTORED) - tc=832
     @testcase(832)
     def test_Task_If_Outcome_2_Sstate_Result_Must_Be_3(self):
-	tasks = Task.objects.filter(outcome = 2).values('id', 'sstate_result')
-	cnt_err = []
-	for task in tasks:
-	    if (row['sstate_result'] != 3):
-		cnt_err.append(task['id'])
-	self.assertEqual(len(cnt_err), 0, msg = 'Errors for task id: %s' % cnt_err)
+        tasks = Task.objects.filter(outcome = 2).values('id', 'sstate_result')
+        cnt_err = []
+        for task in tasks:
+            if (row['sstate_result'] != 3):
+                cnt_err.append(task['id'])
+        self.assertEqual(len(cnt_err), 0, msg = 'Errors for task id: %s' % cnt_err)
 
     # Check if outcome = 1 (COVERED) or 3 (EXISTING) then sstate_result must be 0 (SSTATE_NA) - tc=833
     @testcase(833)
     def test_Task_If_Outcome_1_3_Sstate_Result_Must_Be_0(self):
-	tasks = Task.objects.filter(outcome__in = (1, 3)).values('id', 'sstate_result')
-	cnt_err = []
-	for task in tasks:
-	    if (task['sstate_result'] != 0):
-		cnt_err.append(task['id'])
-	self.assertEqual(len(cnt_err), 0, msg = 'Errors for task id: %s' % cnt_err)
+        tasks = Task.objects.filter(outcome__in = (1, 3)).values('id', 'sstate_result')
+        cnt_err = []
+        for task in tasks:
+            if (task['sstate_result'] != 0):
+                cnt_err.append(task['id'])
+        self.assertEqual(len(cnt_err), 0, msg = 'Errors for task id: %s' % cnt_err)
 
     # Check if outcome is 0 (SUCCESS) or 4 (FAILED) then sstate_result must be 0 (NA), 1 (MISS) or 2 (FAILED) - tc=834
     @testcase(834)
     def test_Task_If_Outcome_0_4_Sstate_Result_Must_Be_0_1_2(self):
-	tasks = Task.objects.filter(outcome__in = (0, 4)).values('id', 'sstate_result')
-	cnt_err = []
-	for task in tasks:
-	    if (task['sstate_result'] not in [0, 1, 2]):
-		cnt_err.append(task['id'])
-	self.assertEqual(len(cnt_err), 0, msg = 'Errors for task id: %s' % cnt_err)
+        tasks = Task.objects.filter(outcome__in = (0, 4)).values('id', 'sstate_result')
+        cnt_err = []
+        for task in tasks:
+            if (task['sstate_result'] not in [0, 1, 2]):
+                cnt_err.append(task['id'])
+        self.assertEqual(len(cnt_err), 0, msg = 'Errors for task id: %s' % cnt_err)
 
     # Check if task_executed = TRUE (1), script_type must be 0 (CODING_NA), 2 (CODING_PYTHON), 3 (CODING_SHELL) - tc=891
     @testcase(891)
     def test_Task_If_Task_Executed_True_Script_Type_0_2_3(self):
-	tasks = Task.objects.filter(task_executed = 1).values('id', 'script_type')
-	cnt_err = []
-	for task in tasks:
-	    if (task['script_type'] not in [0, 2, 3]):
-		cnt_err.append(task['id'])
-	self.assertEqual(len(cnt_err), 0, msg = 'Errors for task id: %s' % cnt_err)
+        tasks = Task.objects.filter(task_executed = 1).values('id', 'script_type')
+        cnt_err = []
+        for task in tasks:
+            if (task['script_type'] not in [0, 2, 3]):
+                cnt_err.append(task['id'])
+        self.assertEqual(len(cnt_err), 0, msg = 'Errors for task id: %s' % cnt_err)
 
     # Check if task_executed = TRUE (1), outcome must be 0 (SUCCESS) or 4 (FAILED) - tc=836
     @testcase(836)
     def test_Task_If_Task_Executed_True_Outcome_0_4(self):
-	tasks = Task.objects.filter(task_executed = 1).values('id', 'outcome')
-	cnt_err = []
-	for task in tasks:
-	    if (task['outcome'] not in [0, 4]):
-		cnt_err.append(task['id'])
-	self.assertEqual(len(cnt_err), 0, msg = 'Errors for task id: %s' % cnt_err)
+        tasks = Task.objects.filter(task_executed = 1).values('id', 'outcome')
+        cnt_err = []
+        for task in tasks:
+            if (task['outcome'] not in [0, 4]):
+                cnt_err.append(task['id'])
+        self.assertEqual(len(cnt_err), 0, msg = 'Errors for task id: %s' % cnt_err)
 
     # Check if task_executed = FALSE (0), script_type must be 0 - tc=890
     @testcase(890)
     def test_Task_If_Task_Executed_False_Script_Type_0(self):
-	tasks = Task.objects.filter(task_executed = 0).values('id', 'script_type')
-	cnt_err = []
-	for task in tasks:
-	    if (task['script_type'] != 0):
-		cnt_err.append(task['id'])
-	self.assertEqual(len(cnt_err), 0, msg = 'Errors for task id: %s' % cnt_err)
+        tasks = Task.objects.filter(task_executed = 0).values('id', 'script_type')
+        cnt_err = []
+        for task in tasks:
+            if (task['script_type'] != 0):
+                cnt_err.append(task['id'])
+        self.assertEqual(len(cnt_err), 0, msg = 'Errors for task id: %s' % cnt_err)
 
     # Check if task_executed = FALSE (0) and build outcome = SUCCEEDED (0), task outcome must be 1 (COVERED), 2 (CACHED), 3 (PREBUILT), 5 (EMPTY) - tc=837
     @testcase(837)
     def test_Task_If_Task_Executed_False_Outcome_1_2_3_5(self):
-	builds = Build.objects.filter(outcome = 0).values('id')
-	cnt_err = []
-	for build in builds:
-	    tasks = Task.objects.filter(build = build['id'], task_executed = 0).values('id', 'outcome')
-	    for task in tasks:
-		if (task['outcome'] not in [1, 2, 3, 5]):
-		    cnt_err.append(task['id'])
-	self.assertEqual(len(cnt_err), 0, msg = 'Errors for task id: %s' % cnt_err)
+        builds = Build.objects.filter(outcome = 0).values('id')
+        cnt_err = []
+        for build in builds:
+            tasks = Task.objects.filter(build = build['id'], task_executed = 0).values('id', 'outcome')
+            for task in tasks:
+                if (task['outcome'] not in [1, 2, 3, 5]):
+                    cnt_err.append(task['id'])
+        self.assertEqual(len(cnt_err), 0, msg = 'Errors for task id: %s' % cnt_err)
 
     # Key verification - tc=888
     @testcase(888)
     def test_Target_Installed_Package(self):
-	rows = Target_Installed_Package.objects.values('id', 'target_id', 'package_id')
-	cnt_err = []
-	for row in rows:
-	    target = Target.objects.filter(id = row['target_id']).values('id')
-	    package = Package.objects.filter(id = row['package_id']).values('id')
-	    if (not target or not package):
-		cnt_err.append(row['id'])
-	self.assertEqual(len(cnt_err), 0, msg = 'Errors for target installed package id: %s' % cnt_err)
+        rows = Target_Installed_Package.objects.values('id', 'target_id', 'package_id')
+        cnt_err = []
+        for row in rows:
+            target = Target.objects.filter(id = row['target_id']).values('id')
+            package = Package.objects.filter(id = row['package_id']).values('id')
+            if (not target or not package):
+                cnt_err.append(row['id'])
+        self.assertEqual(len(cnt_err), 0, msg = 'Errors for target installed package id: %s' % cnt_err)
 
     # Key verification - tc=889
     @testcase(889)
     def test_Task_Dependency(self):
-	rows = Task_Dependency.objects.values('id', 'task_id', 'depends_on_id')
-	cnt_err = []
-	for row in rows:
-	    task_id = Task.objects.filter(id = row['task_id']).values('id')
-	    depends_on_id = Task.objects.filter(id = row['depends_on_id']).values('id')
-	    if (not task_id or not depends_on_id):
-		cnt_err.append(row['id'])
-	self.assertEqual(len(cnt_err), 0, msg = 'Errors for task dependency id: %s' % cnt_err)
+        rows = Task_Dependency.objects.values('id', 'task_id', 'depends_on_id')
+        cnt_err = []
+        for row in rows:
+            task_id = Task.objects.filter(id = row['task_id']).values('id')
+            depends_on_id = Task.objects.filter(id = row['depends_on_id']).values('id')
+            if (not task_id or not depends_on_id):
+                cnt_err.append(row['id'])
+        self.assertEqual(len(cnt_err), 0, msg = 'Errors for task dependency id: %s' % cnt_err)
 
     # Check if build target file_name is populated only if is_image=true AND orm_build.outcome=0 then if the file exists and its size matches the file_size value
     ### Need to add the tc in the test run
     @testcase(1037)
     def test_Target_File_Name_Populated(self):
-	builds = Build.objects.filter(outcome = 0).values('id')
-	for build in builds:
-	    targets = Target.objects.filter(build_id = build['id'], is_image = 1).values('id')
-	    for target in targets:
-		target_files = Target_Image_File.objects.filter(target_id = target['id']).values('id', 'file_name', 'file_size')
-		cnt_err = []
-		for file_info in target_files:
-		    target_id = file_info['id']
-		    target_file_name = file_info['file_name']
-		    target_file_size = file_info['file_size']
-		    if (not target_file_name or not target_file_size):
-			cnt_err.append(target_id)
-		    else:
-			if (not os.path.exists(target_file_name)):
-			    cnt_err.append(target_id)
-			else:
-			    if (os.path.getsize(target_file_name) != target_file_size):
-				cnt_err.append(target_id)
-	self.assertEqual(len(cnt_err), 0, msg = 'Errors for target image file id: %s' % cnt_err)
+        builds = Build.objects.filter(outcome = 0).values('id')
+        for build in builds:
+            targets = Target.objects.filter(build_id = build['id'], is_image = 1).values('id')
+            for target in targets:
+                target_files = Target_Image_File.objects.filter(target_id = target['id']).values('id', 'file_name', 'file_size')
+                cnt_err = []
+                for file_info in target_files:
+                    target_id = file_info['id']
+                    target_file_name = file_info['file_name']
+                    target_file_size = file_info['file_size']
+                    if (not target_file_name or not target_file_size):
+                        cnt_err.append(target_id)
+                    else:
+                        if (not os.path.exists(target_file_name)):
+                            cnt_err.append(target_id)
+                        else:
+                            if (os.path.getsize(target_file_name) != target_file_size):
+                                cnt_err.append(target_id)
+        self.assertEqual(len(cnt_err), 0, msg = 'Errors for target image file id: %s' % cnt_err)
 
     # Key verification - tc=884
     @testcase(884)
     def test_Package_Dependency(self):
-	cnt_err = []
-	deps = Package_Dependency.objects.values('id', 'package_id', 'depends_on_id')
-	for dep in deps:
-	    if (dep['package_id'] == dep['depends_on_id']):
-		cnt_err.append(dep['id'])
-	self.assertEqual(len(cnt_err), 0, msg = 'Errors for package dependency id: %s' % cnt_err)
+        cnt_err = []
+        deps = Package_Dependency.objects.values('id', 'package_id', 'depends_on_id')
+        for dep in deps:
+            if (dep['package_id'] == dep['depends_on_id']):
+                cnt_err.append(dep['id'])
+        self.assertEqual(len(cnt_err), 0, msg = 'Errors for package dependency id: %s' % cnt_err)
 
     # Check if recipe name does not start with a number (0-9) - tc=838
     @testcase(838)
     def test_Recipe_Name(self):
-	recipes = Recipe.objects.values('id', 'name')
-	cnt_err = []
-	for recipe in recipes:
-	    if (recipe['name'][0].isdigit() is True):
-		cnt_err.append(recipe['id'])
-	self.assertEqual(len(cnt_err), 0, msg = 'Errors for recipe id: %s' % cnt_err)
+        recipes = Recipe.objects.values('id', 'name')
+        cnt_err = []
+        for recipe in recipes:
+            if (recipe['name'][0].isdigit() is True):
+                cnt_err.append(recipe['id'])
+        self.assertEqual(len(cnt_err), 0, msg = 'Errors for recipe id: %s' % cnt_err)
 
     # Check if recipe section matches the content of the SECTION variable (if set) in file_path - tc=839
     @testcase(839)
     def test_Recipe_DB_Section_Match_Recipe_File_Section(self):
-	recipes = Recipe.objects.values('id', 'section', 'file_path')
-	cnt_err = []
-	for recipe in recipes:
-	    file_path = self.fix_file_path(recipe['file_path'])
-	    file_exists = os.path.isfile(file_path)
-	    if (not file_path or (file_exists is False)):
-		cnt_err.append(recipe['id'])
-	    else:
-		file_section = self.recipe_parse(file_path, "SECTION = ")
-		db_section = recipe['section']
-		if file_section:
-		    if (db_section != file_section):
-			cnt_err.append(recipe['id'])
-	self.assertEqual(len(cnt_err), 0, msg = 'Errors for recipe id: %s' % cnt_err)
+        recipes = Recipe.objects.values('id', 'section', 'file_path')
+        cnt_err = []
+        for recipe in recipes:
+            file_path = self.fix_file_path(recipe['file_path'])
+            file_exists = os.path.isfile(file_path)
+            if (not file_path or (file_exists is False)):
+                cnt_err.append(recipe['id'])
+            else:
+                file_section = self.recipe_parse(file_path, "SECTION = ")
+                db_section = recipe['section']
+                if file_section:
+                    if (db_section != file_section):
+                        cnt_err.append(recipe['id'])
+        self.assertEqual(len(cnt_err), 0, msg = 'Errors for recipe id: %s' % cnt_err)
 
     # Check if recipe license matches the content of the LICENSE variable (if set) in file_path - tc=840
     @testcase(840)
     def test_Recipe_DB_License_Match_Recipe_File_License(self):
-	recipes = Recipe.objects.values('id', 'license', 'file_path')
-	cnt_err = []
-	for recipe in recipes:
-	    file_path = self.fix_file_path(recipe['file_path'])
-	    file_exists = os.path.isfile(file_path)
-	    if (not file_path or (file_exists is False)):
-		cnt_err.append(recipe['id'])
-	    else:
-		file_license = self.recipe_parse(file_path, "LICENSE = ")
-		db_license = recipe['license']
-		if file_license:
-		    if (db_license != file_license):
-			cnt_err.append(recipe['id'])
-	self.assertEqual(len(cnt_err), 0, msg = 'Errors for recipe id: %s' % cnt_err)
+        recipes = Recipe.objects.values('id', 'license', 'file_path')
+        cnt_err = []
+        for recipe in recipes:
+            file_path = self.fix_file_path(recipe['file_path'])
+            file_exists = os.path.isfile(file_path)
+            if (not file_path or (file_exists is False)):
+                cnt_err.append(recipe['id'])
+            else:
+                file_license = self.recipe_parse(file_path, "LICENSE = ")
+                db_license = recipe['license']
+                if file_license:
+                    if (db_license != file_license):
+                        cnt_err.append(recipe['id'])
+        self.assertEqual(len(cnt_err), 0, msg = 'Errors for recipe id: %s' % cnt_err)
 
     # Check if recipe homepage matches the content of the HOMEPAGE variable (if set) in file_path - tc=841
     @testcase(841)
     def test_Recipe_DB_Homepage_Match_Recipe_File_Homepage(self):
-	recipes = Recipe.objects.values('id', 'homepage', 'file_path')
-	cnt_err = []
-	for recipe in recipes:
-	    file_path = self.fix_file_path(recipe['file_path'])
-	    file_exists = os.path.isfile(file_path)
-	    if (not file_path or (file_exists is False)):
-		cnt_err.append(recipe['id'])
-	    else:
-		file_homepage = self.recipe_parse(file_path, "HOMEPAGE = ")
-		db_homepage = recipe['homepage']
-		if file_homepage:
-		    if (db_homepage != file_homepage):
-			cnt_err.append(recipe['id'])
-	self.assertEqual(len(cnt_err), 0, msg = 'Errors for recipe id: %s' % cnt_err)
+        recipes = Recipe.objects.values('id', 'homepage', 'file_path')
+        cnt_err = []
+        for recipe in recipes:
+            file_path = self.fix_file_path(recipe['file_path'])
+            file_exists = os.path.isfile(file_path)
+            if (not file_path or (file_exists is False)):
+                cnt_err.append(recipe['id'])
+            else:
+                file_homepage = self.recipe_parse(file_path, "HOMEPAGE = ")
+                db_homepage = recipe['homepage']
+                if file_homepage:
+                    if (db_homepage != file_homepage):
+                        cnt_err.append(recipe['id'])
+        self.assertEqual(len(cnt_err), 0, msg = 'Errors for recipe id: %s' % cnt_err)
 
     # Check if recipe bugtracker matches the content of the BUGTRACKER variable (if set) in file_path - tc=842
     @testcase(842)
     def test_Recipe_DB_Bugtracker_Match_Recipe_File_Bugtracker(self):
-	recipes = Recipe.objects.values('id', 'bugtracker', 'file_path')
-	cnt_err = []
-	for recipe in recipes:
-	    file_path = self.fix_file_path(recipe['file_path'])
-	    file_exists = os.path.isfile(file_path)
-	    if (not file_path or (file_exists is False)):
-		cnt_err.append(recipe['id'])
-	    else:
-		file_bugtracker = self.recipe_parse(file_path, "BUGTRACKER = ")
-		db_bugtracker = recipe['bugtracker']
-		if file_bugtracker:
-		    if (db_bugtracker != file_bugtracker):
-			cnt_err.append(recipe['id'])
-	self.assertEqual(len(cnt_err), 0, msg = 'Errors for recipe id: %s' % cnt_err)
+        recipes = Recipe.objects.values('id', 'bugtracker', 'file_path')
+        cnt_err = []
+        for recipe in recipes:
+            file_path = self.fix_file_path(recipe['file_path'])
+            file_exists = os.path.isfile(file_path)
+            if (not file_path or (file_exists is False)):
+                cnt_err.append(recipe['id'])
+            else:
+                file_bugtracker = self.recipe_parse(file_path, "BUGTRACKER = ")
+                db_bugtracker = recipe['bugtracker']
+                if file_bugtracker:
+                    if (db_bugtracker != file_bugtracker):
+                        cnt_err.append(recipe['id'])
+        self.assertEqual(len(cnt_err), 0, msg = 'Errors for recipe id: %s' % cnt_err)
 
     # Recipe key verification, recipe name does not depends on a recipe having the same name - tc=883
     @testcase(883)
     def test_Recipe_Dependency(self):
-	deps = Recipe_Dependency.objects.values('id', 'recipe_id', 'depends_on_id')
-	cnt_err = []
-	for dep in deps:
-	    if (not dep['recipe_id'] or not dep['depends_on_id']):
-		cnt_err.append(dep['id'])
-	    else:
-		name = Recipe.objects.filter(id = dep['recipe_id']).values('name')
-		dep_name = Recipe.objects.filter(id = dep['depends_on_id']).values('name')
-		if (name == dep_name):
-		    cnt_err.append(dep['id'])
-	self.assertEqual(len(cnt_err), 0, msg = 'Errors for recipe dependency id: %s' % cnt_err)
+        deps = Recipe_Dependency.objects.values('id', 'recipe_id', 'depends_on_id')
+        cnt_err = []
+        for dep in deps:
+            if (not dep['recipe_id'] or not dep['depends_on_id']):
+                cnt_err.append(dep['id'])
+            else:
+                name = Recipe.objects.filter(id = dep['recipe_id']).values('name')
+                dep_name = Recipe.objects.filter(id = dep['depends_on_id']).values('name')
+                if (name == dep_name):
+                    cnt_err.append(dep['id'])
+        self.assertEqual(len(cnt_err), 0, msg = 'Errors for recipe dependency id: %s' % cnt_err)
 
     # Check if package name does not start with a number (0-9) - tc=846
     @testcase(846)
     def test_Package_Name_For_Number(self):
-	packages = Package.objects.filter(~Q(size = -1)).values('id', 'name')
-	cnt_err = []
-	for package in packages:
-	    if (package['name'][0].isdigit() is True):
-		cnt_err.append(package['id'])
-	self.assertEqual(len(cnt_err), 0, msg = 'Errors for package id: %s' % cnt_err)
+        packages = Package.objects.filter(~Q(size = -1)).values('id', 'name')
+        cnt_err = []
+        for package in packages:
+            if (package['name'][0].isdigit() is True):
+                cnt_err.append(package['id'])
+        self.assertEqual(len(cnt_err), 0, msg = 'Errors for package id: %s' % cnt_err)
 
     # Check if package version starts with a number (0-9) - tc=847
     @testcase(847)
     def test_Package_Version_Starts_With_Number(self):
-	packages = Package.objects.filter(~Q(size = -1)).values('id', 'version')
-	cnt_err = []
-	for package in packages:
-	    if (package['version'][0].isdigit() is False):
-		cnt_err.append(package['id'])
-	self.assertEqual(len(cnt_err), 0, msg = 'Errors for package id: %s' % cnt_err)
+        packages = Package.objects.filter(~Q(size = -1)).values('id', 'version')
+        cnt_err = []
+        for package in packages:
+            if (package['version'][0].isdigit() is False):
+                cnt_err.append(package['id'])
+        self.assertEqual(len(cnt_err), 0, msg = 'Errors for package id: %s' % cnt_err)
 
     # Check if package revision starts with 'r' - tc=848
     @testcase(848)
     def test_Package_Revision_Starts_With_r(self):
-	packages = Package.objects.filter(~Q(size = -1)).values('id', 'revision')
-	cnt_err = []
-	for package in packages:
-	    if (package['revision'][0].startswith("r") is False):
-		cnt_err.append(package['id'])
-	self.assertEqual(len(cnt_err), 0, msg = 'Errors for package id: %s' % cnt_err)
+        packages = Package.objects.filter(~Q(size = -1)).values('id', 'revision')
+        cnt_err = []
+        for package in packages:
+            if (package['revision'][0].startswith("r") is False):
+                cnt_err.append(package['id'])
+        self.assertEqual(len(cnt_err), 0, msg = 'Errors for package id: %s' % cnt_err)
 
     # Check the validity of the package build_id
     ### TC must be added in test run
     @testcase(1038)
     def test_Package_Build_Id(self):
-	packages = Package.objects.filter(~Q(size = -1)).values('id', 'build_id')
-	cnt_err = []
-	for package in packages:
-	    build_id = Build.objects.filter(id = package['build_id']).values('id')
-	    if (not build_id):
-		cnt_err.append(package['id'])
-	self.assertEqual(len(cnt_err), 0, msg = 'Errors for package id: %s' % cnt_err)
+        packages = Package.objects.filter(~Q(size = -1)).values('id', 'build_id')
+        cnt_err = []
+        for package in packages:
+            build_id = Build.objects.filter(id = package['build_id']).values('id')
+            if (not build_id):
+                cnt_err.append(package['id'])
+        self.assertEqual(len(cnt_err), 0, msg = 'Errors for package id: %s' % cnt_err)
 
     # Check the validity of package recipe_id
     ### TC must be added in test run
     @testcase(1039)
     def test_Package_Recipe_Id(self):
-	packages = Package.objects.filter(~Q(size = -1)).values('id', 'recipe_id')
-	cnt_err = []
-	for package in packages:
-	    recipe_id = Recipe.objects.filter(id = package['recipe_id']).values('id')
-	    if (not recipe_id):
-		cnt_err.append(package['id'])
-	self.assertEqual(len(cnt_err), 0, msg = 'Errors for package id: %s' % cnt_err)
+        packages = Package.objects.filter(~Q(size = -1)).values('id', 'recipe_id')
+        cnt_err = []
+        for package in packages:
+            recipe_id = Recipe.objects.filter(id = package['recipe_id']).values('id')
+            if (not recipe_id):
+                cnt_err.append(package['id'])
+        self.assertEqual(len(cnt_err), 0, msg = 'Errors for package id: %s' % cnt_err)
 
     # Check if package installed_size field is not null
     ### TC must be aded in test run
     @testcase(1040)
     def test_Package_Installed_Size_Not_NULL(self):
-	packages = Package.objects.filter(installed_size__isnull = True).values('id')
-	cnt_err = []
-	for package in packages:
-	    cnt_err.append(package['id'])
-	self.assertEqual(len(cnt_err), 0, msg = 'Errors for package id: %s' % cnt_err)
+        packages = Package.objects.filter(installed_size__isnull = True).values('id')
+        cnt_err = []
+        for package in packages:
+            cnt_err.append(package['id'])
+        self.assertEqual(len(cnt_err), 0, msg = 'Errors for package id: %s' % cnt_err)
 
     # Check if all layers requests return exit code is 200 - tc=843
     @testcase(843)
     def test_Layers_Requests_Exit_Code(self):
-	layers = Layer.objects.values('id', 'layer_index_url')
-	cnt_err = []
-	for layer in layers:
-	    resp = urllib.urlopen(layer['layer_index_url'])
-	    if (resp.getcode() != 200):
-		cnt_err.append(layer['id'])
-	self.assertEqual(len(cnt_err), 0, msg = 'Errors for layer id: %s' % cnt_err)
+        layers = Layer.objects.values('id', 'layer_index_url')
+        cnt_err = []
+        for layer in layers:
+            resp = urllib.urlopen(layer['layer_index_url'])
+            if (resp.getcode() != 200):
+                cnt_err.append(layer['id'])
+        self.assertEqual(len(cnt_err), 0, msg = 'Errors for layer id: %s' % cnt_err)
 
     # Check if the output of bitbake-layers show_layers matches the info from database - tc=895
     @testcase(895)
     def test_Layers_Show_Layers(self):
-	layers = Layer.objects.values('id', 'name', 'local_path')
-	cmd = commands.getoutput('bitbake-layers show_layers')
-	cnt_err = []
-	for layer in layers:
-	    if (layer['name'] or layer['local_path']) not in cmd:
-		cnt_err.append(layer['id'])
-	self.assertEqual(len(cnt_err), 0, msg = 'Errors for layer id: %s' % cnt_err)
+        layers = Layer.objects.values('id', 'name', 'local_path')
+        cmd = commands.getoutput('bitbake-layers show_layers')
+        cnt_err = []
+        for layer in layers:
+            if (layer['name'] or layer['local_path']) not in cmd:
+                cnt_err.append(layer['id'])
+        self.assertEqual(len(cnt_err), 0, msg = 'Errors for layer id: %s' % cnt_err)
 
     # Check if django server starts regardless of the timezone set on the machine - tc=905
     @testcase(905)
     def test_Start_Django_Timezone(self):
         current_path = os.getcwd()
-	zonefilelist = []
+        zonefilelist = []
         ZONEINFOPATH = '/usr/share/zoneinfo/'
-	os.chdir("../bitbake/lib/toaster/")
-	cnt_err = 0
-	for filename in os.listdir(ZONEINFOPATH):
-	    if os.path.isfile(os.path.join(ZONEINFOPATH, filename)):
-		zonefilelist.append(filename)
-	for k in range(len(zonefilelist)):
-	    if k <= 5:
-		files = zonefilelist[k]
-		os.system("export TZ="+str(files)+"; python manage.py runserver > /dev/null 2>&1 &")
-    		time.sleep(3)
-		pid = subprocess.check_output("ps aux | grep '[/u]sr/bin/python manage.py runserver' | awk '{print $2}'", shell = True)
-		if pid:
-		    os.system("kill -9 "+str(pid))
-		else:
-		    cnt_err.append(zonefilelist[k])
-	self.assertEqual(cnt_err, 0, msg = 'Errors django server does not start with timezone: %s' % cnt_err)
+        os.chdir("../bitbake/lib/toaster/")
+        cnt_err = 0
+        for filename in os.listdir(ZONEINFOPATH):
+            if os.path.isfile(os.path.join(ZONEINFOPATH, filename)):
+                zonefilelist.append(filename)
+        for k in range(len(zonefilelist)):
+            if k <= 5:
+                files = zonefilelist[k]
+                os.system("export TZ="+str(files)+"; python manage.py runserver > /dev/null 2>&1 &")
+                    time.sleep(3)
+                pid = subprocess.check_output("ps aux | grep '[/u]sr/bin/python manage.py runserver' | awk '{print $2}'", shell = True)
+                if pid:
+                    os.system("kill -9 "+str(pid))
+                else:
+                    cnt_err.append(zonefilelist[k])
+        self.assertEqual(cnt_err, 0, msg = 'Errors django server does not start with timezone: %s' % cnt_err)
         os.chdir(current_path)
diff --git a/meta/lib/oeqa/selftest/bbtests.py b/meta/lib/oeqa/selftest/bbtests.py
index aeccbf5..b301d8f 100644
--- a/meta/lib/oeqa/selftest/bbtests.py
+++ b/meta/lib/oeqa/selftest/bbtests.py
@@ -120,63 +120,63 @@ class BitbakeTests(oeSelfTest):
 
     @testcase(1028)
     def test_environment(self):
-	self.append_config("TEST_ENV=\"localconf\"")
-	result = runCmd('bitbake -e | grep TEST_ENV=')
-	self.assertTrue('localconf' in result.output)
-	self.remove_config("TEST_ENV=\"localconf\"")
+        self.append_config("TEST_ENV=\"localconf\"")
+        result = runCmd('bitbake -e | grep TEST_ENV=')
+        self.assertTrue('localconf' in result.output)
+        self.remove_config("TEST_ENV=\"localconf\"")
 
     @testcase(1029)
     def test_dry_run(self):
-	result = runCmd('bitbake -n m4-native')
-	self.assertEqual(0, result.status)
+        result = runCmd('bitbake -n m4-native')
+        self.assertEqual(0, result.status)
 
     @testcase(1030)
     def test_just_parse(self):
-	result = runCmd('bitbake -p')
-	self.assertEqual(0, result.status)
+        result = runCmd('bitbake -p')
+        self.assertEqual(0, result.status)
 
     @testcase(1031)
     def test_version(self):
-	result = runCmd('bitbake -s | grep wget')
-	find = re.search("wget *:([0-9a-zA-Z\.\-]+)", result.output)
-	self.assertTrue(find) 
+        result = runCmd('bitbake -s | grep wget')
+        find = re.search("wget *:([0-9a-zA-Z\.\-]+)", result.output)
+        self.assertTrue(find)
 
     @testcase(1032)
     def test_prefile(self):
-	preconf = os.path.join(self.builddir, 'conf/prefile.conf')
-	self.track_for_cleanup(preconf)
-	ftools.write_file(preconf ,"TEST_PREFILE=\"prefile\"")
-	result = runCmd('bitbake -r conf/prefile.conf -e | grep TEST_PREFILE=')
-	self.assertTrue('prefile' in result.output)
-	self.append_config("TEST_PREFILE=\"localconf\"")
-	result = runCmd('bitbake -r conf/prefile.conf -e | grep TEST_PREFILE=')
-	self.assertTrue('localconf' in result.output)
-	self.remove_config("TEST_PREFILE=\"localconf\"")
+        preconf = os.path.join(self.builddir, 'conf/prefile.conf')
+        self.track_for_cleanup(preconf)
+        ftools.write_file(preconf ,"TEST_PREFILE=\"prefile\"")
+        result = runCmd('bitbake -r conf/prefile.conf -e | grep TEST_PREFILE=')
+        self.assertTrue('prefile' in result.output)
+        self.append_config("TEST_PREFILE=\"localconf\"")
+        result = runCmd('bitbake -r conf/prefile.conf -e | grep TEST_PREFILE=')
+        self.assertTrue('localconf' in result.output)
+        self.remove_config("TEST_PREFILE=\"localconf\"")
 
     @testcase(1033)
     def test_postfile(self):
-	postconf = os.path.join(self.builddir, 'conf/postfile.conf')
-	self.track_for_cleanup(postconf)
-	ftools.write_file(postconf , "TEST_POSTFILE=\"postfile\"")
-	self.append_config("TEST_POSTFILE=\"localconf\"")
-	result = runCmd('bitbake -R conf/postfile.conf -e | grep TEST_POSTFILE=')
-	self.assertTrue('postfile' in result.output)
-	self.remove_config("TEST_POSTFILE=\"localconf\"")
+        postconf = os.path.join(self.builddir, 'conf/postfile.conf')
+        self.track_for_cleanup(postconf)
+        ftools.write_file(postconf , "TEST_POSTFILE=\"postfile\"")
+        self.append_config("TEST_POSTFILE=\"localconf\"")
+        result = runCmd('bitbake -R conf/postfile.conf -e | grep TEST_POSTFILE=')
+        self.assertTrue('postfile' in result.output)
+        self.remove_config("TEST_POSTFILE=\"localconf\"")
 
     @testcase(1034)
     def test_checkuri(self):
-	result = runCmd('bitbake -c checkuri m4')
-	self.assertEqual(0, result.status)
+        result = runCmd('bitbake -c checkuri m4')
+        self.assertEqual(0, result.status)
 
     @testcase(1035)
     def test_continue(self):
-	self.write_recipeinc('man',"\ndo_fail_task () {\nexit 1 \n}\n\naddtask do_fail_task before do_fetch\n" )
-	runCmd('bitbake -c cleanall man xcursor-transparent-theme')
-	result = runCmd('bitbake man xcursor-transparent-theme -k', ignore_status=True)
-	errorpos = result.output.find('ERROR: Function failed: do_fail_task')
-	manver = re.search("NOTE: recipe xcursor-transparent-theme-(.*?): task do_unpack: Started", result.output)
-	continuepos = result.output.find('NOTE: recipe xcursor-transparent-theme-%s: task do_unpack: Started' % manver.group(1))
-	self.assertLess(errorpos,continuepos)
+        self.write_recipeinc('man',"\ndo_fail_task () {\nexit 1 \n}\n\naddtask do_fail_task before do_fetch\n" )
+        runCmd('bitbake -c cleanall man xcursor-transparent-theme')
+        result = runCmd('bitbake man xcursor-transparent-theme -k', ignore_status=True)
+        errorpos = result.output.find('ERROR: Function failed: do_fail_task')
+        manver = re.search("NOTE: recipe xcursor-transparent-theme-(.*?): task do_unpack: Started", result.output)
+        continuepos = result.output.find('NOTE: recipe xcursor-transparent-theme-%s: task do_unpack: Started' % manver.group(1))
+        self.assertLess(errorpos,continuepos)
 
     @testcase(1119)
     def test_non_gplv3(self):
-- 
2.1.0




More information about the Openembedded-core mailing list