Commit 0d6330a

committed

Add test

1 parent 2cb9f2a commit 0d6330aCopy full SHA for 0d6330a

File tree

-4

lines changed

schemaregistry
- rules/encryption
  - encrypt-executor.ts
- test/rules/encryption
  - encrypt-executor.spec.ts

-4

lines changed

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -97,7 +97,7 @@ export class FieldEncryptionExecutor extends FieldRuleExecutor {`
`97`	`97`	`let v = this.config.get(key)`
`98`	`98`	`if (v != null) {`
`99`	`99`	`if (v !== value) {`
`100`		`- throw new RuleError('executor already configured')`
	`100`	`+ throw new RuleError('rule config key already set: {key}')`
`101`	`101`	`}`
`102`	`102`	`} else {`
`103`	`103`	`this.config.set(key, value)`

Lines changed: 3 additions & 3 deletions

Original file line number	Diff line number	Diff line change
`@@ -12,11 +12,11 @@ describe('FieldEncryptionExecutor', () => {`
`12`	`12`	`const config = new Map<string, string>();`
`13`	`13`	`config.set('key', 'value');`
`14`	`14`	`executor.configure(clientConfig, config);`
`15`		`- // configure with same args should be fine`
	`15`	`+ // configure with same args is fine`
`16`	`16`	`executor.configure(clientConfig, config);`
`17`	`17`	`const config2 = new Map<string, string>();`
`18`	`18`	`config2.set('key2', 'value2');`
`19`		`- // configure with addition config keys should be fine`
	`19`	`+ // configure with additional config keys is fine`
`20`	`20`	`executor.configure(clientConfig, config);`
`21`	`21`
`22`	`22`	`const clientConfig2: ClientConfig = {`
`@@ -26,7 +26,7 @@ describe('FieldEncryptionExecutor', () => {`
`26`	`26`	`expect(() => executor.configure(clientConfig2, config)).toThrowError()`
`27`	`27`
`28`	`28`	`const config3 = new Map<string, string>();`
`29`		`- config3.set('key', 'value2');`
	`29`	`+ config3.set('key', 'value3');`
`30`	`30`	`expect(() => executor.configure(clientConfig, config3)).toThrowError()`
`31`	`31`	`})`
`32`	`32`	`})`

Comments

(0)