Skip to content

Refactor integration tests#164

Merged
Nadegamra merged 14 commits intomainfrom
nadegamra/tests-increase-consistency
Jun 4, 2025
Merged

Refactor integration tests#164
Nadegamra merged 14 commits intomainfrom
nadegamra/tests-increase-consistency

Conversation

@Nadegamra
Copy link
Collaborator

@Nadegamra Nadegamra commented Jun 2, 2025

  • chai lengthOf() is now used in tests where applicable.
  • Improved TS type usage in integration tests.
  • Removed unnecessary JSON.parse() calls from integration tests.
  • runCommand call results in tests are now always destructured.
  • Fixed a formating test incorrectly checking if all flags have required property.
  • Added required property to all flags.

@Nadegamra Nadegamra changed the title Nadegamra/tests increase consistency Refactor integration tests Jun 2, 2025
@Nadegamra Nadegamra marked this pull request as ready for review June 2, 2025 14:25
@Nadegamra Nadegamra merged commit ba0cccd into main Jun 4, 2025
17 of 19 checks passed
@Nadegamra Nadegamra deleted the nadegamra/tests-increase-consistency branch June 4, 2025 10:03
linasburneika pushed a commit that referenced this pull request Aug 19, 2025
* chai lengthOf() is now used in tests where applicable.

* Improved typing in integration tests. Removed unnecessary JSON.parse() calls.

* Improved typing in integration tests.

* Added missing 'required' property to a flag.

* Added missing 'required' property to a flag.

* Fixed failing tests. Added missing 'required' properties in flags.

* Added missing 'required' property to a flag.

* Added missing 'required' property to a flag.

* Fixed failing native client tests.

* Re-enabled a disabled test.

* Undid removal of .to.have.property() statements from tests.

* Undid more removals of .to.have.property() statements from tests.

* Undid addition of 'required: false' properties to some flags.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants