From e389b27d6c6971cee5be5a9b45703d9af6b6cb39 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Bj=C3=B6rn=20Gustavsson?= Date: Mon, 16 Sep 2013 10:12:01 +0200 Subject: No longer compile the X420 specs for plain BER We used to compile the X420 specs for both DER and PER, but 9ec0a0ba changed that to compile for DER and plain BER (probably accidentally). Since the X420 suite contains 99 source files and compilations takes more than 40 seconds on my computer for one backend, we only want to do compilations that are likely to find bugs. Compiling with the BER backend after having compiled for the DER backend is very unlikely to detect any bugs. Compiling for PER (as well as DER) is slightly more likely to expose bugs, but I don't think it is worth the extra running time for the test suite. --- lib/asn1/test/asn1_SUITE.erl | 8 +++----- lib/asn1/test/testX420.erl | 2 +- 2 files changed, 4 insertions(+), 6 deletions(-) (limited to 'lib/asn1/test') diff --git a/lib/asn1/test/asn1_SUITE.erl b/lib/asn1/test/asn1_SUITE.erl index 8a17620531..a558a2941f 100644 --- a/lib/asn1/test/asn1_SUITE.erl +++ b/lib/asn1/test/asn1_SUITE.erl @@ -993,12 +993,10 @@ testX420(Config) -> "sparc-sun-solaris2.10" -> {skip,"Too slow for an old Sparc"}; _ -> - test(Config, fun testX420/3, [ber]) + Rule = ber, + testX420:compile(Rule, [der], Config), + ok = testX420:ticket7759(Rule, Config) end. -testX420(Config, Rule, Opts) -> - testX420:compile(Rule, [der|Opts], Config), - ok = testX420:ticket7759(Rule, Config), - testX420:compile(Rule, Opts, Config). test_x691(Config) -> test(Config, fun test_x691/3, [per, uper]). diff --git a/lib/asn1/test/testX420.erl b/lib/asn1/test/testX420.erl index 00c9221e9d..4ddc55dc16 100644 --- a/lib/asn1/test/testX420.erl +++ b/lib/asn1/test/testX420.erl @@ -26,7 +26,7 @@ -include_lib("test_server/include/test_server.hrl"). -compile(Erule, Options, Config) when Erule =:= ber; Erule =:= per -> +compile(Erule, Options, Config) -> Specs0 = specs(), 99 = length(Specs0), CaseDir = ?config(case_dir, Config), -- cgit v1.2.3