Added information about the phone home feature and the option to opt out

Platika · Apr 11, 2011 · 334c75a · 334c75a
1 parent 7be05a1
commit 334c75a
Show file tree

Hide file tree

Showing 8 changed files with 33 additions and 8 deletions.
diff --git a/00HelloWorld/tests/appTest.js b/00HelloWorld/tests/appTest.js
@@ -89,7 +89,9 @@ YUI().use('node', 'console', 'test', function(Y) {
 
   reporter.render("#testReport");
   Y.Test.Runner.add(unitTests);
-  Y.Test.Runner.subscribe(Y.Test.Runner.COMPLETE_EVENT, logResultsToServer);
+  if ( !( isDeclared("phoneHome") && (phoneHome === false) ) ) {
+    Y.Test.Runner.subscribe(Y.Test.Runner.COMPLETE_EVENT, logResultsToServer);
+  } 
   Y.Test.Runner.run();
 
  });
diff --git a/01DieRoll/tests/appTest.js b/01DieRoll/tests/appTest.js
@@ -102,7 +102,7 @@ YUI().use('node', 'console', 'test', function(Y) {
 
   function logResultsToServer(data){
     var testResults, serverReporter;
-
+        
     testResults = Y.Test.Runner.getResults();
     serverReporter = new Y.Test.Reporter("http://nwghost.com/tdd-collector.php", Y.Test.Format.TAP);
     serverReporter.addField("authorsName", authorsNameExists() ? authorsName : "UNKNOWN AUTHOR");
@@ -115,7 +115,9 @@ YUI().use('node', 'console', 'test', function(Y) {
 
   reporter.render("#testReport");
   Y.Test.Runner.add(unitTests);
-  Y.Test.Runner.subscribe(Y.Test.Runner.COMPLETE_EVENT, logResultsToServer);
+  if ( !( isDeclared("phoneHome") && (phoneHome === false) ) ) {
+    Y.Test.Runner.subscribe(Y.Test.Runner.COMPLETE_EVENT, logResultsToServer);
+  } 
   Y.Test.Runner.run();
 
  });
diff --git a/02GuessingGame/tests/appTest.js b/02GuessingGame/tests/appTest.js
@@ -173,7 +173,9 @@ YUI().use('node', 'console', 'test', function(Y) {
 
   reporter.render("#testReport");
   Y.Test.Runner.add(unitTests);
-  Y.Test.Runner.subscribe(Y.Test.Runner.COMPLETE_EVENT, logResultsToServer);
+  if ( !( isDeclared("phoneHome") && (phoneHome === false) ) ) {
+    Y.Test.Runner.subscribe(Y.Test.Runner.COMPLETE_EVENT, logResultsToServer);
+  } 
   Y.Test.Runner.run();
 
  });
diff --git a/03LoopedGuessingGame/tests/appTest.js b/03LoopedGuessingGame/tests/appTest.js
@@ -173,7 +173,9 @@ YUI().use('node', 'console', 'test', function(Y) {
 
   reporter.render("#testReport");
   Y.Test.Runner.add(unitTests);
-  Y.Test.Runner.subscribe(Y.Test.Runner.COMPLETE_EVENT, logResultsToServer);
+  if ( !( isDeclared("phoneHome") && (phoneHome === false) ) ) {
+    Y.Test.Runner.subscribe(Y.Test.Runner.COMPLETE_EVENT, logResultsToServer);
+  } 
   Y.Test.Runner.run();
 
  });
diff --git a/04ParameterizedGuessingGame/tests/appTest.js b/04ParameterizedGuessingGame/tests/appTest.js
@@ -184,7 +184,9 @@ YUI().use('node', 'console', 'test', function(Y) {
 
   reporter.render("#testReport");
   Y.Test.Runner.add(unitTests);
-  Y.Test.Runner.subscribe(Y.Test.Runner.COMPLETE_EVENT, logResultsToServer);
+  if ( !( isDeclared("phoneHome") && (phoneHome === false) ) ) {
+    Y.Test.Runner.subscribe(Y.Test.Runner.COMPLETE_EVENT, logResultsToServer);
+  } 
   Y.Test.Runner.run();
 
  });
diff --git a/05UserSpecifiedMinMaxGame/tests/appTest.js b/05UserSpecifiedMinMaxGame/tests/appTest.js
@@ -178,7 +178,9 @@ YUI().use('node', 'console', 'test', function(Y) {
 
   reporter.render("#testReport");
   Y.Test.Runner.add(unitTests);
-  Y.Test.Runner.subscribe(Y.Test.Runner.COMPLETE_EVENT, logResultsToServer);
+  if ( !( isDeclared("phoneHome") && (phoneHome === false) ) ) {
+    Y.Test.Runner.subscribe(Y.Test.Runner.COMPLETE_EVENT, logResultsToServer);
+  } 
   Y.Test.Runner.run();
 
  });
diff --git a/index.html b/index.html
@@ -15,6 +15,10 @@
         width: 800px;
       }
 
+      #notice {
+        text-align: left;
+      }
+
       th {
         background-color: #ccccee;
       }
@@ -43,6 +47,13 @@ <h1>Test-Driven Learning (TDL)<br />Introduction to JavaScript</h1>
 
         </script>
       </table>
+      <div id="notice">
+        <p><em>Privacy Notice:</em> These tasks contain a "phone home" feature that, if you are online while running the tests, sends information to a remote server including the value of the authorsName variable and which tests have failed or passed. The purpose of this information is to see the progress that learners make while working through the tasks and to spot difficulties that many learners encounter that might indicate that a task should be revised.</p>
+        <p>You can disable the phone home feature by including the following line at the top of your app.js files:</p>
+        <pre>
+          var phoneHome = false;
+        </pre>
+      </div>
     </div>
   </body>
 </html>
diff --git a/template/tests/appTest.js b/template/tests/appTest.js
@@ -83,7 +83,9 @@ YUI().use('node', 'console', 'test', function(Y) {
 
   reporter.render("#testReport");
   Y.Test.Runner.add(unitTests);
-  Y.Test.Runner.subscribe(Y.Test.Runner.COMPLETE_EVENT, logResultsToServer);
+  if ( !( isDeclared("phoneHome") && (phoneHome === false) ) ) {
+    Y.Test.Runner.subscribe(Y.Test.Runner.COMPLETE_EVENT, logResultsToServer);
+  } 
   Y.Test.Runner.run();
 
  });