fix(ses): review suggestions

packages/ses/test/test-tame-faux-data-properties.js

@@ -67,7 +67,15 @@ test('unit test tameFauxDataProperty', t => {
       defineProperty(this, 'foo', { value: newValue });
     },
   };
-  t.is(tfdp(subject2, 'foo', 'bar'), false);
+  // This setter is not behaving like a proper setter for
+  // enabling property overrides, so we would prefer that the answer
+  // immediately below be `false` instead of `true`. But we
+  // commented out the test in tame-faux-data-properties.js
+  // because we cannot be confident it won't cause persistent
+  // side effects.
+  // We include the test anyway, to document that this case does
+  // pass, and to become aware if it stops passing.
+  t.is(tfdp(subject2, 'foo', 'bar'), true);
 
   const subject3 = {
     get foo() {