diff options
author | Siri Hansen <[email protected]> | 2017-02-16 15:59:35 +0100 |
---|---|---|
committer | Siri Hansen <[email protected]> | 2017-02-20 12:40:08 +0100 |
commit | 2031e6ceb5588bbecd480f1fb9ca5026f189d7d2 (patch) | |
tree | 69c7698502976581985af26e5dea1a343a768ab7 /lib | |
parent | c802f5d8d0ea08154005abfdcb0d958f126f26d1 (diff) | |
download | otp-2031e6ceb5588bbecd480f1fb9ca5026f189d7d2.tar.gz otp-2031e6ceb5588bbecd480f1fb9ca5026f189d7d2.tar.bz2 otp-2031e6ceb5588bbecd480f1fb9ca5026f189d7d2.zip |
Add dummy end_per_suite/1
common_test requires that if init_per_suite/1 exists, then
end_per_suite/1 must also exist. If end_per_suite/1 does not exist,
then it will be marked in the log as failed with reason 'undef'. Some
test suites are corrected to avoid this.
Diffstat (limited to 'lib')
-rw-r--r-- | lib/dialyzer/test/abstract_SUITE.erl | 6 | ||||
-rw-r--r-- | lib/ssl/test/ssl_npn_hello_SUITE.erl | 4 |
2 files changed, 9 insertions, 1 deletions
diff --git a/lib/dialyzer/test/abstract_SUITE.erl b/lib/dialyzer/test/abstract_SUITE.erl index 269db3e836..0e84dfab24 100644 --- a/lib/dialyzer/test/abstract_SUITE.erl +++ b/lib/dialyzer/test/abstract_SUITE.erl @@ -7,7 +7,7 @@ -include_lib("common_test/include/ct.hrl"). -include("dialyzer_test_constants.hrl"). --export([suite/0, all/0, init_per_suite/0, init_per_suite/1]). +-export([suite/0, all/0, init_per_suite/0, init_per_suite/1, end_per_suite/1]). -export([generated_case/1]). suite() -> @@ -24,6 +24,10 @@ init_per_suite(Config) -> ok -> [{dialyzer_options, []}|Config] end. +end_per_suite(_Config) -> + %% This function is required since init_per_suite/1 exists. + ok. + generated_case(Config) when is_list(Config) -> %% Equivalent to: %% diff --git a/lib/ssl/test/ssl_npn_hello_SUITE.erl b/lib/ssl/test/ssl_npn_hello_SUITE.erl index 69aeea10c5..0b1de1dc1c 100644 --- a/lib/ssl/test/ssl_npn_hello_SUITE.erl +++ b/lib/ssl/test/ssl_npn_hello_SUITE.erl @@ -50,6 +50,10 @@ init_per_suite(Config) -> {skip, "Crypto did not start"} end. +end_per_suite(_Config) -> + %% This function is required since init_per_suite/1 exists. + ok. + init_per_testcase(_TestCase, Config) -> ssl_test_lib:ct_log_supported_protocol_versions(Config), ct:timetrap({seconds, 5}), |