/*
* Copyright 2015 herd contributors
*
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
* You may obtain a copy of the License at
*
*     http://www.apache.org/licenses/LICENSE-2.0
*
* Unless required by applicable law or agreed to in writing, software
* distributed under the License is distributed on an "AS IS" BASIS,
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
* See the License for the specific language governing permissions and
* limitations under the License.
*/
package org.finra.herd.service.impl;

import org.springframework.context.annotation.Primary;
import org.springframework.stereotype.Service;
import org.springframework.transaction.annotation.Transactional;

import org.finra.herd.dao.config.DaoSpringModuleConfig;
import org.finra.herd.model.api.xml.EmrCluster;
import org.finra.herd.model.api.xml.EmrClusterCreateRequest;
import org.finra.herd.model.api.xml.EmrMasterSecurityGroup;
import org.finra.herd.model.api.xml.EmrMasterSecurityGroupAddRequest;
import org.finra.herd.model.dto.EmrClusterAlternateKeyDto;

/**
 * This is a EMR service implementation for test, to overwrite any behavior needed for test.
 */
@Service
@Transactional(value = DaoSpringModuleConfig.HERD_TRANSACTION_MANAGER_BEAN_NAME)
@Primary
public class TestEmrServiceImpl extends EmrServiceImpl
{
    /**
     * {@inheritDoc}
     * <p/>
     * Overwrite the base class method to change transactional attributes.
     */
    @Override
    public EmrMasterSecurityGroup addSecurityGroupsToClusterMaster(EmrMasterSecurityGroupAddRequest request) throws Exception
    {
        return addSecurityGroupsToClusterMasterImpl(request);
    }

    /**
     * {@inheritDoc}
     * <p/>
     * Overwrite the base class method to change transactional attributes.
     */
    @Override
    public Object addStepToCluster(Object request) throws Exception
    {
        return addStepToClusterImpl(request);
    }

    /**
     * {@inheritDoc}
     * <p/>
     * Overwrite the base class method to change transactional attributes.
     */
    @Override
    public EmrCluster createCluster(EmrClusterCreateRequest request) throws Exception
    {
        return createClusterImpl(request);
    }

    /**
     * {@inheritDoc}
     * <p/>
     * Overwrite the base class method to change transactional attributes.
     */
    @Override
    public EmrCluster getCluster(EmrClusterAlternateKeyDto alternateKey, String emrClusterId, String emrStepId, boolean verbose, String accountId,
        Boolean retrieveInstanceFleets) throws Exception
    {
        return getClusterImpl(alternateKey, emrClusterId, emrStepId, verbose, null, retrieveInstanceFleets);
    }

    /**
     * {@inheritDoc}
     * <p/>
     * Overwrite the base class method to change transactional attributes.
     */
    @Override
    public EmrCluster terminateCluster(EmrClusterAlternateKeyDto emrClusterAlternateKeyDto, boolean overrideTerminationProtection, String emrClusterId,
        String accountId) throws Exception
    {
        return terminateClusterImpl(emrClusterAlternateKeyDto, overrideTerminationProtection, emrClusterId, null);
    }
}

@@ -2708,20 +2708,25 @@
 erbose, 
-null
+accountId
 , retrie
@@ -3202,12 +3202,17 @@
 Id, 
-null
+accountId
 );%0A 

Fix emr unit test account id bug