From f89a5c600434483b739101c618e3d64e16267431 Mon Sep 17 00:00:00 2001
From: Jeff Geerling <geerlingguy@mac.com>
Date: Sun, 14 Jun 2020 15:42:33 -0500
Subject: [PATCH] Issue #202: More idiomatic Ansible Python style.

---
 test-plugin/test_plugins/blue.py | 21 +++++++++++++++------
 1 file changed, 15 insertions(+), 6 deletions(-)

diff --git a/test-plugin/test_plugins/blue.py b/test-plugin/test_plugins/blue.py
index c201bd2..f0db156 100644
--- a/test-plugin/test_plugins/blue.py
+++ b/test-plugin/test_plugins/blue.py
@@ -1,14 +1,23 @@
-# blue Ansible test plugin definition.
+# Ansible custom 'blue' test plugin definition.
 
 def is_blue(string):
-    blue_values = ['blue', '#0000ff', '#00f', 'rgb(0,0,255)']
+    ''' Return True if a valid CSS value of 'blue'. '''
+    blue_values = [
+        'blue',
+        '#0000ff',
+        '#00f',
+        'rgb(0,0,255)',
+        'rgb(0%,0%,100%)',
+    ]
     if string in blue_values:
-      return True
+        return True
     else:
-      return False
+        return False
 
 class TestModule(object):
-    ''' custom playbook jinja2 tests '''
+    ''' Return dict of custom jinja tests. '''
 
     def tests(self):
-        return dict(blue=is_blue)
+        return {
+            'blue': is_blue
+        }