GitHub user snoopdave opened a pull request:

    https://github.com/apache/usergrid/pull/439

    Results of ignored tests audit

    I've completed the review of ignored tests made some changes:
    * Fixed a dozen or so tests and remove the ignore annotation from them
    * Used JUnit Categories to mark some tests as StressTests and some as 
Experimental tests, and exclude them
    * Marked some tests as "Pending merge of export-feature"
    * Marked some tests as "Pending re-enable of delete functionality"
    * Marked some tests as "Pending implementation of backpressure"
    * All other ignore tests are now linked to JIRA issues
    
    I created these JIRA issues to fix the remaining ignored tests:
    * USERGRID-1119 Fix GroupResourceIT.postGroupActivity() test
    * USERGRID-1115 Fix AdminUsersIT tests
    * USERGRID-1106 Fix test: clientCredentialsFlowWithHeaderAuthorization()
    * USERGRID-1113 Improve NotificationsService test coverage
    * USERGRID-1118 Fix EventsResource and IT
    * USERGRID-1116 Fix MessagesIT tests
    * USERGRID-1120 Fix CounterIT.testIncrementAndDecrement() test
    
    See also:
    https://issues.apache.org/jira/browse/USERGRID-876

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/apache/usergrid ignores

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/usergrid/pull/439.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #439
    
----
commit f1aa4506da471ad01ce0fb0c57af9cdc33ea1a70
Author: Dave Johnson <[email protected]>
Date:   2015-11-13T15:51:26Z

    Add references to JIRA issues in @Ignore annotations

commit 1434deaace2bd9bb9a237b1a5da9fbaab70ec1e7
Author: Dave Johnson <[email protected]>
Date:   2015-11-13T15:54:13Z

    Remove some unnecessary ignore annotations and some unnecessary code.

commit 59d2a71c761e1419d143acafd02a08f80a3293eb
Author: Dave Johnson <[email protected]>
Date:   2015-11-13T17:52:39Z

    Fix no-org-delete and test.

commit 5c85b7598586abe0b11e29918b09be11addec831
Author: Dave Johnson <[email protected]>
Date:   2015-11-13T21:32:55Z

    Un-ignore some tests that now appear to be working (again).

commit 40cd37988b72ceaabe7d265eb7c4e5c6e0dd1e95
Author: Dave Johnson <[email protected]>
Date:   2015-11-16T14:18:05Z

    Unignoring and fixing tests more tests.

commit bba77f0af590a634415694d8b1f59e65bb85b2ff
Author: Dave Johnson <[email protected]>
Date:   2015-11-16T18:10:44Z

    Correct Jacoco profile command arg

commit f2cbf1a98c723fe75ec6ff967e74c95b09aa82a3
Author: Dave Johnson <[email protected]>
Date:   2015-11-16T18:25:58Z

    Tests run more reliably with 20 iterations.

commit 13853662ee07ab4ec5b812fd3d5fe42ef8b99d9a
Author: Dave Johnson <[email protected]>
Date:   2015-11-16T18:53:02Z

    Merge branch 'master' into ignores

commit f62e0aa5869785b459571cab306579ca21bf0be1
Author: Dave Johnson <[email protected]>
Date:   2015-11-17T12:51:29Z

    Fix spelling error in class names.

commit d1f17107631c3e7c83ae0efccb371e3979c174ed
Author: Dave Johnson <[email protected]>
Date:   2015-11-17T12:52:21Z

    Fix spelling error in class names.

commit 99b932ada3809a33ad8a824b9177d427edb6feba
Author: Dave Johnson <[email protected]>
Date:   2015-11-17T12:54:39Z

    Use JUnut Categories to exclude too heavy (aka stress) tests and 
experimental tests.

commit 41dde128cc04d211a29bf5af13520a7755246325
Author: Dave Johnson <[email protected]>
Date:   2015-11-17T12:55:32Z

    Remove unnecessary ignores in some places, add new ignores for broken tests 
that were once excluded via moms.

commit 4cf69e6f0498dbdc55e55739d38dd78f32a87e58
Author: Dave Johnson <[email protected]>
Date:   2015-11-17T13:12:30Z

    Add an additional reindex/wait to test that fails intermittently.

commit 6af43dc9b5c3963eefb7a93afb029a4e365cc65f
Author: Dave Johnson <[email protected]>
Date:   2015-11-17T14:30:57Z

    Adding JIRA issue links to ignore annotations

commit a9c1e840172fb399a1b4212b4d9f9237b271ff61
Author: Dave Johnson <[email protected]>
Date:   2015-11-17T14:31:30Z

    Further categorization of tests.

commit 22c29fb7ca455c05739d1c2724e33dd0429bab1a
Author: Dave Johnson <[email protected]>
Date:   2015-11-17T14:32:08Z

    Add pending comment to import/export tests that are awaiting merge of the 
export-feature branch.

commit 68be7d7027c51940445a43bed5fcb3270ff75846
Author: Dave Johnson <[email protected]>
Date:   2015-11-17T15:13:38Z

    Adding more JIRA links; all but 10 ignores are now linked to JIRA issues.

commit 43f7ee05b20b57f37f28eac5ff3baa41cee1326c
Author: Dave Johnson <[email protected]>
Date:   2015-11-17T15:52:47Z

    Un-ignore some working tests.

commit 78f3da6fcb194190af1d67151bb8d419f620c22e
Author: Dave Johnson <[email protected]>
Date:   2015-11-17T15:53:38Z

    Adding "Pending" comments and JIRA links.

commit c96e2944fa170cb4bc68c18943310a42de9fa352
Author: Dave Johnson <[email protected]>
Date:   2015-11-17T19:51:08Z

    Error checking in WriteStart, fixing timing in UV serialization test and 
ignore test broken by disabled delete.

commit 6f3541be60998f23f95c9794a77d7230396db3e3
Author: Dave Johnson <[email protected]>
Date:   2015-11-17T20:14:51Z

    Use variable instead of hardcoded JUnit version number

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to