Made three public attributes of cmd2.Cmd no longer settable at runtime by end users

The 3 attributes are:
- continuation_prompt
- locals_in_py
- prompt

16 files changed, 49 insertions, 72 deletions

diff --git a/CHANGELOG.md b/CHANGELOG.md
index bc5fa5d4..f6e71d6a 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -1,3 +1,10 @@
+## 0.9.26 (January 26, 2020)
+* Breaking changes
+    * The following public attributes of `cmd2.Cmd` are no longer settable at runtime by end users:
+        * `continuation_prompt`
+        * `locals_in_py`
+        * `prompt`
+    
 ## 0.9.25 (January 26, 2020)
 * Enhancements
     * Reduced what gets put in package downloadable from PyPI (removed irrelevant CI config files and such)
diff --git a/cmd2/cmd2.py b/cmd2/cmd2.py
index 648aadb1..609f01ad 100644
--- a/cmd2/cmd2.py
+++ b/cmd2/cmd2.py
@@ -188,21 +188,18 @@ class Cmd(cmd.Cmd):
         self.allow_redirection = allow_redirection  # Security setting to prevent redirection of stdout
 
         # Attributes which ARE dynamically settable via the set command at runtime
-        self.continuation_prompt = '> '
         self.debug = False
         self.echo = False
         self.editor = Cmd.DEFAULT_EDITOR
         self.feedback_to_output = False  # Do not include nonessentials in >, | output by default (things like timing)
-        self.locals_in_py = False
+        self.quiet = False  # Do not suppress nonessential output
+        self.timing = False  # Prints elapsed time for each command
 
         # The maximum number of CompletionItems to display during tab completion. If the number of completion
         # suggestions exceeds this number, they will be displayed in the typical columnized format and will
         # not include the description value of the CompletionItems.
         self.max_completion_items = 50
 
-        self.quiet = False  # Do not suppress nonessential output
-        self.timing = False  # Prints elapsed time for each command
-
         # To make an attribute settable with the "do_set" command, add it to this ...
         self.settable = \
             {
@@ -211,18 +208,21 @@ class Cmd(cmd.Cmd):
                                 '(valid values: {}, {}, {})'.format(ansi.STYLE_TERMINAL,
                                                                     ansi.STYLE_ALWAYS,
                                                                     ansi.STYLE_NEVER)),
-                'continuation_prompt': 'On 2nd+ line of input',
                 'debug': 'Show full error stack on error',
                 'echo': 'Echo command issued into output',
                 'editor': 'Program used by ``edit``',
                 'feedback_to_output': 'Include nonessentials in `|`, `>` results',
-                'locals_in_py': 'Allow access to your application in py via self',
                 'max_completion_items': 'Maximum number of CompletionItems to display during tab completion',
-                'prompt': 'The prompt issued to solicit input',
                 'quiet': "Don't print nonessential feedback",
                 'timing': 'Report execution times'
             }
 
+        # Use as prompt for multiline commands on the 2nd+ line of input
+        self.continuation_prompt = '> '
+
+        # Allow access to your application in embedded Python shells and scripts py via self
+        self.locals_in_py = False
+
         # Commands to exclude from the help menu and tab completion
         self.hidden_commands = ['eof', '_relative_load', '_relative_run_script']
 
diff --git a/docs/features/builtin_commands.rst b/docs/features/builtin_commands.rst
index 540cfbb1..025149b3 100644
--- a/docs/features/builtin_commands.rst
+++ b/docs/features/builtin_commands.rst
@@ -93,14 +93,11 @@ within a running application:
 
     (Cmd) set --long
     allow_style: Terminal          # Allow ANSI text style sequences in output (valid values: Terminal, Always, Never)
-    continuation_prompt: >         # On 2nd+ line of input
     debug: False                   # Show full error stack on error
     echo: False                    # Echo command issued into output
     editor: vim                    # Program used by ``edit``
     feedback_to_output: False      # include nonessentials in `|`, `>` results
-    locals_in_py: False            # Allow access to your application in py via self
     max_completion_items: 50       # Maximum number of CompletionItems to display during tab completion
