Fixed #111 -- allow the actor defined within URL configuration to be a Group.

src/Agent.js

@@ -289,6 +289,10 @@ TinCan client library
         Agent.prototype.log("fromJSON");
         var _agent = JSON.parse(agentJSON);
 
+        if (_agent.hasOwnProperty("objectType") && _agent.objectType !== Agent.prototype.objectType) {
+            throw "Unexpected object type for agent: " + _agent.objectType;
+        }
+
         return new Agent(_agent);
     };
 }());

src/Group.js

@@ -156,6 +156,10 @@ TinCan client library
         Group.prototype.log("fromJSON");
         var _group = JSON.parse(groupJSON);
 
+        if (_group.hasOwnProperty("objectType") && _group.objectType !== Group.prototype.objectType) {
+            throw "Unexpected object type for group: " + _group.objectType;
+        }
+
         return new Group(_group);
     };
 }());

src/TinCan.js

@@ -211,7 +211,13 @@ var TinCan;
                     delete qsParams.actor;
                 }
                 catch (ex) {
-                    this.log("_initFromQueryString - failed to set actor: " + ex);
+                    try {
+                        this.actor = TinCan.Group.fromJSON(qsParams.actor);
+                        delete qsParams.actor;
+                    }
+                    catch (ex2) {
+                        this.log("_initFromQueryString - failed to set actor: " + ", ".join([ex, ex2]));
+                    }
                 }
             }
 

test/js/unit/Agent.js

@@ -29,7 +29,19 @@
             ;
 
             result = TinCan.Agent.fromJSON(JSON.stringify(raw));
-            ok(result instanceof TinCan.Agent, "returns TinCan.Agent");
+            ok(result instanceof TinCan.Agent, "returns TinCan.Agent (objectType omitted)");
+
+            raw.objectType = 'Agent';
+            result = TinCan.Agent.fromJSON(JSON.stringify(raw));
+            ok(result instanceof TinCan.Agent, "returns TinCan.Agent (objectType set explicitly)");
+
+            raw.objectType = 'invalid';
+            throws(
+                function () {
+                    TinCan.Agent.fromJSON(JSON.stringify(raw));
+                },
+                "exception on invalid objectType"
+            );
         }
     );
 

test/js/unit/Group.js

@@ -34,6 +34,18 @@
 
             result = TinCan.Group.fromJSON(JSON.stringify(raw));
             ok(result instanceof TinCan.Group, "returns TinCan.Group");
+
+            raw.objectType = 'Group';
+            result = TinCan.Group.fromJSON(JSON.stringify(raw));
+            ok(result instanceof TinCan.Group, "returns TinCan.Group (objectType set explicitly)");
+
+            raw.objectType = 'invalid';
+            throws(
+                function () {
+                    TinCan.Group.fromJSON(JSON.stringify(raw));
+                },
+                "exception on invalid objectType"
+            );
         }
     );