Skip to content

Commit

Permalink
[aws-cli] - issue devcontainers#1224 - installing less for paginated …
Browse files Browse the repository at this point in the history
…output and avoiding errors (devcontainers#1226)

Co-authored-by: Daniel Doyle <[email protected]>
  • Loading branch information
gauravsaini04 and ddoyle2017 authored Jan 28, 2025
1 parent d9c2de0 commit 28b75a6
Show file tree
Hide file tree
Showing 3 changed files with 22 additions and 1 deletion.
2 changes: 1 addition & 1 deletion src/aws-cli/install.sh
Original file line number Diff line number Diff line change
Expand Up @@ -68,7 +68,7 @@ check_packages() {

export DEBIAN_FRONTEND=noninteractive

check_packages curl ca-certificates gnupg2 dirmngr unzip bash-completion
check_packages curl ca-certificates gnupg2 dirmngr unzip bash-completion less

verify_aws_cli_gpg_signature() {
local filePath=$1
Expand Down
13 changes: 13 additions & 0 deletions test/aws-cli/less_installed.sh
Original file line number Diff line number Diff line change
@@ -0,0 +1,13 @@
#!/bin/bash

set -e

# Import test library for `check` command
source dev-container-features-test-lib

check "less is installed, pagination works !" less --version
check "less binary installation path" which less
check "Testing paginated output with less" ls -R / | less

# Report result
reportResults
8 changes: 8 additions & 0 deletions test/aws-cli/scenarios.json
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
{
"less_installed": {
"image": "mcr.microsoft.com/devcontainers/base:ubuntu",
"features": {
"aws-cli": {}
}
}
}

0 comments on commit 28b75a6

Please sign in to comment.