Hello, How can we go further with this pull request? Somehow my latest comment was not published in the mailing list:
" I've added the test org.apache.geode.cache30.CacheXmlNotFoundUnitTest#testCacheXmlNotFoundInRealLog. It tests that an error about missing cache-xml file is indeed printed in the text log file, specified via "log-file" parameter. The existing test org.apache.geode.cache30.CacheXml66DUnitTest#testNonExistentFile() is supposed to test the same, but actually it doesn't. It only checks for an CacheXmlException exception to be thrown." BR, Anton -----Original Message----- From: anton-mironenko [mailto:g...@git.apache.org] Sent: Tuesday, August 29, 2017 20:10 To: dev@geode.apache.org Subject: [GitHub] geode issue #677: GEODE-3038: A server process shuts down quietly when path ... Github user anton-mironenko commented on the issue: https://github.com/apache/geode/pull/677 Well, this test category is DistributedTest, not Test. This is why I didn't see it in regular unit tests run. Now the question is - why this test doesn't fail, whereas it should fail without my fix. --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- This message and the information contained herein is proprietary and confidential and subject to the Amdocs policy statement, you may review at https://www.amdocs.com/about/email-disclaimer <https://www.amdocs.com/about/email-disclaimer>
