package uk.ac.ebi.quickgo.geneproduct.controller;

import uk.ac.ebi.quickgo.common.solr.TemporarySolrDataStore;
import uk.ac.ebi.quickgo.geneproduct.GeneProductREST;
import uk.ac.ebi.quickgo.geneproduct.common.GeneProductRepository;
import uk.ac.ebi.quickgo.geneproduct.common.common.GeneProductDocMocker;
import uk.ac.ebi.quickgo.geneproduct.common.document.GeneProductDocument;

import java.util.Arrays;
import java.util.List;
import java.util.stream.Collectors;
import org.junit.Before;
import org.junit.ClassRule;
import org.junit.Test;
import org.junit.runner.RunWith;
import org.springframework.beans.factory.annotation.Autowired;
import org.springframework.boot.test.SpringApplicationConfiguration;
import org.springframework.http.MediaType;
import org.springframework.test.context.junit4.SpringJUnit4ClassRunner;
import org.springframework.test.context.web.WebAppConfiguration;
import org.springframework.test.web.servlet.MockMvc;
import org.springframework.test.web.servlet.ResultActions;
import org.springframework.test.web.servlet.setup.MockMvcBuilders;
import org.springframework.web.context.WebApplicationContext;

import static org.hamcrest.CoreMatchers.is;
import static org.hamcrest.MatcherAssert.assertThat;
import static org.hamcrest.Matchers.greaterThan;
import static org.hamcrest.Matchers.hasItem;
import static org.hamcrest.Matchers.hasSize;
import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.get;
import static org.springframework.test.web.servlet.result.MockMvcResultHandlers.print;
import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.content;
import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.jsonPath;
import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.status;

/**
 * Performs tests on GeneProduct REST controller.
 * Uses an embedded Solr server that is cleaned up automatically after tests complete.
 *
 * @author Tony Wardell
 * Date: 04/04/2016
 */
@RunWith(SpringJUnit4ClassRunner.class)
@SpringApplicationConfiguration(classes = {GeneProductREST.class})
@WebAppConfiguration
public class GeneProductControllerIT {

    // temporary data store for solr's data, which is automatically cleaned on exit
    @ClassRule
    public static final TemporarySolrDataStore solrDataStore = new TemporarySolrDataStore();
    private static final String RESOURCE_URL = "/geneproduct";

    protected static final String COMMA = ",";
    public static final String NON_EXISTANT_ID = "Y0Y000";
    public static final String INVALID_ID = "ZZZZ";

    @Autowired
    private WebApplicationContext webApplicationContext;

    @Autowired
    private GeneProductRepository geneProductRepository;

    private MockMvc mockMvc;

    private String validId;
    private String validIdsCSV;
    private List<String> validIdList;

    @Before
    public void setup() {
        geneProductRepository.deleteAll();

        mockMvc = MockMvcBuilders.webAppContextSetup(webApplicationContext).build();

        List<GeneProductDocument> basicDocs = createBasicDocs();
        assertThat(basicDocs.size(), is(greaterThan(1)));

        validId = basicDocs.get(0).id;
        validIdsCSV = basicDocs.stream().map(doc -> doc.id).collect(Collectors.joining(","));
        validIdList = Arrays.asList(validIdsCSV.split(COMMA));

        geneProductRepository.save(basicDocs);
    }

    @Test
    public void canRetrieveOneGeneProductById() throws Exception {
        ResultActions response = mockMvc.perform(get(buildGeneProductURL(validId)));

        response.andDo(print())
                .andExpect(jsonPath("$.results.*.id", hasSize(1)))
                .andExpect(content().contentType(MediaType.APPLICATION_JSON_VALUE))
                .andExpect(status().isOk());
    }

    @Test
    public void canRetrieveMultiGeneProductById() throws Exception {
        ResultActions result = mockMvc.perform(get(buildGeneProductURL(validIdsCSV)));

        result.andDo(print())
                .andExpect(jsonPath("$.results.*.id", hasSize(3)))
                .andExpect(content().contentType(MediaType.APPLICATION_JSON_VALUE))
                .andExpect(status().isOk());

        int index = 0;
        for (String id : validIdList) {
            expectFields(result, id, "$.results[" + index++ + "].");
        }
    }

    @Test
    public void finds400IfUrlIsEmpty() throws Exception {
        mockMvc.perform(get(RESOURCE_URL + "/"))
                .andDo(print())
                .andExpect(status().isBadRequest());
    }

    @Test
    public void finds400IfTermsIdIsEmpty() throws Exception {
        mockMvc.perform(get(buildGeneProductURL("")))
                .andDo(print())
                .andExpect(status().isBadRequest());
    }


    @Test
    public void finds400IfIdIsInvalid() throws Exception {
        mockMvc.perform(get(buildGeneProductURL(INVALID_ID)))
                .andDo(print())
                .andExpect(jsonPath("$.messages", hasItem(is("Provided ID: '" + INVALID_ID + "' is invalid"))))
                .andExpect(status().isBadRequest());
    }

    @Test
    public void finds200IfNoResultsBecauseIdsDoNotExist() throws Exception {
        mockMvc.perform(get(buildGeneProductURL(NON_EXISTANT_ID)))
                .andDo(print())
                .andExpect(status().isOk());
    }

    private ResultActions expectFields(ResultActions result, String id, String path) throws Exception {
        return result
                .andDo(print())
                .andExpect(jsonPath(path + "id").value(id))
                .andExpect(jsonPath(path + "type").value("PROTEIN"))
                .andExpect(jsonPath(path + "taxonId").value(35758))
                .andExpect(jsonPath(path + "symbol").value("Streptomyces ghanaensis - symbol"))
                .andExpect(jsonPath(path + "parentId").value("UniProtKB:OK0206"))
                .andExpect(jsonPath(path + "databaseSubset[0]").value("RRR"))
                .andExpect(jsonPath(path + "databaseSubset[1]").value("QQQ"))
                .andExpect(jsonPath(path + "isAnnotated").value(true))
                .andExpect(jsonPath(path + "isIsoform").value(true))
                .andExpect(jsonPath(path + "isCompleteProteome").value(true))
                .andExpect(jsonPath(path + "name").value("moeA5"))
                .andExpect(jsonPath(path + "referenceProteome").value("AAAA"))
                .andExpect(jsonPath(path + "synonyms[0]").value("3SSW23"));

    }

    private String buildGeneProductURL(String id) {
        return RESOURCE_URL + "/" + id;
    }

    private List<GeneProductDocument> createBasicDocs() {
        return Arrays.asList(
                GeneProductDocMocker.createDocWithId("A0A000"),
                GeneProductDocMocker.createDocWithId("A0A001"),
                GeneProductDocMocker.createDocWithId("A0A002"));
    }
}

@@ -5925,19 +5925,16 @@
 seSubset
-%5B0%5D
 %22).value
@@ -5946,86 +5946,8 @@
 %22))%0A
-                .andExpect(jsonPath(path + %22databaseSubset%5B1%5D%22).value(%22QQQ%22))%0A
     

Fix broken test case. Database subsets are now single value instead of collections.