This project is mirrored from https://:*****@github.com/hashicorp/terraform.git.
Pull mirroring failed .
Repository mirroring has been paused due to too many failed attempts. It can be resumed by a project maintainer.
Repository mirroring has been paused due to too many failed attempts. It can be resumed by a project maintainer.
- 17 Oct, 2018 40 commits
-
-
Kristin Laemmert authored
-
Martin Atkins authored
This is a more specialized version of ConfigValueFromHCL2 which is specifically for config values that represent the content of a block body in the configuration. By using the schema of that block we can more precisely emulate the old HCL1/HIL behaviors by distinguishing attributes from blocks and applying some slightly different behaviors for the handling of null values and of empty collections that are representing the absense of blocks of a particular type.
-
Kristin Laemmert authored
-
James Bardin authored
The last 2 broken tests will be hanlded later
-
James Bardin authored
Use the new SimpleDiff method of the provider so that the diff isn't altered by ForceNew attributes. Always set an "id" as RequiresReplace so core knows an instance will be replaced, even if all ForceNew attributes are filtered out due to ignore_changes.
-
Martin Atkins authored
The underlying bug that was causing this test to hang has since been fixed.
-
Martin Atkins authored
If we don't override these then the tests can only pass on one platform.
-
Martin Atkins authored
This reinstates an old behavior that was lost in the reorganization of how we deal with the -var and -var-file options. This fix is verified by TestApply_planVars now passing.
-
Martin Atkins authored
In the new implementation of collecting variables I initially forgot the JSON variant of terraform.tfvars. This fix is verified by TestApply_varFileDefaultJSON now passing.
-
James Bardin authored
Terraform now handles any actual "diffing" of resource, and the existing Diff functions are only used to shim the schema.Provider to the new methods. Since terraform is handling what used to be the Diff, the provider now should not modify the diff based on RequiresNew due to it interfering with the ignore_changes handling.
-
James Bardin authored
The helper resource tests won't pass for now, as they use a terraform.MockProvider which can't be used in the schema.Provider shims.
-
James Bardin authored
We need to ensure that a destroyed value is returned as such from apply
-
James Bardin authored
Tests often call Plan multiple times on the same context to verify there are no changes, so we need to make sure changes don't accumulate.
-
James Bardin authored
We also need to convert legacy states for helper resource tests.
-
Martin Atkins authored
-
Martin Atkins authored
Since we need to know the index to know the result type for a tuple, we need a special case here to deal with that situation and return cty.DynamicVal; we can't predict the result type exactly until we know the element type.
-
Martin Atkins authored
-
Martin Atkins authored
This was previously targeting the old state manager and state types, so it needed some considerable rework to get it working again with the new state types. Since our new resource address syntax lacks the weird extra .deposed special case we had before, we instead interpret addresses as whole-instance addresses here and remove the deposed objects along with the current one (if present), since this is more likely to match with user expectations because we don't consider deposed objects to be independently addressable in any other situation. With that said, to be more explicit about what is going on we do now have a -dry-run mode and maintain separate counts of current and deposed instances so that we can expose that in the UI where relevant.
-
Martin Atkins authored
-
Martin Atkins authored
The tests in here are illustrating that this package is not yet finished, but we plan to run a release before we finish this and so we'll skip those tests for now with the intent of reinstating this again once we return to finish this up.
-
Martin Atkins authored
-
Martin Atkins authored
-
Martin Atkins authored
-
Martin Atkins authored
-
Martin Atkins authored
-
Martin Atkins authored
-
Martin Atkins authored
-
Martin Atkins authored
We temporarily disabled this because it needed some further work to update it for the new state models, which has now been done. We no longer need the configuration objects for the outputs because the state itself contains all of the information needed for displaying these.
-
Martin Atkins authored
-
Martin Atkins authored
-
Martin Atkins authored
-
Martin Atkins authored
We no longer support legacy remote state, so the behaviors these tests were covering are no longer present.
-
Martin Atkins authored
-
Martin Atkins authored
We used to treat the "id" attribute of a resource as special and elevate it into its own struct field "ID" in the state, but the new state format and provider protocol treats it just as any other attribute. However, it's still useful to show the value of a single identifying attribute when there isn't room in the UI for showing all of the attributes, and so here we take a new strategy of considering "id" along with some other conventional names as special only in the UI layer. This new heuristic approach can be adjusted over time as new provider patterns emerge, but for now it covers some common conventions we've seen in real providers. With that said, since all existing providers made for Terraform versions prior to v0.12 were forced to set "id", we won't see any use of other attributes here until providers are updated to remove the placeholder ids they were generating in cases where an id was not actually relevant but was forced by the old protocol. At that point the UX should be improved by showing a more relevant attribute instead. We now also allow for the possibility of no id at all, since that is valid for resources that exist only within the Terraform state, like the ones from the "random" and "tls" providers.
-
Martin Atkins authored
This command isn't yet updated for the new state types, but since we were not returning a non-successful error status here the tests were just failing in a weird way instead. Now we'll fail with a message that makes it clear there is still work to do in the real implementation here.
-
Martin Atkins authored
We now need to give a provider schema that the test fixtures conform to, so we can validate against it.
-
Martin Atkins authored
We previously stubbed most of this out because it hadn't yet been updated to support the new state types, etc. This restores all of the previous behavior as covered by the tests. We intentionally remove one behavior that was not covered by the tests: we used to allow retrieval of outputs from non-root modules using the -module option, but since we no longer persist non-root outputs in the state we can no longer support this without a full expression evaluation walk, and that'd be overkill for this otherwise-simple command. Descendant module outputs are not part of the public interface of a configuration anyway, so accessing them from outside in this way is an anti-pattern. (For debugging scenarios it is still possible to access these from "terraform console", which _does_ do a full evaluation graph walk to prepare its evaluation scope.)
-
Martin Atkins authored
The plan file writer requires a backend config to be present, but we don't really need one for the sake of _this_ test, since we don't activate the backend to render a plan graph, and so we just write in a placeholder.
-
Martin Atkins authored
Some underlying assumptions have shifted here, so although the behavior is still broadly the same we need to accommodate some different details.
-
Martin Atkins authored
This fixes TestConsole_tfvars.
-