Merge pull request #106 from dev-sec/apparmor_cmp

revert to using cmp for 1.6.3.2 because of implicit string conversion
dev-sec · Feb 1, 2021 · 9ae527c · 9ae527c
2 parents 201cbcf + bce9c4f
commit 9ae527c
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/.rubocop.yml b/.rubocop.yml
@@ -32,3 +32,5 @@ Lint/AmbiguousBlockAssociation:
  Enabled: false
 Lint/AmbiguousRegexpLiteral:
  Enabled: false
+Style/NumericPredicate:
+ Enabled: false
diff --git a/controls/1_6_mandatory_access_control.rb b/controls/1_6_mandatory_access_control.rb
@@ -183,7 +183,7 @@
  only_if { cis_level == 2 && package('apparmor').installed? }
 
  describe command('apparmor_status --profiled') do
- its('stdout') { should cmp.positive? }
+ its('stdout') { should cmp > 0 }
  end
 
  describe command('apparmor_status --complaining') do