From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by yocto-www.yoctoproject.org (Postfix, from userid 118) id DDFBEE00A7F; Mon, 10 Aug 2015 09:26:16 -0700 (PDT) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on yocto-www.yoctoproject.org X-Spam-Level: X-Spam-Status: No, score=-6.9 required=5.0 tests=BAYES_00,RCVD_IN_DNSWL_HI autolearn=ham version=3.3.1 X-Spam-HAM-Report: * -5.0 RCVD_IN_DNSWL_HI RBL: Sender listed at http://www.dnswl.org/, high * trust * [134.134.136.24 listed in list.dnswl.org] * -1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1% * [score: 0.0000] Received: from mga09.intel.com (mga09.intel.com [134.134.136.24]) by yocto-www.yoctoproject.org (Postfix) with ESMTP id A0C35E00A7B for ; Mon, 10 Aug 2015 09:26:15 -0700 (PDT) Received: from orsmga001.jf.intel.com ([10.7.209.18]) by orsmga102.jf.intel.com with ESMTP; 10 Aug 2015 09:26:15 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.15,646,1432623600"; d="scan'208";a="745762666" Received: from linux.intel.com ([10.23.219.25]) by orsmga001.jf.intel.com with ESMTP; 10 Aug 2015 09:26:15 -0700 Received: from linux.intel.com (unknown [10.237.72.65]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by linux.intel.com (Postfix) with ESMTP id 6CBE16A4005; Mon, 10 Aug 2015 09:25:27 -0700 (PDT) Date: Mon, 10 Aug 2015 19:26:09 +0300 From: Ed Bartosh To: Sujith H Message-ID: <20150810162609.GA7124@linux.intel.com> References: <1439197390-9570-1-git-send-email-sujith.h@gmail.com> MIME-Version: 1.0 In-Reply-To: <1439197390-9570-1-git-send-email-sujith.h@gmail.com> Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo User-Agent: Mutt/1.5.21 (2010-09-15) Cc: toaster@yoctoproject.org Subject: Re: [review-request][PATCH][v2] toastergui: Add tests for xhr_importlayer X-BeenThere: toaster@yoctoproject.org X-Mailman-Version: 2.1.13 Precedence: list Reply-To: ed.bartosh@linux.intel.com List-Id: Web based interface for BitBake List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 10 Aug 2015 16:26:16 -0000 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Hi Sujith, Thank you for updating your patch. My comments are below. On Mon, Aug 10, 2015 at 02:33:10PM +0530, Sujith H wrote: > Inorder to make sure if layer name which we import is already > available in toaster database, a test case has been included. > This would help user to identify if layer name which has been > provided in the import layer web page already exists or not. > > Signed-off-by: Sujith Haridasan > Signed-off-by: Sujith Haridasan > --- > bitbake/lib/toaster/toastergui/tests.py | 32 ++++++++++++++++++++++++++++++++ > 1 file changed, 32 insertions(+) > > diff --git a/bitbake/lib/toaster/toastergui/tests.py b/bitbake/lib/toaster/toastergui/tests.py > index 93a85ac..2986e57 100644 > --- a/bitbake/lib/toaster/toastergui/tests.py > +++ b/bitbake/lib/toaster/toastergui/tests.py > @@ -148,3 +148,35 @@ class ViewTests(TestCase): > # After "typeing" the alpabet we should have result true > # from each of the urls > self.assertTrue(results) > + > +class XHRImportLayerTestCase(ViewTests): > + """ Tests to verify xhr_importlayer API""" > + def setUp(self): > + super(XHRImportLayerTestCase, self).setUp() Do you really need a new class for your test case? If you add it to ViewTests you don't need above 4 lines. > + > + def test_import(self): > + """Tests to verify import layer already exist, if the imported layer is ok, response > + for empty data, response for html tag in data""" Please, split dosctring to subject line and the body to make output of ./manage.py test -v2 toastergui meaningful. > + #Test for importing an already existing layer > + data = {'vcs_url' : "git://git.example.com/test", I'd suggest to rename data->args. > + 'name' : "base-layer", 'git_ref': "c12b9596afd236116b25ce26dbe0d793de9dc7ce", > + 'project_id': 1, 'dir_path' : "/path/in/repository"} > + result = self.client.post(reverse('xhr_importlayer'), data) > + self.assertEqual(result.status_code, 200) > + self.assertTrue("hint-layer-exists-with-different-url" in result.content) It would be better to convert result from json using json.load as it's done in other test cases, i.e. data = json.loads(response.content) self.assertNotEqual(data["error"], "ok") > + #Test to verify import of a layer successful > + data['name'] = "meta-oe" > + result = self.client.post(reverse('xhr_importlayer'), data) > + self.assertTrue('"error": "ok"' in result.content) > + > + #Test for html tag in the data > + data['<'] = "testing html tag" > + result = self.client.post(reverse('xhr_importlayer'), data) > + self.assertTrue('"error": "Invalid character <"' in result.content) > + > + #Empty data passed > + data = {} > + result = self.client.post(reverse('xhr_importlayer'), data) > + self.assertTrue('"error": "Missing parameters; requires vcs_url, \ > + name, git_ref and project_id"' in result.content) Would it be enough to check that error is not 'ok'? If you change a message you will not have to change test code. Please, make your code similar to the existing code. Use the same vairable names, checks, etc. It would make test code look and behave in a consistent way. -- Regards, Ed