> On Dec. 11, 2014, 7:45 a.m., Nate Cole wrote: > > ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariServer.java, > > line 606 > > <https://reviews.apache.org/r/28940/diff/1/?file=789265#file789265line606> > > > > Should be able to use @StaticallyInject instead. > > Robert Levas wrote: > I didn't think that was working... but I will give it a shot.
For mocks you may still need the init method, I was only referring to initialization from AmbariServer. Make sure you mark ServerActionExecutor's injector field with @Inject. - Nate ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28940/#review64726 ----------------------------------------------------------- On Dec. 11, 2014, 7:34 a.m., Robert Levas wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/28940/ > ----------------------------------------------------------- > > (Updated Dec. 11, 2014, 7:34 a.m.) > > > Review request for Ambari, dilli dorai, John Speidel, Nate Cole, Robert > Nettleton, and Tom Beerbower. > > > Bugs: AMBARI-8647 > https://issues.apache.org/jira/browse/AMBARI-8647 > > > Repository: ambari > > > Description > ------- > > Pass Injector to ServerActionExecutor so object can be injected into > ServerAction implementations. > > This is needed so that server-side actions can get resources like the > AmbariManagementController, Clusters, or etc... > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariServer.java > 51f3a51 > > ambari-server/src/main/java/org/apache/ambari/server/serveraction/ServerActionExecutor.java > 880c596 > > ambari-server/src/test/java/org/apache/ambari/server/actionmanager/TestActionScheduler.java > b389644 > > ambari-server/src/test/java/org/apache/ambari/server/serveraction/MockServerAction.java > ba9a5af > > ambari-server/src/test/java/org/apache/ambari/server/serveraction/ServerActionExecutorTest.java > e89477a > > Diff: https://reviews.apache.org/r/28940/diff/ > > > Testing > ------- > > Performed a serices of manual tests. Updated > `org.apache.ambari.server.serveraction.MockServerAction` to receive an > injected `org.apache.ambari.server.state.Clusters` object for test casesit > appears in - throw an expection if the object is null. > > Tests run: 2413, Failures: 0, Errors: 0, Skipped: 22 > > [INFO] > ------------------------------------------------------------------------ > [INFO] BUILD SUCCESS > [INFO] > ------------------------------------------------------------------------ > [INFO] Total time: 39:56 min > [INFO] Finished at: 2014-12-11T12:21:29+00:00 > [INFO] Final Memory: 43M/479M > [INFO] > ------------------------------------------------------------------------ > > > Thanks, > > Robert Levas > >
