changelog.md

@@ -1,39 +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
-
-- Rename 'test/test.diffOutput.ts' to 'test/test.diffoutput.ts' to standardize filename casing
-- Rename 'test/test.propertyPath.ts' to 'test/test.propertypath.ts' for consistent file naming
-
-## 2025-04-30 - 2.3.2 - fix(object)
-Update toHaveProperty matcher to support nested property paths using dot notation
-
-- Changed toHaveProperty implementation to split property strings on '.' and traverse nested objects
-- Fixed value comparison for nested properties by comparing the final drilled value instead of direct property access
-- Added tests for nested property access in test/propertyPath.ts
-
 ## 2025-04-30 - 2.3.1 - fix(readme)
 Improve README documentation with detailed 'Why SmartExpect' benefits section
 

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@push.rocks/smartexpect",
-  "version": "2.4.2",
+  "version": "2.3.1",
   "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

@@ -1,17 +0,0 @@
-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');
-  // Existence check
-  smartexpect.expect(testObject).object.toHaveProperty('level1.level2.level3');
-  // Value check
-  smartexpect.expect(testObject).object.toHaveProperty('level1.level2.level3', 'value');
-  // Negation for missing deep property
-  smartexpect.expect(testObject).not.object.toHaveProperty('level1.level2.missing');
-});
-
-export default tap.start();

ts/00_commitinfo_data.ts

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

ts/namespaces/object.ts

@@ -62,24 +62,11 @@ 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 (property in obj) {
+          return false;
-          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];
         }
         if (arguments.length === 2) {
-          return plugins.fastDeepEqual(current, value);
+          return plugins.fastDeepEqual(obj[property], value);
         }
         return true;
       },

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) {
+  public toHaveProperty(property: string, value?: any) { return this.object.toHaveProperty(property, value); }
-    // 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 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); }