Doc update. (#76)

Author: StephenMolloy

Readme.md

@@ -44,7 +44,9 @@ The specific settings available for the new session state module and providers a
   * :warning: ***Action Required*** Sql provider `RepositoryType` - This new setting provides a little more flexibility for repository configuration beyond a single boolean for in-memory optimized tables. Possible values are `SqlServer|InMemory|InMemoryDurable|FrameworkCompat`. Read about what each configuration is at our [SqlSessionStateProviderAsync](docs/SqlSessionStateProviderAsync.md) doc page.
 
     > The mechanics of nuget package upgrade results in removing old elements and re-adding the boiler-plate elements in configuration. To restore In-Memory functionality, set 'RepositoryType' to `InMemory`. To continue using an existing non-memory-optimized table without stored procedures, set 'RespositoryType' to `FrameworkCompat`. The recommendation is to update to the new table schema and use stored procedures with `SqlServer`, but this will ignore existing sessions in previously existing tables.
+  * Moved to use `Microsoft.Data.SqlClient` instead of old `System.Data.SqlClient`. This allows for more modern features such as Token Authorization.
   * The Sql provider's `UseInMemoryTable` is deprecated. It will continue to be respected in the absence of `RepositoryType`, but is overridden by that setting if given.
   * Sql provider `SessionTableName` - A new setting that allows users to target a specific table in their database rather than being forced to use the default table names.
   * CosmosDB `collectionId` is now `containerId` in keeping with the updated terminology from the CosmosDB offering. Please use the updated parameter name when configuring your provider. (The old name will continue to work just the same.)
+  * CosmosDB `partitionKeyPath` and `partitionNumUsedByProvider` are now ignored. Logical partitions are free and encouraged to be as numerous as possible in CosmosDB. As such, we have eliminated the old way of limiting and (unevenly) distributing sessions to partitions and now simply use the full SessionID as the partition key.
   * CosmosDB `connectionProtocol` is obsolete. It will not cause errors to have it in configuration, but it is ignored. The current [CosmosDB SDK chooses the protocol based on connection mode](https://learn.microsoft.com/en-us/azure/cosmos-db/nosql/sdk-connection-modes).

test/Microsoft.AspNet.SessionState.CosmosDBSessionStateProviderAsync.Test/CosmosDBSessionStateProviderAsyncTest.cs

@@ -1132,8 +1132,6 @@ private void SetupDatabaseMock(Mock<Database> databaseMoq, string id = Container
                 PartitionKeyPath = partitionKeyPath,
                 DefaultTimeToLive = DefaultSessionTimeoutInSec
             };
-// smolloy TODO DEBUG
-//containerResponseMoq.SetupGet(cr => cr.Resource).Returns(existingContainerProperties);
 
             databaseMoq.Setup(database => database.CreateContainerIfNotExistsAsync(
                            It.IsAny<ContainerProperties>(), It.IsAny<int>(),
@@ -1145,18 +1143,6 @@ private void SetupDatabaseMock(Mock<Database> databaseMoq, string id = Container
                                containerResponseMoq.SetupGet(cr => cr.Resource).Returns(props);
                            return (ContainerResponse)containerResponseMoq.Object;
                        });
-//                Task.FromResult((ContainerResponse)containerResponseMoq.Object));
-
-            // smolloy TODO DEBUG
-            //databaseMoq.Setup(database => database.CreateContainerIfNotExistsAsync(
-            //               It.Is<ContainerProperties>(properties => properties.Id == id),
-            //               It.IsAny<int>(), It.IsAny<RequestOptions>(), It.IsAny<CancellationToken>()))
-            //           .Returns(Task.FromResult((ContainerResponse)containerResponseMoq.Object));
-
-            //databaseMoq.Setup(database => database.CreateContainerIfNotExistsAsync(
-            //               It.Is<ContainerProperties>(properties => properties.Id != id),
-            //               It.IsAny<int>(), It.IsAny<RequestOptions>(), It.IsAny<CancellationToken>()))
-            //           .Returns(Task.FromResult((ContainerResponse)null));
         }
 
         private StoredProcedureExecuteResponse<T> CreateStoredProcedureResponseInstance<T>(HttpStatusCode statusCode, T responseBody)