Feat/modules installed update #463
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ules methods - Add modulesInstalled() to check if specific modules are installed - Add getInstalledModules() to get all installed pgpm modules - Add upgradeModules() to update installed modules to latest npm versions - Supports dry-run mode to preview updates - Updates package.json for ALL modules in workspace that reference upgraded packages - Add upgrade-modules CLI command with interactive checkbox prompts - Update README with documentation for new command - Add comprehensive tests for core methods and CLI command
- Remove jest.mock() usage for PgpmPackage and fetchLatestVersion - Use TestFixture from core package for real filesystem testing - Test actual CLI command behavior with real module installations - Use relative import path for TestFixture
- Create pgpm/pgpm/test-utils with TestFixture class - Remove cross-package relative import (../../../core/test-utils) - Follow same pattern as packages/cli/test-utils Co-Authored-By: Dan Lynch <[email protected]>
- Update tests to use @pgpm-testing packages with versions 1.1.0 and 1.2.0 - Add test for recursive dependencies using @pgpm-testing/totp - Add real upgrade test from 1.1.0 to 1.2.0 - Fix upgradeModules to filter out already-up-to-date modules in dry-run - Update snapshots for new package structure (pgpm-base32 vs launchql-base32) Co-Authored-By: Dan Lynch <[email protected]>
- Delete incorrect test-utils from pgpm/pgpm/ - Delete incorrect CLI tests from pgpm/pgpm/src/__tests__/ - Add proper CLI tests in packages/cli/__tests__/upgrade-modules.test.ts - Follow same pattern as init.install.test.ts Co-Authored-By: Dan Lynch <[email protected]>
- Rename pgpm/pgpm directory to pgpm/cli for clarity - Delete mock-heavy tests from pgpm/pgpm/src/__tests__/ - Add pgpm/cli to CI test matrix in run-tests.yaml - Update pnpm-lock.yaml for new directory structure Co-Authored-By: Dan Lynch <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.