-    prompt: (Cmd)                  # The prompt issued to solicit input
     quiet: False                   # Don't print nonessential feedback
     timing: False                  # Report execution times
 
diff --git a/docs/features/embedded_python_shells.rst b/docs/features/embedded_python_shells.rst
index 9d721c2c..3386e8cd 100644
--- a/docs/features/embedded_python_shells.rst
+++ b/docs/features/embedded_python_shells.rst
@@ -10,9 +10,7 @@ your cmd2 application while maintaining isolation.
 You may optionally enable full access to to your application by setting
 ``locals_in_py`` to ``True``.  Enabling this flag adds ``self`` to the python
 session, which is a reference to your Cmd2 application. This can be useful for
-debugging your application.  To prevent users from enabling this ability
-manually you'll need to remove ``locals_in_py`` from the ``settable``
-dictionary.
+debugging your application.
 
 The ``app`` object (or your custom name) provides access to application
 commands through raw commands.  For example, any application command call be
diff --git a/docs/features/initialization.rst b/docs/features/initialization.rst
index 39e36428..864741cd 100644
--- a/docs/features/initialization.rst
+++ b/docs/features/initialization.rst
@@ -110,7 +110,7 @@ override:
   DisabledCommand objects.
 - **echo**: if ``True``, each command the user issues will be repeated to the
   screen before it is executed. This is particularly useful when running
-  scripts. This behavior does not occur when a running command at the prompt.
+  scripts. This behavior does not occur when running a command at the prompt.
   (Default: ``False``)
 - **editor**: text editor program to use with *edit* command (e.g. ``vim``)
 - **exclude_from_history**: commands to exclude from the *history* command
diff --git a/docs/features/multiline_commands.rst b/docs/features/multiline_commands.rst
index d6502058..769e0a64 100644
--- a/docs/features/multiline_commands.rst
+++ b/docs/features/multiline_commands.rst
@@ -10,3 +10,13 @@ blank line is *always* considered a command terminator (cannot be overridden).
 
 In multiline commands, output redirection characters like ``>`` and ``|`` are
 part of the command arguments unless they appear after the terminator.
+
+Continuation prompt
+-------------------
+
+When a user types a :ref:`Multiline Command
+<features/multiline_commands:Multiline Commands>` it may span more than one
+line of input. The prompt for the first line of input is specified by the
+``cmd2.Cmd.prompt`` instance attribute - see
+:ref:`features/prompt:Customizing the Prompt`. The prompt for subsequent lines
+of input is defined by the ``cmd2.Cmd.continuation_prompt`` attribute.
diff --git a/docs/features/prompt.rst b/docs/features/prompt.rst
index 40c50d2b..244ffb31 100644
--- a/docs/features/prompt.rst
+++ b/docs/features/prompt.rst
@@ -3,6 +3,13 @@ Prompt
 
 ``cmd2`` can issue a prompt before soliciting user input.
 
+Customizing the Prompt
+----------------------
+
+This prompt can be configured by setting the `cmd2.Cmd.prompt` instance
+attribute. This contains the string which should be printed as a prompt
+for user input.
+
 Asynchronous Feedback
 ---------------------
 
diff --git a/docs/features/settings.rst b/docs/features/settings.rst
index b8f75934..55b6a10d 100644
--- a/docs/features/settings.rst
+++ b/docs/features/settings.rst
@@ -47,16 +47,6 @@ This setting can be one of three values:
 - ``Always`` - ANSI escape sequences are always passed through to the output
 
 
-continuation_prompt
-~~~~~~~~~~~~~~~~~~~
-
-When a user types a :ref:`Multiline Command
-<features/multiline_commands:Multiline Commands>` it may span more than one
-line of input. The prompt for the first line of input is specified by the
-:ref:`features/settings:prompt` setting. The prompt for subsequent lines of
-input is defined by this setting.
-
-
 debug
 ~~~~~
 
@@ -71,7 +61,7 @@ echo
 
 If ``True``, each command the user issues will be repeated to the screen
 before it is executed. This is particularly useful when running scripts.
