rafaelweingartner commented on a change in pull request #2414:
[CLOUDSTACK-10241] Duplicated file SRs being created in XenServer pools
URL: https://github.com/apache/cloudstack/pull/2414#discussion_r164819588
##########
File path:
plugins/hypervisors/xenserver/src/test/java/com/cloud/hypervisor/xenserver/resource/XenServer625ResourceTest.java
##########
@@ -30,7 +31,7 @@
@Before
public void beforeTest() {
- super.citrixResourceBase = new Xenserver625Resource();
+ super.citrixResourceBase = Mockito.spy(new Xenserver625Resource());
Review comment:
Oh, this one... It is because there is a hierarchy of test classes.. :(..
the property `citrixResourceBase` needs to be either a mock or spy object
because of the testes executed in the super class. Therefore, needed to create
a spy object whenever attributing an object to this property.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
