diff --git a/manifests/ringbuilder.pp b/manifests/ringbuilder.pp
index 764e4656..4fe0416b 100644
--- a/manifests/ringbuilder.pp
+++ b/manifests/ringbuilder.pp
@@ -2,10 +2,9 @@
 # Rings are used to make decicions about how to map objects in the cluster
 #
 # Specifies the following relationship:
-#  Rings shoudl be created before any devices are added to them
+#  Rings should be created before any devices are added to them
 #  Rings should be rebalanced if anything changes
 # == Parameters
-#  # TODO - I need to review the ringbuilder docs
 #  [*part_power*] The total number of partitions that should exist in the ring.
 #    This is expressed as a power of 2.
 #  [*replicas*] Numer of replicas that should be maintained of each stored object.
diff --git a/spec/defines/swift_storage_loopback_spec.rb b/spec/defines/swift_storage_loopback_spec.rb
index 9291c283..feb75c94 100644
--- a/spec/defines/swift_storage_loopback_spec.rb
+++ b/spec/defines/swift_storage_loopback_spec.rb
@@ -1,3 +1,4 @@
 describe 'swift::storage::loopback' do
-
+  # TODO add unit tests
+  # this is not the highest priority b/c it is really for testing
 end
diff --git a/spec/defines/swift_storage_mount_spec.rb b/spec/defines/swift_storage_mount_spec.rb
index bf14f6f0..95ac9afc 100644
--- a/spec/defines/swift_storage_mount_spec.rb
+++ b/spec/defines/swift_storage_mount_spec.rb
@@ -1,3 +1,4 @@
 describe 'swift::storage::mount' do
-
+  # TODO add unit tests
+  # not a super high priority b/c this is just for testing
 end
diff --git a/spec/defines/swift_storage_node_spec.rb b/spec/defines/swift_storage_node_spec.rb
index c81ece98..637867f6 100644
--- a/spec/defines/swift_storage_node_spec.rb
+++ b/spec/defines/swift_storage_node_spec.rb
@@ -1,2 +1,4 @@
 describe 'swift::storage::node' do
+  # this is just for the SAOI
+  # add tests
 end