Merge pull request #119 from dolittle/validate-collection-name

Validate collection names, upgrade contracts and add specs.

Author: woksin

.eslintrc.js

@@ -13,7 +13,7 @@ module.exports = {
         '@typescript-eslint/unified-signatures': 'off',
         'import/no-extraneous-dependencies': 'off',
         'eol-last': 'error',
-        'no-multiple-empty-lines': ['error', { 'max': 1, 'maxEOF': 0 }],
+        'no-multiple-empty-lines': ['error', {'max': 1, 'maxEOF': 0 }],
         'header/header': [
             2,
             'line',
@@ -61,7 +61,7 @@ module.exports = {
 		        'MethodDefinition[accessibility!="private"][value.type="FunctionExpression"]:not(MethodDefinition[value.type="TSEmptyBodyFunctionExpression"] + MethodDefinition[value.type="FunctionExpression"])',
             ]
         }],
-        
+
         'jsdoc/require-returns': ['error', {
             'contexts': [
                 'ArrowFunctionExpression',

.gitignore

@@ -65,3 +65,5 @@ Samples/**/data
 Source/**/Distribution
 Samples/**/Distribution
 Source/**/README.md
+
+.idea

Source/artifacts/package.json

@@ -47,7 +47,7 @@
         "@dolittle/concepts": "6.0.0",
         "@dolittle/rudiments": "6.0.0",
         "@dolittle/types": "6.0.0",
-        "@dolittle/runtime.contracts": "6.6.0-sam.2"
+        "@dolittle/runtime.contracts": "6.6.0-sam.3"
     },
     "devDependencies": {
         "@types/is-natural-number": "^4.0.0"

Source/embeddings/package.json

@@ -45,9 +45,9 @@
     },
     "dependencies": {
         "@dolittle/concepts": "6.0.0",
-        "@dolittle/contracts": "6.6.0-sam.2",
+        "@dolittle/contracts": "6.6.0-sam.3",
         "@dolittle/rudiments": "6.0.0",
-        "@dolittle/runtime.contracts": "6.6.0-sam.2",
+        "@dolittle/runtime.contracts": "6.6.0-sam.3",
         "@dolittle/sdk.artifacts": "22.2.0-sam.3",
         "@dolittle/sdk.common": "22.2.0-sam.3",
         "@dolittle/sdk.dependencyinversion": "22.2.0-sam.3",

Source/eventHorizon/package.json

@@ -46,7 +46,7 @@
     "dependencies": {
         "@dolittle/concepts": "6.0.0",
         "@dolittle/rudiments": "6.0.0",
-        "@dolittle/runtime.contracts": "6.6.0-sam.2",
+        "@dolittle/runtime.contracts": "6.6.0-sam.3",
         "@dolittle/sdk.events": "22.2.0-sam.3",
         "@dolittle/sdk.execution": "22.2.0-sam.3",
         "@dolittle/sdk.protobuf": "22.2.0-sam.3",

Source/events.filtering/package.json

@@ -45,9 +45,9 @@
     },
     "dependencies": {
         "@dolittle/concepts": "6.0.0",
-        "@dolittle/contracts": "6.6.0-sam.2",
+        "@dolittle/contracts": "6.6.0-sam.3",
         "@dolittle/rudiments": "6.0.0",
-        "@dolittle/runtime.contracts": "6.6.0-sam.2",
+        "@dolittle/runtime.contracts": "6.6.0-sam.3",
         "@dolittle/sdk.common": "22.2.0-sam.3",
         "@dolittle/sdk.dependencyinversion": "22.2.0-sam.3",
         "@dolittle/sdk.events": "22.2.0-sam.3",

Source/events.handling/package.json

@@ -45,9 +45,9 @@
     },
     "dependencies": {
         "@dolittle/concepts": "6.0.0",
-        "@dolittle/contracts": "6.6.0-sam.2",
+        "@dolittle/contracts": "6.6.0-sam.3",
         "@dolittle/rudiments": "6.0.0",
-        "@dolittle/runtime.contracts": "6.6.0-sam.2",
+        "@dolittle/runtime.contracts": "6.6.0-sam.3",
         "@dolittle/sdk.artifacts": "22.2.0-sam.3",
         "@dolittle/sdk.common": "22.2.0-sam.3",
         "@dolittle/sdk.dependencyinversion": "22.2.0-sam.3",

Source/events.processing/package.json

@@ -45,9 +45,9 @@
     },
     "dependencies": {
         "@dolittle/concepts": "6.0.0",
-        "@dolittle/contracts": "6.6.0-sam.2",
+        "@dolittle/contracts": "6.6.0-sam.3",
         "@dolittle/rudiments": "6.0.0",
-        "@dolittle/runtime.contracts": "6.6.0-sam.2",
+        "@dolittle/runtime.contracts": "6.6.0-sam.3",
         "@dolittle/sdk.common": "22.2.0-sam.3",
         "@dolittle/sdk.dependencyinversion": "22.2.0-sam.3",
         "@dolittle/sdk.execution": "22.2.0-sam.3",

Source/events/package.json

@@ -45,9 +45,9 @@
     },
     "dependencies": {
         "@dolittle/concepts": "6.0.0",
-        "@dolittle/contracts": "6.6.0-sam.2",
+        "@dolittle/contracts": "6.6.0-sam.3",
         "@dolittle/rudiments": "6.0.0",
-        "@dolittle/runtime.contracts": "6.6.0-sam.2",
+        "@dolittle/runtime.contracts": "6.6.0-sam.3",
         "@dolittle/sdk.artifacts": "22.2.0-sam.3",
         "@dolittle/sdk.execution": "22.2.0-sam.3",
         "@dolittle/sdk.protobuf": "22.2.0-sam.3",

Source/projections/Builders/Copies/CopyToMongoDBBuilder.ts

@@ -59,6 +59,12 @@ export class CopyToMongoDBBuilder<T> extends ICopyToMongoDBBuilder<T> {
             return undefined;
         }
 
+        const [collectionNameIsValid, collectionNameValidationError] = this._collectionName.isValid();
+        if (!collectionNameIsValid) {
+            results.addFailure(`Cannot create MongoDB read model copies. ${collectionNameValidationError?.message}`);
+            return undefined;
+        }
+
         return new MongoDBCopies(true, this._collectionName, this.buildPropertyConversions());
     }