Http oracle timeout

[omursahin]

Python test case

    # Calls:
    # (null) GET:/api/timeout/slow/{id}
    # Found 1 potential fault of type-code 921
    @timeout_decorator.timeout(60)
    def test_0_get_on_slow_timeout(self):
        
        # Fault921. Timeout.
        headers = {}
        headers['Accept'] = "*/*"
        with self.assertRaises(Exception):
            requests \
                    .get(self.baseUrlOfSut + "/api/timeout/slow/683",
                        headers=headers, timeout=EM_HTTP_TIMEOUT, verify=False)

JS test case

/**
* Calls:
* (null) GET:/api/timeout/slow/{id}
* Found 1 potential fault of type-code 921
*/
test("test_0_GetOnSlowTimeout", async () => {
    
    // Fault921. Timeout.
    await expect(superagent
            .get(baseUrlOfSut + "/api/timeout/slow/683")
            .timeout({response: EM_HTTP_TIMEOUT_MS, deadline: EM_HTTP_TIMEOUT_MS}).set('Accept', "*/*")
            .ok(res => res.status)).rejects.toThrow();
});

Kotlin:

        @BeforeAll
        @JvmStatic
        fun initClass() {
            RestAssured.enableLoggingOfRequestAndResponseIfValidationFails()
            RestAssured.useRelaxedHTTPSValidation()
            RestAssured.urlEncodingEnabled = false
            RestAssured.config = RestAssured.config()
                .jsonConfig(JsonConfig.jsonConfig().numberReturnType(JsonPathConfig.NumberReturnType.DOUBLE))
                .redirect(redirectConfig().followRedirects(false))
                .httpClient(HttpClientConfig.httpClientConfig()
                    .setParam("http.socket.timeout", 2000)
                    .setParam("http.connection.timeout", 2000))
                .jsonConfig(JsonConfig.jsonConfig().numberReturnType(JsonPathConfig.NumberReturnType.DOUBLE))
                .redirect(redirectConfig().followRedirects(false))
                .encoderConfig(EncoderConfig.encoderConfig().encodeContentTypeAs("application/octet-stream", ContentType.TEXT))
        }

    /**
    * Calls:
    * (null) GET:/api/timeout/slow/{id}
    * Found 1 potential fault of type-code 921
    */
    @Test @Timeout(60)
    fun test_0_getOnSlowTimeout()  {
        
        try{
            // Fault921. Timeout.
            given().accept("*/*")
                    .get("${baseUrlOfSut}/api/timeout/slow/683")
            fail("Expected a timeout");
        } catch(e: Exception){
        }
    }

Java

    @BeforeClass
    public static void initClass() {
        RestAssured.enableLoggingOfRequestAndResponseIfValidationFails();
        RestAssured.useRelaxedHTTPSValidation();
        RestAssured.urlEncodingEnabled = false;
        RestAssured.config = RestAssured.config()
            .jsonConfig(JsonConfig.jsonConfig().numberReturnType(JsonPathConfig.NumberReturnType.DOUBLE))
            .redirect(redirectConfig().followRedirects(false))
            .httpClient(HttpClientConfig.httpClientConfig()
                .setParam("http.socket.timeout", 2000)
                .setParam("http.connection.timeout", 2000))
            .jsonConfig(JsonConfig.jsonConfig().numberReturnType(JsonPathConfig.NumberReturnType.DOUBLE))
            .redirect(redirectConfig().followRedirects(false))
            .encoderConfig(EncoderConfig.encoderConfig().encodeContentTypeAs("application/octet-stream", ContentType.TEXT));
    }
        /**
    * Calls:
    * (null) GET:/api/timeout/slow/{id}
    * Found 1 potential fault of type-code 921
    */
    @Test(timeout = 60000)
    public void test_0_getOnSlowTimeout() throws Exception {
        
        try{
            // Fault921. Timeout.
            given().accept("*/*")
                    .get(baseUrlOfSut + "/api/timeout/slow/683");
            fail("Expected a timeout");
        } catch(Exception e){
        }
    }

[arcuri82]

this EM_HTTP_TIMEOUT_MS is referring to a variable declared elsewhere? in such case, the name of the variable should be put in a shared constant

[arcuri82]

use JavaDoc style /** */ for comments on fields and methos

[arcuri82]

this name EM_HTTP_TIMEOUT_MS should be in a shared constant, as re-used in few places

[arcuri82]

shared constant

[arcuri82]

add comment specifying why we are skipping for white-box testing

[arcuri82]

constant

[arcuri82]

otherwise, otherwise if we change name in future, we will break lot of places

[arcuri82]

constant. also, we need a way to make it easier and do not confuse when we us MS or S version

[arcuri82]

shouldn't this test fail because by default we have httpOracles being false?