Fix some more rubocop issues; regenerate TODO file

Author: DavidS

.rubocop_todo.yml

@@ -1,6 +1,6 @@
 # This configuration was generated by
 # `rubocop --auto-gen-config`
-# on 2019-11-01 14:59:36 +0000 using RuboCop version 0.49.1.
+# on 2019-11-01 16:00:27 +0000 using RuboCop version 0.49.1.
 # The point is for the user to remove these configuration records
 # one by one as the offenses are removed from the code base.
 # Note that changes in the inspected code, or installation of new
@@ -30,12 +30,6 @@ Lint/Void:
   Exclude:
     - 'spec/unit/facter/lvm_support_spec.rb'
 
-# Offense count: 3
-# Configuration parameters: AllowHeredoc, AllowURI, URISchemes, IgnoreCopDirectives, IgnoredPatterns.
-# URISchemes: http, https
-Metrics/LineLength:
-  Max: 309
-
 # Offense count: 252
 # Configuration parameters: AssignmentOnly.
 RSpec/InstanceVariable:
@@ -54,11 +48,6 @@ RSpec/MultipleDescribes:
   Exclude:
     - 'spec/unit/facter/lvm_support_spec.rb'
 
-# Offense count: 4
-RSpec/RepeatedDescription:
-  Exclude:
-    - 'spec/unit/type/logical_volume_spec.rb'
-
 # Offense count: 2
 RSpec/ScatteredSetup:
   Exclude:
@@ -83,13 +72,6 @@ Style/ClassAndModuleChildren:
   Exclude:
     - 'lib/puppet_x/lvm/output.rb'
 
-# Offense count: 1
-Style/Documentation:
-  Exclude:
-    - 'lib/puppet/parser/functions/**/*'
-    - 'spec/**/*'
-    - 'lib/puppet_x/lvm/output.rb'
-
 # Offense count: 1
 # Configuration parameters: ExpectMatchingDefinition, Regex, IgnoreExecutableScripts, AllowedAcronyms.
 # AllowedAcronyms: CLI, DSL, ACL, API, ASCII, CPU, CSS, DNS, EOF, GUID, HTML, HTTP, HTTPS, ID, IP, JSON, LHS, QPS, RAM, RHS, RPC, SLA, SMTP, SQL, SSH, TCP, TLS, TTL, UDP, UI, UID, UUID, URI, URL, UTF8, VM, XML, XMPP, XSRF, XSS

spec/unit/type/logical_volume_spec.rb

@@ -208,7 +208,7 @@
     }.not_to raise_error
   end
 
-  it 'invalid stripesize raises error' do
+  it 'string stripesize raises error' do
     expect {
       resource = Puppet::Type.type(:logical_volume).new(
         name: 'bert',
@@ -258,7 +258,7 @@
     }.not_to raise_error
   end
 
-  it 'invalid readahead raises error' do
+  it 'string readahead raises error' do
     expect {
       resource = Puppet::Type.type(:logical_volume).new(
         name: 'bert',
@@ -333,7 +333,7 @@
     }.not_to raise_error
   end
 
-  it 'region_size with invalid value throws error' do
+  it 'region_size with string value throws error' do
     expect {
       resource = Puppet::Type.type(:logical_volume).new(
         name: 'fred',