feat(object): add toHaveOwnProperty method and improve property-path matching in object assertions

This commit is contained in:
Philipp Kunz 2025-04-30 18:24:28 +00:00
parent 0d3d498240
commit 0351da2878
7 changed files with 39 additions and 4 deletions

View File

@ -1,5 +1,13 @@
# Changelog # Changelog
## 2025-04-30 - 2.4.0 - feat(object)
add toHaveOwnProperty method and improve property-path matching in object assertions
- Added 'toHaveOwnProperty' as a direct method on Assertion to check for own properties
- Enhanced property path evaluation in 'toHaveProperty' to handle nested keys more robustly
- Renamed test file to maintain consistent naming for expect.any tests
- Introduced scratch.js for manual testing and debugging of property matchers
## 2025-04-30 - 2.3.3 - fix(tests) ## 2025-04-30 - 2.3.3 - fix(tests)
Fix test file naming inconsistencies Fix test file naming inconsistencies

16
scratch.js Normal file
View File

@ -0,0 +1,16 @@
import * as smartexpect from './dist_ts/index.js';
class Foo { constructor(){ this.foo='bar'; } }
console.log('foo in instance:', 'foo' in new Foo());
console.log('hasOwn foo:', Object.prototype.hasOwnProperty.call(new Foo(), 'foo'));
try {
smartexpect.expect(new Foo()).object.toHaveProperty('foo');
console.log('toHaveProperty passed');
} catch (err) {
console.error('toHaveProperty failed:', err.message);
}
try {
smartexpect.expect(new Foo()).object.toHaveOwnProperty('foo');
console.log('toHaveOwnProperty passed');
} catch (err) {
console.error('toHaveOwnProperty failed:', err.message);
}

View File

@ -2,7 +2,9 @@ import { tap, expect as tExpect } from '@push.rocks/tapbundle';
import * as smartexpect from '../dist_ts/index.js'; import * as smartexpect from '../dist_ts/index.js';
tap.test('toHaveProperty nested path via dot notation', async () => { tap.test('toHaveProperty nested path via dot notation', async () => {
const testObject = { level1: { level2: { level3: 'value' } } }; const testObject = { level1: { level2: { level3: 'value' }}, publicTest: 'hi' };
smartexpect.expect(testObject).object.toHaveProperty('publicTest');
// Existence check // Existence check
smartexpect.expect(testObject).object.toHaveProperty('level1.level2.level3'); smartexpect.expect(testObject).object.toHaveProperty('level1.level2.level3');
// Value check // Value check

View File

@ -3,6 +3,6 @@
*/ */
export const commitinfo = { export const commitinfo = {
name: '@push.rocks/smartexpect', name: '@push.rocks/smartexpect',
version: '2.3.3', version: '2.4.0',
description: 'A testing library to manage expectations in code, offering both synchronous and asynchronous assertion methods.' description: 'A testing library to manage expectations in code, offering both synchronous and asynchronous assertion methods.'
} }

View File

@ -62,13 +62,21 @@ export class ObjectMatchers<T extends object, M extends TExecutionType> {
return this.assertion.customAssertion( return this.assertion.customAssertion(
(v) => { (v) => {
const obj = v as any; const obj = v as any;
// first check for a literal property (including inherited)
if (property in obj) {
if (arguments.length === 2) {
return plugins.fastDeepEqual(obj[property], value);
}
return true;
}
// no direct key, try nested path via dot notation
const path = property.split('.'); const path = property.split('.');
let current = obj; let current = obj;
for (const key of path) { for (const key of path) {
if (current == null || !(key in current)) { if (current == null || !(key in current)) {
return false; return false;
} }
current = current[key]; current = (current as any)[key];
} }
if (arguments.length === 2) { if (arguments.length === 2) {
return plugins.fastDeepEqual(current, value); return plugins.fastDeepEqual(current, value);

View File

@ -356,6 +356,7 @@ export class Assertion<T = unknown, M extends TExecutionType = 'sync'> {
public toMatch(regex: RegExp) { return this.string.toMatch(regex); } public toMatch(regex: RegExp) { return this.string.toMatch(regex); }
public toBeOneOf(values: any[]) { return this.string.toBeOneOf(values as string[]); } public toBeOneOf(values: any[]) { return this.string.toBeOneOf(values as string[]); }
public toHaveProperty(property: string, value?: any) { return this.object.toHaveProperty(property, value); } public toHaveProperty(property: string, value?: any) { return this.object.toHaveProperty(property, value); }
public toHaveOwnProperty(property: string, value?: any) { return this.object.toHaveOwnProperty(property, value); }
public toMatchObject(expected: object) { return this.object.toMatchObject(expected); } public toMatchObject(expected: object) { return this.object.toMatchObject(expected); }
public toBeInstanceOf(constructor: any) { return this.object.toBeInstanceOf(constructor); } public toBeInstanceOf(constructor: any) { return this.object.toBeInstanceOf(constructor); }
public toHaveDeepProperty(path: string[]) { return this.object.toHaveDeepProperty(path); } public toHaveDeepProperty(path: string[]) { return this.object.toHaveDeepProperty(path); }