-
Notifications
You must be signed in to change notification settings - Fork 27
feat: add semantic sorting and shortcuts to list-installed #1079
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Changes from all commits
c30fdef
b913dc8
fcc4781
5df4bc8
faccdd6
aab50ef
d92dd52
3168294
662f4e0
8aff3fc
71049cd
1d41039
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -20,8 +20,6 @@ Method %OnNew( | |
| set ..Version = pResolvedReference.Version | ||
| set ..Deployed = pResolvedReference.Deployed | ||
| set ..PlatformVersions = pResolvedReference.PlatformVersions | ||
| set ..VersionString = pResolvedReference.VersionString | ||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This seems like a mistake in merging from main. These should not be removed. |
||
| set ..AllVersions = pResolvedReference.AllVersions | ||
| } | ||
| quit $$$OK | ||
| } | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -352,4 +352,16 @@ Method TestUninstallWithoutModuleName() | |
| do $$$AssertNotTrue(exists, "Module removed successfully.") | ||
| } | ||
|
|
||
| Method TestListSortingFlags() | ||
| { | ||
| do $$$LogMessage("Testing list-installed sorting flags") | ||
| set status = ..RunCommand("list") | ||
| set status = ..RunCommand("list -s n") | ||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Tests should read output and confirm its sorted in the correct order. Should also test ascending and descending for each sort modifier |
||
| do $$$AssertStatusOK(status, "Sort by name command executed successfully") | ||
| set status = ..RunCommand("list -s v-d") | ||
| do $$$AssertStatusOK(status, "Sort by version descending executed successfully") | ||
| set status = ..RunCommand("list -s d-d") | ||
| do $$$AssertStatusOK(status, "Sort by date descending executed successfully") | ||
| } | ||
|
|
||
| } | ||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Default sort (if flag not provided) should be ascending by name. Should indicate that here.