From 62ef1032d5759c87ef4934202cd7e04918104414 Mon Sep 17 00:00:00 2001 From: Lev Proleev Date: Tue, 5 May 2020 14:11:23 +0100 Subject: Fix caching bug in quantization coupling test Execute function in our tests among other things makes sure that the models work with caching enabled. Caching token and directory are constant and therefore the same for all tests. This is usually not a problem since the tests check only one model with a set of different configs but this was not the case for QuantizationCouplingTest suite. This suite ran signed and unsigned models in one test making them share the same caching token and directory which is an incorrect use of caching API. This CL solves this problem by removing the execution of an unsigned model from the test. This is safe since the unsigned model is still verified in GeneratedTests suite. Bug: 155604227 Test: NNTest_static Change-Id: I6aa2c8cd3fa41e37a5de53259d1fcc784f327093 --- nn/runtime/test/TestGenerated.cpp | 1 - 1 file changed, 1 deletion(-) (limited to 'nn/runtime/test') diff --git a/nn/runtime/test/TestGenerated.cpp b/nn/runtime/test/TestGenerated.cpp index cc0b61983..a76aecd92 100644 --- a/nn/runtime/test/TestGenerated.cpp +++ b/nn/runtime/test/TestGenerated.cpp @@ -382,7 +382,6 @@ TEST_P(GeneratedValidationTests, Test) { } TEST_P(QuantizationCouplingTest, Test) { - execute(testModel); execute(convertQuant8AsymmOperandsToSigned(testModel)); } -- cgit v1.2.3