From 60d54449c12ed139911df35a9983c14019262d37 Mon Sep 17 00:00:00 2001
From: asterix <asterix@38d2e660-2303-0410-9eaa-f027e97ec537>
Date: Thu, 22 May 2008 08:35:13 +0000
Subject: [PATCH] Mark these test module with warning.

git-svn-id: https://src.develer.com/svnoss/bertos/trunk@1321 38d2e660-2303-0410-9eaa-f027e97ec537
---
 bertos/drv/flash25_test.c   |  2 ++
 bertos/drv/lcd_hd44_test.c  |  9 +++++++--
 bertos/drv/timer_test.c     | 21 ++++++++++---------
 bertos/kern/kfile_test.c    |  2 ++
 bertos/kern/proc_test.c     | 40 +++++++++++++++++++++++++++++++++++++
 bertos/mware/sprintf_test.c | 20 +++++++++++++------
 6 files changed, 77 insertions(+), 17 deletions(-)

diff --git a/bertos/drv/flash25_test.c b/bertos/drv/flash25_test.c
index 691c9113..a221992f 100644
--- a/bertos/drv/flash25_test.c
+++ b/bertos/drv/flash25_test.c
@@ -49,6 +49,8 @@
 
 #include <cfg/debug.h>
 
+#warning FIXME:Review and refactor this test..
+
 bool flash25_test(KFile *channel)
 {
 	KFileFlash25 fd;
diff --git a/bertos/drv/lcd_hd44_test.c b/bertos/drv/lcd_hd44_test.c
index 65107b85..a84c156c 100644
--- a/bertos/drv/lcd_hd44_test.c
+++ b/bertos/drv/lcd_hd44_test.c
@@ -39,8 +39,6 @@
 
 #include "hw_lcd.h"
 
-#include <cfg/arch_config.h>
-
 #include <drv/timer.h>
 #include <drv/lcd_hd44.h>
 
@@ -76,6 +74,11 @@ void lcd_hw_test(void)
 }
 
 
+#ifdef _TEST
+
+#include <drv/timer.c>
+#include <drv/lcd_hd44.c>
+
 int main(void)
 {
 		lcd_hw_test();
@@ -86,3 +89,5 @@ int main(void)
 
 		return 0;
 }
+
+#endif
diff --git a/bertos/drv/timer_test.c b/bertos/drv/timer_test.c
index 3e9338a8..5fc42570 100644
--- a/bertos/drv/timer_test.c
+++ b/bertos/drv/timer_test.c
@@ -36,17 +36,12 @@
  * \author Bernardo Innocenti <bernie@develer.com>
  */
 
-#ifdef _TEST
-	#include "timer.c"
-	#include "mware/event.c"
-	#include "os/hptime.c"
+#include <cfg/test.h>
 
-#else
-	#include <drv/timer.c>
-	#include <mware/event.c>
-	#include <drv/wdt.h>
+#include <drv/timer.h>
+#include <drv/wdt.h>
 
-#endif
+#include <mware/event.h>
 
 
 static void timer_test_constants(void)
@@ -131,6 +126,12 @@ static void timer_test_poll(void)
 	}
 }
 
+#ifdef _TEST
+
+#include "timer.c"
+#include "mware/event.c"
+#include "os/hptime.c"
+
 int main(void)
 {
 	wdt_init(7);
@@ -141,3 +142,5 @@ int main(void)
 	timer_test_poll();
 	return 0;
 }
+#endif
+
diff --git a/bertos/kern/kfile_test.c b/bertos/kern/kfile_test.c
index 76e78092..20da9ec9 100644
--- a/bertos/kern/kfile_test.c
+++ b/bertos/kern/kfile_test.c
@@ -47,6 +47,8 @@
 #include <mware/formatwr.h>
 #include <string.h>
 
+#warning TODO:Refactor this module..
+
 /**
  * KFile read/write subtest.
  * Try to write/read in the same \a f file location \a size bytes.
diff --git a/bertos/kern/proc_test.c b/bertos/kern/proc_test.c
index 4c2f3126..d59bc749 100644
--- a/bertos/kern/proc_test.c
+++ b/bertos/kern/proc_test.c
@@ -1,7 +1,47 @@
+/**
+ * \file
+ * <!--
+ * This file is part of BeRTOS.
+ *
+ * Bertos is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA
+ *
+ * As a special exception, you may use this file as part of a free software
+ * library without restriction.  Specifically, if other files instantiate
+ * templates or use macros or inline functions from this file, or you compile
+ * this file and link it with other files to produce an executable, this
+ * file does not by itself cause the resulting executable to be covered by
+ * the GNU General Public License.  This exception does not however
+ * invalidate any other reasons why the executable file might be covered by
+ * the GNU General Public License.
+ *
+ * Copyright 2005 Develer S.r.l. (http://www.develer.com/)
+ * -->
+ *
+ *
+ * \brief Test kernel process.
+ *
+ * \version $Id$
+ *
+ * \author Daniele Basile <asterix@develer.com>
+ */
 
 #include <kern/proc.h>
 #include <drv/timer.h>
 
+#warning FIXME: Review this test and refactor for all target.
+
 /**
  * Proc scheduling test subthread 1
  */
diff --git a/bertos/mware/sprintf_test.c b/bertos/mware/sprintf_test.c
index 1bcedfd3..3534953a 100644
--- a/bertos/mware/sprintf_test.c
+++ b/bertos/mware/sprintf_test.c
@@ -36,18 +36,25 @@
  * \author Bernardo Innocenti <bernie@develer.com>
  */
 
-#include "sprintf.c"
-#include "formatwr.c"
-#include "hex.c"
+
 #include <cfg/compiler.h>
+#include <cfg/test.h>
+#include <cfg/debug.h>
+
 #include <mware/pgm.h>
+
 #include <stdio.h>
 
-#include <assert.h> /* assert() */
 #include <string.h> /* strcmp() */
 
+#warning FIXME:Review and refactor this test..
 
-int main(UNUSED_ARG(int, argc), UNUSED_ARG(char **,argv))
+#ifdef _TEST
+#include "sprintf.c"
+#include "formatwr.c"
+#include "hex.c"
+
+int main(void)
 {
 	char buf[256];
 	static const char test_string[] = "Hello, world!\n";
@@ -80,7 +87,6 @@ int main(UNUSED_ARG(int, argc), UNUSED_ARG(char **,argv))
 	TEST("%-8.2f", -123.456,    "-123.46 ");
 	TEST("%8.0f",  -123.456,    "    -123");
 
-#undef TEST
 
 	/*
 	 * Stress tests.
@@ -94,3 +100,5 @@ int main(UNUSED_ARG(int, argc), UNUSED_ARG(char **,argv))
 	return 0;
 }
 
+#endif /* _TEST */
+
-- 
2.25.1