changelog.md

@@ -1,26 +1,5 @@
 # Changelog
 
-## 2025-05-01 - 2.4.2 - fix(cleanup)
-Remove unused scratch files
-
-- Deleted scratch-alias.js, scratch-alias2.js, scratch-alias3.js, scratch-alias4.js, scratch-alias5.js, and scratch.js
-- Clean up temporary alias and scratch test files
-
-## 2025-04-30 - 2.4.1 - fix(Assertion)
-Improve toHaveProperty alias by forwarding arguments correctly for intuitive object property assertions
-
-- Updated the toHaveProperty method in the Assertion class to check the number of arguments and call the appropriate object matcher
-- Added several scratch alias files to demonstrate and test the alias usage
-- Enhanced test cases in test/propertypath to cover alias behavior
-
-## 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)
 Fix test file naming inconsistencies
 

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@push.rocks/smartexpect",
-  "version": "2.4.2",
+  "version": "2.3.3",
   "private": false,
   "description": "A testing library to manage expectations in code, offering both synchronous and asynchronous assertion methods.",
   "main": "dist_ts/index.js",

test/test.propertypath.ts

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

ts/00_commitinfo_data.ts

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

ts/namespaces/object.ts

@@ -62,21 +62,13 @@ export class ObjectMatchers<T extends object, M extends TExecutionType> {
     return this.assertion.customAssertion(
       (v) => {
         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('.');
         let current = obj;
         for (const key of path) {
           if (current == null || !(key in current)) {
             return false;
           }
-          current = (current as any)[key];
+          current = current[key];
         }
         if (arguments.length === 2) {
           return plugins.fastDeepEqual(current, value);

ts/smartexpect.classes.assertion.ts

@@ -355,14 +355,7 @@ export class Assertion<T = unknown, M extends TExecutionType = 'sync'> {
   public toInclude(substring: string) { return this.string.toInclude(substring); }
   public toMatch(regex: RegExp) { return this.string.toMatch(regex); }
   public toBeOneOf(values: any[]) { return this.string.toBeOneOf(values as string[]); }
-  public toHaveProperty(property: string, value?: any) {
-    // Forward only provided arguments to object matcher to preserve argument count
-    if (arguments.length === 2) {
-      return this.object.toHaveProperty(property, value);
-    }
-    return this.object.toHaveProperty(property);
-  }
-  public toHaveOwnProperty(property: string, value?: any) { return this.object.toHaveOwnProperty(property, value); }
+  public toHaveProperty(property: string, value?: any) { return this.object.toHaveProperty(property, value); }
   public toMatchObject(expected: object) { return this.object.toMatchObject(expected); }
   public toBeInstanceOf(constructor: any) { return this.object.toBeInstanceOf(constructor); }
   public toHaveDeepProperty(path: string[]) { return this.object.toHaveDeepProperty(path); }