Removed setters that were only used in tests #87
Conversation
| @@ -18,7 +17,5 @@ | |||
| @Path("/fcr:import") | |||
| public class FedoraRepositoryImport extends FedoraImport { | |||
|
|
|||
| @InjectedSession | |||
There was a problem hiding this comment.
I believe you can't remove this line. Yes, it looks silly, but I think @ajs6f said there's something nasty about the magic (proxying? reflection? whatever) that means we need this in every class we want an injected session in.
There was a problem hiding this comment.
So it wont be enough if the super class has the annotation and the session? since the field is a duplicate...
There was a problem hiding this comment.
Guess not...Ill try and catch Adam to ask him about the particulars...
There was a problem hiding this comment.
If you tried it, it shouldn't pass the tests, right? Because it doesn't inherit the injection behavior from FedoraImport or AbstractResource, right?
There was a problem hiding this comment.
Nah all the tests do pass on this branch...
ghost
assigned 
… to set fields via reflection and changed the tests to use the new method
|
I excised the InjectedSession change and rebased. We can figure out whether the InjectedSession notation is needed everywhere in a separate issue. |
Added Testhelper method to set fields via reflection and changed the tests to use the new method
see https://www.pivotaltracker.com/story/show/52055161