improved doc, tests

README.md

@@ -10,15 +10,23 @@ Automatically add visitor as anonymous guest with userId
 
 ##Installation
 ```sh
+mrt add accounts-base
+mrt add accounts-password
 mrt add accounts-guest
 ```
 
-then in a client accessible code
+then in client-only code
 ```javascript
-Meteor.loginVisitor
+Meteor.loginVisitor()
 ```
 
-and that's it!  You'll want to clean out the users db of old accounts
+
+optionally (to clean out old guest accounts) in server-only code
+```javascript
+Meteor.loginVisitor();
+```
+
+Now Meteor.userId() will be populated for each new visitor, including across reloads
 
 
 

accounts-guest-client-tests.js

@@ -0,0 +1,9 @@
+Tinytest.add(
+	'guest - account creation',
+	function (test) {
+		res = Meteor.createGuest();
+		console.log(res + 'ko');
+		uid = Meteor.userId();
+		test.isTrue((typeof uid !== undefined && typeof uid !== null), 'no userId returned');
+	}
+);

accounts-guest-client.js

@@ -4,7 +4,8 @@
 * 
 */
 Meteor.createGuest = function (fn) {
-  Accounts.createUser({password: Meteor.uuid(), username: Meteor.uuid(), profile: {guest: "guest", name: 'Guest'}});
+ res = Accounts.createUser({password: Meteor.uuid(), username: Meteor.uuid(), profile: {guest: "guest", name: 'Guest'}});
+ console.log(res);
 };
 
 

accounts-guest-server-tests.js

@@ -0,0 +1,19 @@
+
+Tinytest.add(
+  'guest - remove guests',
+  function (test) {
+	  var before = new Date();
+	  before.setHours(before.getHours() - 100000);
+		Meteor.createGuest();
+		var username = Meteor.uuid();
+		Accounts.createUser({password: Meteor.uuid(), username: username, profile: {guest: "guest", name: 'Guest'}});
+		Meteor.users.update({username: username},{$set: {createdAt: before}});
+		usr = Meteor.users.findOne({username: username});
+		test.isFalse((typeof usr === 'undefined'), 'guest account not defined');
+		before.setHours(before.getHours() + 1);
+		res = Accounts.removeOldGuests(before);
+		test.equal(res, 1, 'should exactly one deleted');
+		usr = Meteor.users.findOne({username: username});
+		test.isTrue((typeof usr === 'undefined'), 'guest account not deleted');}
+ );
+

accounts-guest-server.js

@@ -3,5 +3,6 @@ Accounts.removeOldGuests = function (before) {
     before = new Date();
     before.setHours(before.getHours() - 1);
   }
-  return Meteor.users.remove({createdAt: {$lte: before}, 'profile.anon': 'anon'});
+  res = Meteor.users.remove({createdAt: {$lte: before}, 'profile.guest': 'guest'});
+  return res;
 };

package.js

@@ -11,5 +11,8 @@ Package.on_use(function (api) {
 
 Package.on_test(function (api) {
   api.use(['accounts-base', 'accounts-password', 'tinytest'], ['client','server']);
-  api.add_files('accounts-guest-tests.js', ['client']);
+  api.add_files('accounts-guest-client.js', ['client','server']);
+  api.add_files('accounts-guest-server.js', 'server');
+  api.add_files('accounts-guest-client-tests.js', 'client');
+  api.add_files('accounts-guest-server-tests.js', 'server');
 });