TokenStore: Provide document id via constructor

The primary use case for this is currently the DataAPI, and the quote id is
only available at the highest level of the server, before dapis are
processed.

In any case, the TokenStore was already described in terms of a combination
of document id; namespace; and root field; so it makes sense for doc id to
be part of the constructor.

If a more generic TokenStore is needed in the future, we could go back to
the previous API and wrap it in another class, like a partially applied
function (e.g. `DocumentTokenStore`).

* src/server/token/TokenStore.ts: Move doc_id out of arguments and into the
  constructor.
* test/server/token/TokenStoreTest.ts: Update accordingly.

1 files changed, 8 insertions, 8 deletions

diff --git a/test/server/token/TokenStoreTest.ts b/test/server/token/TokenStoreTest.ts
index 01a263c..76e97af 100644
--- a/test/server/token/TokenStoreTest.ts
+++ b/test/server/token/TokenStoreTest.ts
@@ -142,8 +142,8 @@ describe( 'TokenStore', () =>
             }();
 
             return expect(
-                new Sut( dao, ns, voidIdgen )
-                    .lookupToken( doc_id, token_id )
+                new Sut( dao, doc_id, ns, voidIdgen )
+                    .lookupToken( token_id )
             )
                 .to.eventually.deep.equal( expected );
         } ) );
@@ -171,8 +171,8 @@ describe( 'TokenStore', () =>
             }();
 
             return expect(
-                new Sut( dao, ns, voidIdgen )
-                    .lookupToken( doc_id, token_id )
+                new Sut( dao, doc_id, ns, voidIdgen )
+                    .lookupToken( token_id )
             ).to.eventually.be.rejectedWith( expected_e );
         } );
     } );
@@ -270,8 +270,8 @@ describe( 'TokenStore', () =>
             }();
 
             return expect(
-                new Sut( dao, ns, () => token_id )
-                    .createToken( doc_id )
+                new Sut( dao, doc_id, ns, () => token_id )
+                    .createToken()
             ).to.eventually.deep.equal( expected );
         } ) );
     } );
@@ -386,8 +386,8 @@ describe( 'TokenStore', () =>
             // dispatch, so it's not testing the state transition
             // restrictions that are enforced by the compiler
             return expect(
-                new Sut( dao, ns, voidIdgen )[ method ](
-                    doc_id, <any>token, data
+                new Sut( dao, doc_id, ns, voidIdgen )[ method ](
+                    <any>token, data
                 )
             ).to.eventually.deep.equal( expected );
         } );