-This behavior does not occur when a running command at the prompt.
+This behavior does not occur when running a command at the prompt.
 
 
 editor
@@ -95,13 +85,6 @@ feedback output will be mixed in with and indistinguishable from output
 generated with :meth:`~cmd2.cmd2.Cmd.poutput`.
 
 
-locals_in_py
-~~~~~~~~~~~~
-
-Allow access to your application in one of the
-:ref:`features/embedded_python_shells:Embedded Python Shells` via ``self``.
-
-
 max_completion_items
 ~~~~~~~~~~~~~~~~~~~~
 
@@ -115,13 +98,6 @@ they will be displayed in the typical columnized format and will not include
 the description text of the CompletionItem.
 
 
-prompt
-~~~~~~
-
-This setting contains the string which should be printed as a prompt for user
-input.
-
-
 quiet
 ~~~~~
 
@@ -180,14 +156,13 @@ You may want to prevent a user from modifying a builtin setting. A setting
 must appear in the :attr:`cmd2.cmd2.Cmd.settable` dictionary in order for it
 to be available to the :ref:`features/builtin_commands:set` command.
 
-Let's say your program does not have any
-:ref:`features/multiline_commands:Multiline Commands`. You might want to hide
-the :ref:`features/settings:continuation_prompt` setting from your users since
-it is only applicable to multiline commands. To do so, remove it from the
+Let's say that you never want end users of your program to be able to enable
+full debug tracebacks to print out if an error occurs. You might want to hide
+the :ref:`features/settings:debug` setting. To do so, remove it from the
 :attr:`cmd2.cmd2.Cmd.settable` dictionary after you initialize your object::
 
   class MyApp(cmd2.Cmd):
 
     def __init__(self):
         super().__init__()
-        self.settable.pop('continuation_prompt')
+        self.settable.pop('debug')
diff --git a/docs/features/transcripts.rst b/docs/features/transcripts.rst
index 9f524e0a..1af2a74f 100644
--- a/docs/features/transcripts.rst
+++ b/docs/features/transcripts.rst
@@ -146,8 +146,8 @@ the path instead of specifying it verbatim, or we can escape the slashes::
    invisible, you can add a regular expression to match them, so that you can
    see where they are when you look at the transcript::
 
-      (Cmd) set prompt
-      prompt: (Cmd)/ /
+      (Cmd) set editor
+      editor: vim/ /
 
    Some terminal emulators strip trailing space when you copy text from them.
    This could make the actual data generated by your app different than the
diff --git a/examples/scripts/save_help_text.py b/examples/scripts/save_help_text.py
index 12d66d6d..2d1c2c41 100644
--- a/examples/scripts/save_help_text.py
+++ b/examples/scripts/save_help_text.py
@@ -60,7 +60,7 @@ def main() -> None:
 
     # Make sure we have access to self
     if 'self' not in globals():
-        print("Run 'set locals_in_py true' and then rerun this script")
+        print("Re-run this script from a cmd2 application where locals_in_py is True")
         return
 
     # Make sure the user passed in an output file
diff --git a/examples/transcripts/exampleSession.txt b/examples/transcripts/exampleSession.txt
index 5a75235b..54419f91 100644
--- a/examples/transcripts/exampleSession.txt
+++ b/examples/transcripts/exampleSession.txt
@@ -4,13 +4,11 @@
 # regexes on prompts just make the trailing space obvious
 (Cmd) set
 allow_style: /(Terminal|Always|Never)/
-continuation_prompt: >/ /
 debug: False
 echo: False
 editor: /.*?/
 feedback_to_output: False
-locals_in_py: False
+max_completion_items: 50
 maxrepeats: 3
-prompt: (Cmd)/ /
 quiet: False
 timing: False
diff --git a/examples/transcripts/transcript_regex.txt b/examples/transcripts/transcript_regex.txt
index 276f7d22..35fc5817 100644
--- a/examples/transcripts/transcript_regex.txt
+++ b/examples/transcripts/transcript_regex.txt
@@ -4,13 +4,11 @@
 # regexes on prompts just make the trailing space obvious
 (Cmd) set
 allow_style: /(Terminal|Always|Never)/
-continuation_prompt: >/ /
 debug: False
 echo: False
 editor: /.*?/
 feedback_to_output: False
-locals_in_py: False
+max_completion_items: 50
 maxrepeats: 3
-prompt: (Cmd)/ /
 quiet: False
 timing: False
diff --git a/tests/conftest.py b/tests/conftest.py
index fe74ffe2..b8abc4a5 100644
--- a/tests/conftest.py
+++ b/tests/conftest.py
@@ -89,28 +89,22 @@ SHORTCUTS_TXT = """Shortcuts for other commands:
 
 # Output from the show command with default settings
 SHOW_TXT = """allow_style: Terminal
-continuation_prompt: >
 debug: False
 echo: False
 editor: vim
 feedback_to_output: False
-locals_in_py: False
 max_completion_items: 50
-prompt: (Cmd)
 quiet: False
 timing: False
 """
 
 SHOW_LONG = """
 allow_style: Terminal     # Allow ANSI text style sequences in output (valid values: Terminal, Always, Never)
-continuation_prompt: >    # On 2nd+ line of input
 debug: False              # Show full error stack on error
 echo: False               # Echo command issued into output
 editor: vim               # Program used by ``edit``
 feedback_to_output: False # Include nonessentials in `|`, `>` results
-locals_in_py: False       # Allow access to your application in py via self
 max_completion_items: 50  # Maximum number of CompletionItems to display during tab completion
-prompt: (Cmd)             # The prompt issued to solicit input
 quiet: False              # Don't print nonessential feedback
 timing: False             # Report execution times
 """
diff --git a/tests/test_cmd2.py b/tests/test_cmd2.py
index b5473609..21cd941e 100755
--- a/tests/test_cmd2.py
+++ b/tests/test_cmd2.py
@@ -255,16 +255,12 @@ def test_base_py(base_app):
     assert out[0].rstrip() == 'spaces in this command'
 
     # Set locals_in_py to True and make sure we see self
-    out, err = run_cmd(base_app, 'set locals_in_py True')
-    assert 'now: True' in out
-
+    base_app.locals_in_py = True
     out, err = run_cmd(base_app, 'py print(self)')
     assert 'cmd2.cmd2.Cmd object' in out[0]
 
     # Set locals_in_py to False and make sure we can't see self
-    out, err = run_cmd(base_app, 'set locals_in_py False')
-    assert 'now: False' in out
-
+    base_app.locals_in_py = False
     out, err = run_cmd(base_app, 'py print(self)')
     assert "NameError: name 'self' is not defined" in err
 
diff --git a/tests/transcripts/from_cmdloop.txt b/tests/transcripts/from_cmdloop.txt
index aede6659..95537665 100644
--- a/tests/transcripts/from_cmdloop.txt
+++ b/tests/transcripts/from_cmdloop.txt
@@ -39,6 +39,6 @@ OODNIGHT, GRACIEGAY
 OODNIGHT, GRACIEGAY
 OODNIGHT, GRACIEGAY
 OODNIGHT, GRACIEGAY
-(Cmd) set prompt "---> "
-prompt - was: (Cmd)/ */
-now: --->/ */
+(Cmd) set debug True
+debug - was: False/ */
+now: True/ */
diff --git a/tests/transcripts/regex_set.txt b/tests/transcripts/regex_set.txt
index 17f43ede..5bf9add3 100644
--- a/tests/transcripts/regex_set.txt
+++ b/tests/transcripts/regex_set.txt
@@ -5,14 +5,11 @@
 
 (Cmd) set
 allow_style: /(Terminal|Always|Never)/
-continuation_prompt: >/ /
 debug: False
 echo: False
 editor: /.*/
 feedback_to_output: False
-locals_in_py: False
 max_completion_items: 50
 maxrepeats: 3
-prompt: (Cmd)/ /
 quiet: False
 timing: False