Hi Lucian,
On Fri, May 16, 2014 at 11:44 AM, Lucian Musat
<georgex.l.mu...@intel.com> wrote:
>
> class SmartRepoTest(SmartTest):
> @@ -54,11 +54,12 @@ class SmartRepoTest(SmartTest):
> @classmethod
> def tearDownClass(self):
> self.repo_server.stop()
> + smartsRepo = SmartRepoTest
>
Again, this line doesn't make sense and it probably shouldn't be here.
> class SyslogTest(oeRuntimeTest):
>
> - @skipUnlessPassed("test_ssh")
> - def test_syslog_help(self):
> - (status,output) = self.target.run('/sbin/syslogd --help')
> - self.assertEqual(status, 0, msg="status and output: %s and %s" %
> (status,output))
> + @skipUnlessPassed("test_ssh_tc_224")
> + def test_syslog_help_tc_202(self):
> + (status,output) = self.target.run('/sbin/syslogd --help')
> + self.assertEqual(status, 0, msg="status and output: %s and
> %s" % (status,output))
>
There are some whitespaces/tabs added here that probably shouldn't be.
Please also add a description why this patch is needed. Why do we need
to add testopia numbers to test names, how is it useful?
Also, isn't there any other way we could map these tests to testopia testcases?
Cheers,
Stefan
--
_______________________________________________
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core
