aboutsummaryrefslogtreecommitdiffstats
path: root/lib
diff options
context:
space:
mode:
authorSiri Hansen <[email protected]>2017-02-16 15:59:35 +0100
committerSiri Hansen <[email protected]>2017-02-20 12:40:08 +0100
commit2031e6ceb5588bbecd480f1fb9ca5026f189d7d2 (patch)
tree69c7698502976581985af26e5dea1a343a768ab7 /lib
parentc802f5d8d0ea08154005abfdcb0d958f126f26d1 (diff)
downloadotp-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.erl6
-rw-r--r--lib/ssl/test/ssl_npn_hello_SUITE.erl4
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}),