Hi Nuwan, I raised a JIRA to track this at [1]. Also, when it comes to CopyNewVersionTestCase I believe this scenario is covered under test cases at [2]. This is another issue we observed, You guys have to take this into consideration when writing test cases to avoid duplication of test cases.
[1] - https://wso2.org/jira/browse/APIMANAGER-4727 [2] - https://github.com/wso2/product-apim/tree/master/modules/integration/tests-integration/tests-backend/src/test/java/org/wso2/am/integration/tests/version Regards, On Mon, May 2, 2016 at 10:44 PM, Aparna Karunarathna <[email protected]> wrote: > Hi Nuwan, > > We have seen a lot of discrepancies in the automation test, so it would be > good if you guys can fix those before the release. > > Regards, > Aparna > > On Mon, May 2, 2016 at 6:36 PM, Nuwan Dias <[email protected]> wrote: > >> Not sure why this was disabled. Maybe its because it uses the deprecated >> YouTube API (it no longer works). These are very old tests coming even >> before the wso2-dev repo days. So can't recall its entire history. >> >> However, there is a new test case called CopyNewVersionTestCase that >> covers this scenario. So we haven't lost on coverage by disabling it. >> >> We have ~500 tests in the product now. And some of these are pretty old. >> Yes, we need to refine and remove obsolete ones and we do that as we >> encounter them. We will do this refining as priorities allow us to. >> >> Thanks, >> NuwanD. >> >> On Mon, May 2, 2016 at 5:59 PM, Dharshana Warusavitharana < >> [email protected]> wrote: >> >>> Hi APIM team, >>> >>> Please add comments when you are disabling tests like this. Or remove >>> test completely if they are obsolete please remove them from the suite. >>> And i see invalid test method description here. >>> >>> Please fix these ASAP. >>> >>> Thank you, >>> Dharshana. >>> >>> On Mon, May 2, 2016 at 5:19 PM, Yasassri Ratnayake <[email protected]> >>> wrote: >>> >>>> Hi APIM Team, >>>> >>>> Observed the following test case at [1]. As shown below most test >>>> mrethods are disabled except for 2 in this test class and I couldn't find >>>> any comment explaining why these tests were disabled. Can we look into >>>> these and get them enabled? >>>> >>>> >>>> @Test(groups = {"wso2.am"}, description = "API visibility", enabled = >>>> false) >>>> public void copyAPILifeCycleTestCase() throws Exception { >>>> >>>> >>>> [1] - >>>> https://github.com/wso2/product-apim/blob/master/modules/integration/tests-integration/tests-backend/src/test/java/org/wso2/am/integration/tests/other/APIApplicationLifeCycleTestCase.java#L363 >>>> >>>> >>>> With Regards, >>>> -- >>>> Yasassri Ratnayake >>>> Software Engineer - QA >>>> WSO2 Inc ; http://wso2.com >>>> lean.enterprise.middleware >>>> *Mobile : +94715933168 <%2B94715933168>* >>>> *Blogs : http://yasassriratnayake.blogspot.com >>>> <http://yasassriratnayake.blogspot.com/>* >>>> * http://wso2logs.blogspot.com >>>> <http://wso2logs.blogspot.com>* >>>> >>> >>> >>> >>> -- >>> >>> Dharshana Warusavitharana >>> Senior Software Engineer , Test Automation >>> WSO2 Inc. http://wso2.com >>> email : [email protected] <[email protected]> >>> Tel : +94 11 214 5345 >>> Fax :+94 11 2145300 >>> cell : +94770342233 >>> blog : http://dharshanaw.blogspot.com >>> >>> lean . enterprise . middleware >>> >> >> >> >> -- >> Nuwan Dias >> >> Technical Lead - WSO2, Inc. http://wso2.com >> email : [email protected] >> Phone : +94 777 775 729 >> > > > > -- > *Regards,* > > *Aparna Karunarathna.* > > > *Associate Technical Lead - QAWSO2 Inc.Mobile: 0714002533* > -- Yasassri Ratnayake Software Engineer - QA WSO2 Inc ; http://wso2.com lean.enterprise.middleware *Mobile : +94715933168* *Blogs : http://yasassriratnayake.blogspot.com <http://yasassriratnayake.blogspot.com/>* * http://wso2logs.blogspot.com <http://wso2logs.blogspot.com>*
_______________________________________________ Dev mailing list [email protected] http://wso2.org/cgi-bin/mailman/listinfo/dev
