Closed tkishel closed 3 years ago
FWIW: I cannot set a "Do Not Merge" (until tested) label 🤷🏻♂️
~ shellspec spec/aws_resource-count-aws.sh
Running: /bin/sh [bash 3.2.57(1)-release]
............
Finished in 4.45 seconds (user 3.84 seconds, sys 0.70 seconds)
12 examples, 0 failures
Testing: I've added spec tests, covering each function in the script (and refactored the script to use functions: for all but parsing the one optional org
parameter, and the jq
check) ... including all of the aws
commands. For the mocking of those commands, I used the example output from the AWS CLI documentation for each command.
Documentation: I don't have the source of the PDF to update it, but the optional org
parameter is documented via a comment.
Description
Accept an
org
parameter to size AWS Member Accounts attached to an AWS Organization. Based upon code written by Adam Clark. Also, corrects an error resettingIFS
Motivation and Context
How Has This Been Tested?
Original functionality tested.
AWS Organization untested (I have no access to one)
Types of changes
Checklist