Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Silence FAST tests warnings #1635

Merged
merged 2 commits into from
Aug 29, 2023
Merged

Silence FAST tests warnings #1635

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
804e7c9
Silence FAST tests warnings
juliocc Aug 28, 2023
3d487c8
Merge branch 'master' into jccb/fast-tests
juliocc Aug 29, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
4 changes: 2 additions & 2 deletions fast/stages/1-resman/README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -145,7 +145,7 @@ The first set of default tags cannot be overridden and defines the following key

The second set is optional and allows defining a custom tag hierarchy, including IAM bindings that can refer to specific identities, or to the internally defined automation service accounts via their names, like in the following example:

```hcl
```tfvars
tags = {
my-custom-tag = {
values = {
Expand Down Expand Up @@ -258,7 +258,7 @@ classDiagram

This is an example that shows how to populate the relevant variables.

```hcl
```tfvars
tenants = {
tn0 = {
admin_group_email = "tn-0-admins@tenant.example.org"
Expand Down
7 changes: 6 additions & 1 deletion tests/fixtures.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -133,6 +133,12 @@ def plan_summary(module_path, basedir, tf_var_files=None, extra_files=None,
# extract planned outputs
outputs = plan.get('planned_values', {}).get('outputs', {})

# force the destruction of the tftest object, otherwise pytest
# will complain about unraisable exceptions caused by the context
# manager deleting temporary files, including the extra_files that
# tftest tries to remove on cleanup
del tf

return PlanSummary(values, dict(counts), outputs)


Expand All @@ -148,7 +154,6 @@ def inner(module_path, basedir=None, tf_var_files=None, extra_files=None,
**tf_vars):
if basedir is None:
basedir = Path(request.fspath).parent
print(f"{basedir=}")
return plan_summary(module_path=module_path, basedir=basedir,
tf_var_files=tf_var_files, extra_files=extra_files,
**tf_vars)
Expand Down