Hi, If I recall, when I was starting this exercise, I needed the separation to conceptualise all the parameters apart from the behaviour to allow me to write the initial exploratory tests. It could probably be folded back in with only minor changes.
Regards Paul Campbell From: Guillaume Nodet-2 [via Karaf] <[email protected]> Sent: Tue, 9 May 2017 3:48 PM To: kemitix <[email protected]> Subject: Re: Unit tests and refactoring - karaf-maven-plugin - AssemblyMojo Thx for this work, better code coverage is really nice ! I have one question though, what's the benefit of splitting the AssemblyMojo / AssemblyMojoExec ? It leads to duplication of all the fields for no real gain to me ... Most of the actual execution is already extracted in the Builder class, so I'm not sure we'll benefit from another split. 2017-05-09 16:37 GMT+02:00 kemitix < [hidden email]>: > Hi, > > Branch: https://github.com/kemitix/karaf/tree/assembly-mojo-refactoring > > I've finished refactoring into the assembly package the AssemblyMojo in the > karaf-maven-plugin module. > > There are unit tests providing 100% code coverage of the assembly package. > > The original mockito-based test I had written has been adapted to use > Easymock and a 'BuilderSpy' class. It has been renamed as > AssemblyMojoSystemTest, and it too also provides 100% code coverage for the > assembly package. > > Currently, this work is based on the karaf-4.1.x branch. There have been no > modifications to AssemblyMojo between karaf-4.1.x and master, so rebasing > it > onto master should not cause any problems. > > What should I do next in order to progress this towards being merged with > the main line? > > Regards > Paul Campbell > > > > -- > View this message in context: http://karaf.922171.n3.nabble. > com/Unit-tests-and-refactoring-karaf-maven-plugin-AssemblyMojo- > tp4050098p4050322.html > Sent from the Karaf - Dev mailing list archive at Nabble.com. > -- ------------------------ Guillaume Nodet If you reply to this email, your message will be added to the discussion below: http://karaf.922171.n3.nabble.com/Unit-tests-and-refactoring-karaf-maven-plugin-AssemblyMojo-tp4050098p4050323.html To unsubscribe from Unit tests and refactoring - karaf-maven-plugin - AssemblyMojo, click here. NAML -- View this message in context: http://karaf.922171.n3.nabble.com/Unit-tests-and-refactoring-karaf-maven-plugin-AssemblyMojo-tp4050098p4050324.html Sent from the Karaf - Dev mailing list archive at Nabble.com.
