Commit 5424808c authored by unknown's avatar unknown

Post review fixes


mysql-test/include/show_msg80.inc:
  Post review fixes: improved description
mysql-test/r/mysqltest.result:
  Updated result
mysql-test/t/mysqltest.test:
  one sub testcase added
parent 445c0fd2
...@@ -16,14 +16,19 @@ ...@@ -16,14 +16,19 @@
# Attention: # Attention:
# - Please do not write any spaces between $message and the "=", because the # - Please do not write any spaces between $message and the "=", because the
# assignment will not work. # assignment will not work.
# - Be careful with single quotes. They must be escaped like "''" or "\'". # - Be careful with single quotes within the value. They must be escaped like
# "''" or "\'".
# - Do not keep the value between single quotes.
# #
# #
# Content of "$message" and protocol output depending on the assignement: # Content of "$message" and protocol output depending on the assignment:
# ----------------------------------------------------------------------- # ----------------------------------------------------------------------
# <x> first char after "$message=", #
# I is assumed, that the value is not kept between double quotes.
#
# <x> first character after "$message=",
# where the content is not (space or tab) # where the content is not (space or tab)
# <y*> first chat after beginning of the line, # <y*> first character after beginning of the line,
# where the content is not (space or tab) # where the content is not (space or tab)
# <z> last char before ";" # <z> last char before ";"
# | beginning or end of line # | beginning or end of line
...@@ -31,7 +36,7 @@ ...@@ -31,7 +36,7 @@
# script: let $message= <x><whatever0>| # script: let $message= <x><whatever0>|
# | <y1><whatever1>| # | <y1><whatever1>|
# |................| # |................|
# | <yn><whatevern><z>; # | <yn><whatevern><z>;
# content: "<x><whatever0><new line><y1><whatever1><new line> # content: "<x><whatever0><new line><y1><whatever1><new line>
# ....<new line><yn><whatevern><z>" # ....<new line><yn><whatevern><z>"
# protocol output: |<x><whatever0>| # protocol output: |<x><whatever0>|
...@@ -40,24 +45,17 @@ ...@@ -40,24 +45,17 @@
# |<yn><whatevern><z>| # |<yn><whatevern><z>|
# |--- 80 dashes ---| # |--- 80 dashes ---|
# #
# Attention:
# <x> and <y*> set to characters like "-$#" which are also used
# to start comments, options and the names of mysqltest variables
# lead to syntax errors or mangled messages.
#
# #
# Examples of messages: # Examples of messages:
# --------------------- # ---------------------
# #
# Several lines with indentation variant 1: # Variant1 (ease of use):
# script: |let $message= . Testcase 3.1 : Ensure that Pi is not an| # Several lines with indentation kept between double quotes
# | . integer number|
# | . Third line;
# protocol: |. Testcase 3.1 : Ensure that Pi is not an|
# |. integer number|
# |. Third line|
# |------ 80 dashes ----|
# Please mention that
# - the auxiliary "." preserves the indentation.
# - it is easy to write the script lines to get a fine indentation
# - the "." is printed
#
# Several lines with indentation variant 2:
# script: |let $message= # script: |let $message=
# |" Testcase 3.1 : Ensure that Pi is not an| # |" Testcase 3.1 : Ensure that Pi is not an|
# | integer number.| # | integer number.|
...@@ -69,10 +67,32 @@ ...@@ -69,10 +67,32 @@
# #
# Please mention that # Please mention that
# - the '"' preserves the indentation. # - the '"' preserves the indentation.
# - it is not so easy to write the script lines to get a fine indentation # - it is easy to write the script lines to get a fine indentation,
# if the value starts at the beginning of a new line
# - the '"' is printed # - the '"' is printed
# - there are the least or no problems with characters like "-#$"
# #
# Several lines with lost indentation (negative example) #
# Variant 2 (grep the messages from the protocol is easy):
# Several lines with indentation + auxiliary character (".")
# at the (non tab or space) beginning of every message line
# script: |let $message= . Testcase 3.1 : Ensure that Pi is not an|
# | . integer number.|
# | . Third line;
# protocol: |. Testcase 3.1 : Ensure that Pi is not an|
# |. integer number.|
# |. Third line|
# |------ 80 dashes ----|
# Please mention that
# - the auxiliary character preserves the indentation.
# - it is easy to write the script lines to get a fine indentation
# - the auxiliary character is printed
# - it is recommended to use "." as auxiliary character
# - auxiliary characters like "-'$#" cause problems
#
#
#
# Bad variant1: Several lines with lost indentation
# script: |let $message= Here is message line 1 # script: |let $message= Here is message line 1
# | message line 2; # | message line 2;
# protocol: |Here is message line 1| # protocol: |Here is message line 1|
...@@ -80,6 +100,15 @@ ...@@ -80,6 +100,15 @@
# |------ 80 dashes ----| # |------ 80 dashes ----|
# Please mention, that the leading spaces of the message lines disappeared. # Please mention, that the leading spaces of the message lines disappeared.
# #
# Bad variant2: Several lines leading to a syntax error, because of "-"
# script: |let $message= - This is a message
# | - with a second and
# | - third line;
# protocol: | - third line;;
# |ERROR 42000: You have an error ... near '- third line'
# + several following errors
#
#
--disable_query_log --disable_query_log
eval SET @utf8_message = CONVERT('$message' using utf8); eval SET @utf8_message = CONVERT('$message' using utf8);
......
...@@ -159,3 +159,8 @@ root@localhost ...@@ -159,3 +159,8 @@ root@localhost
- is longer then 80 characters and - is longer then 80 characters and
- consists of several lines" - consists of several lines"
-------------------------------------------------------------------------------- --------------------------------------------------------------------------------
. Here comes a very very long message that
. - is longer then 80 characters and
. - consists of several lines
--------------------------------------------------------------------------------
...@@ -311,9 +311,17 @@ let $message= `SELECT USER()`; ...@@ -311,9 +311,17 @@ let $message= `SELECT USER()`;
--source include/show_msg.inc --source include/show_msg.inc
# The message contains more then 80 characters on multiple lines # The message contains more then 80 characters on multiple lines
# and is kept between double quotes.
let $message= let $message=
"Here comes a very very long message that "Here comes a very very long message that
- is longer then 80 characters and - is longer then 80 characters and
- consists of several lines"; - consists of several lines";
--source include/show_msg80.inc --source include/show_msg80.inc
# The message contains more then 80 characters on multiple lines
# and uses the auxiliary character "." at the beginning of the message lines.
let $message= . Here comes a very very long message that
. - is longer then 80 characters and
. - consists of several lines;
--source include/show_msg80.inc
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment