Autoformat only normal mode

autoload/puppet/format.vim

@@ -1,6 +1,13 @@
 "
 " Simple format using puppet's l:indents and align hashrockets function
 function! puppet#format#Format() abort
+  " only allow reformatting through the gq command
+  " (e.g. Vim is in normal mode)
+  if mode() !=# 'n'
+    " do not fall back to internal formatting
+    return 0
+  endif
+
   let l:start_lnum = v:lnum
   let l:end_lnum = v:lnum + v:count - 1
   " Don't modify indentation or alignment if called by textwidth. We'll only

test/format/textwidth.vader

@@ -1,3 +1,5 @@
+# First let's test that formatting without textwidth set doesn't do unexpected things
+
 Given puppet (long line):
   # Long comment 123456789 123456789 123456789 123456789 123456789 123456789 123456789 123456789 123456789 123456789 123456789 123456789 123456789 123456789
   file { 'foo':
@@ -13,12 +15,18 @@ Expect puppet (nothing changed):
     ensure => present,
   }
 
+-------------------------------------------------------------------------------
+# Now let's check that the plugin does the right thing when textwidth is set.
+#
+# All of the tests below this point expect to have 'textwidth' set during the test.
 Before (set textwidth):
   set textwidth=76
 
 After (unset textwidth):
   set textwidth=0
 
+-------------------------------------------------------------------------------
+# note: using the same fixture as before
 Do (format all text with textwidth set):
   gqG
 
@@ -30,3 +38,19 @@ Expect puppet (comment is wrapped into more lines):
     ensure => present,
   }
 
+-------------------------------------------------------------------------------
+Given puppet (long line before editing):
+  file { 'foo':
+    ensure => present,
+    source => ['puppet:///modules/very_long_module_name_that_will_not_stop'],
+  }
+
+Do (type in some more on long line):
+  jjA mode
+
+Expect puppet (formatting did not change line):
+  file { 'foo':
+    ensure => present,
+    source => ['puppet:///modules/very_long_module_name_that_will_not_stop'], mode
+  }
+