garrensmith commented on a change in pull request #1144: [partitioned-dbs]
Updates to doc editor
URL: https://github.com/apache/couchdb-fauxton/pull/1144#discussion_r226594110
##########
File path: app/addons/documents/__tests__/resources.test.js
##########
@@ -82,6 +83,25 @@ describe('AllDocs', () => {
});
});
+describe('NewDoc', () => {
+
+ let getUUID;
+ beforeEach(() => {
+ getUUID = sinon.stub(Helpers, 'getUUID').resolves({ uuids: ['abc9876'] });
+ });
+
+ afterEach(() => {
+ getUUID.restore();
+ });
+
+ it('adds partition key to auto-generated ID', () => {
+ const newDoc = new Models.NewDoc(null, { database: {},
defaultPartitionKey: 'part_key' });
Review comment:
I'm not sure we should add a default partition key. That seems to defeat the
purpose of partition keys.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
