[jira] [Created] (SAMZA-1410) TestLocalApplicationRunner testPlanIdWithShuffledStreamSpecs and testGeneratePlanIdWithDifferentStreamSpecs may fail in different Jnit

["Fred Ji (JIRA)" <ji...@apache.org>]

Fred Ji created SAMZA-1410:
------------------------------

             Summary: TestLocalApplicationRunner testPlanIdWithShuffledStreamSpecs and testGeneratePlanIdWithDifferentStreamSpecs may fail in different Jnit
                 Key: SAMZA-1410
                 URL: https://issues.apache.org/jira/browse/SAMZA-1410
             Project: Samza
          Issue Type: Test
    Affects Versions: 0.13.1
            Reporter: Fred Ji
            Assignee: Fred Ji
             Fix For: 0.14.0


The following two tests failed in Junit 4.8.3 which were the explicit Junit version in the dependency. 
testPlanIdWithShuffledStreamSpecs
testGeneratePlanIdWithDifferentStreamSpecs

Here is the stack trace: 
[ant:scalac] Element '/Users/fji/workspace/samza-li_trunk/samza/build/samza-core_2.10/resources/main' does not exist.
[ant:scalac] Element '/Users/fji/workspace/samza-li_trunk/samza/build/samza-api/resources/test' does not exist.
/Users/fji/workspace/samza-li_trunk/samza/samza-core/src/test/java/org/apache/samza/runtime/TestLocalApplicationRunner.java:374: error: cannot find symbol
    assertNotEquals("Expected both of the latch ids to be different", planIdBeforeShuffle,
    ^
  symbol:   method assertNotEquals(String,String,String)
  location: class TestLocalApplicationRunner
/Users/fji/workspace/samza-li_trunk/samza/samza-core/src/test/java/org/apache/samza/runtime/TestLocalApplicationRunner.java:412: error: cannot find symbol
    assertNotEquals("Expected both of the latch ids to be different", planIdBeforeShuffle,
    ^
  symbol:   method assertNotEquals(String,String,String)
  location: class TestLocalApplicationRunner
Note: Some input files use or override a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note: Some input files use unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.
2 errors
:samza-core_2.10:compileTestScala FAILED


If Junit 4.12 is picked, the tests may be passed, but if 4.8.3 is picked, the tests will fail.

It is better to use consistent Junit version for all tests in the project. Currently, if Junit 4.8.3 is used, these two tests will fail, if 4.12 is used, all the unit tests which use ExpectedException will fail due to the pulling of hamcrest lib from mockito-all instead of hamcrest-core itself. 

For the long term, we need to remove all dependency on mockito-all since it is bad practice to depend on that lib, since it includes the other dependency code in mockito-all directly which may cause dependency resolving problem. 

For the short term, we refactor these two tests to make sure they can pass in different versions of Junit. 

This JIRA is for short term purpose. 





--
This message was sent by Atlassian JIRA
(v6.4.14#64029)