added egs for groups fxns; also reexporting magrittr pipe so its avai…

[sckott]

#68

Series of PRs for re-working examples.

Groups set of fxns

Also reexporting magrittr pipe here

[seankross]

Looks great

CI issues

[seankross]

Whelp CI isn't happy! I tried running test() locally and got this error, which is different than the error gha is throwing:

── Failed tests ────────────────────────────────────────────────────────────
Error (test-billing.R:5:5): aws_billing
Error in `including(list(Metrics = list("UnblendedCost")))`: could not find function "including"
Backtrace:
    ▆
 1. ├─withr::with_package(...) at test-billing.R:2:3
 2. │ └─base::force(code)
 3. ├─webmockr::to_return(...) at test-billing.R:5:5
 4. │ └─webmockr:::assert(.data, "StubbedRequest")
 5. └─webmockr::wi_th(...)

[ FAIL 1 | WARN 0 | SKIP 10 | PASS 151 ]

[sckott]

Thanks. Yeah we've thus far had dummy aws access key and secret key b/c we just needed any string to run the package for those env vars, but if we try to run examples for real of course those don't work. so thinking about whether to just skip examples on CI or run them but take precautions.

[seankross]

My instinct is that software testing that costs money (hits the aws api for real) should not be automated, at least at this point in the trajectory of this project. Maybe we'd do it in the future though.

[sckott]

software testing that costs money (hits the aws api for real) should not be automated,

yeah, good thinking, yeah think i'll just skip egs on CI then.

It'd be cool to run examples against minio/localstack, and I can see about doing that locally for running egs and generating pkgdown docs - but it wouldn't make sense to do if it requires code in the examples to do that since users aren't going to spin up those two services to run examples.

[sckott]

@seankross For the webmockr error, the dev version is specified in Remotes in DESCRIPTION file, guessing that locally you have to install it manually pak::pak("ropensci/webmockr")