terraspace 0.3.3 → 0.4.1

data/spec/terraspace/dependency/helper/depends_on_spec.rb

@@ -0,0 +1,27 @@
+describe Terraspace::Dependency::Helper::DependsOn do
+  let(:depends_on) do
+    described_class.new(mod, identifier, options)
+  end
+  let (:mod) { Terraspace::Mod.new("a1") }
+  # follow args dont matter for spec
+  let (:identifier) { "b1" }
+  let (:options) { {} }
+
+  context "unresolved" do
+    before(:each) { mod.resolved = false }
+    it "result calls Marker::Output" do
+      allow(Terraspace::Terraform::RemoteState::Marker::Output).to receive(:new).and_return(double(:marker_output).as_null_object)
+      result = depends_on.result
+      expect(result).to be_a(RSpec::Mocks::Double)
+      expect(result.instance_variable_get(:@name)).to eq :marker_output
+    end
+  end
+
+  context "resolved" do
+    before(:each) { mod.resolved = true }
+    it "return an raw String" do
+      result = depends_on.result
+      expect(result).to eq "# a1 depends on b1"
+    end
+  end
+end

data/spec/terraspace/dependency/helper/output_spec.rb

@@ -0,0 +1,29 @@
+describe Terraspace::Dependency::Helper::Output do
+  let(:output) do
+    described_class.new(mod, identifier, options)
+  end
+  let (:mod) { Terraspace::Mod.new("foo") }
+  # doesnt matter for spec
+  let (:identifier) { nil }
+  let (:options) { nil }
+
+  context "unresolved" do
+    before(:each) { mod.resolved = false }
+    it "result calls Marker::Output" do
+      allow(Terraspace::Terraform::RemoteState::Marker::Output).to receive(:new).and_return(double(:marker_output).as_null_object)
+      result = output.result
+      expect(result).to be_a(RSpec::Mocks::Double)
+      expect(result.instance_variable_get(:@name)).to eq :marker_output
+    end
+  end
+
+  context "resolved" do
+    before(:each) { mod.resolved = true }
+    it "result calls Fetcher" do
+      allow(Terraspace::Terraform::RemoteState::Fetcher).to receive(:new).and_return(double(:fetcher_output).as_null_object)
+      result = output.result
+      expect(result).to be_a(RSpec::Mocks::Double)
+      expect(result.instance_variable_get(:@name)).to eq :fetcher_output
+    end
+  end
+end

data/spec/terraspace/{terraform/hooks → hooks}/builder_spec.rb

@@ -1,7 +1,6 @@
-describe Terraspace::Terraform::Hooks::Builder do
+describe Terraspace::Hooks::Builder do
   let(:builder) do
-    builder = described_class.new(mod, name)
-    builder.instance_variable_set(:@file, file) # override @file for spec
+    builder = described_class.new(mod, dsl_file, name)
     builder
   end
   let(:mod) do
@@ -11,7 +10,7 @@ describe Terraspace::Terraform::Hooks::Builder do
   end
 
   context "single" do
-    let(:file) { fixture("terraform/hooks/single.rb") }
+    let(:dsl_file) { fixture("terraform/hooks/single.rb") }
     let(:name) { "apply" }
     it "build creates the @hooks structure" do
       hooks = builder.build
@@ -25,7 +24,7 @@ describe Terraspace::Terraform::Hooks::Builder do
   end
 
   context "multiple" do
-    let(:file) { fixture("terraform/hooks/multiple.rb") }
+    let(:dsl_file) { fixture("terraform/hooks/multiple.rb") }
     let(:name) { "apply" }
     it "build creates the @hooks structure" do
       hooks = builder.build

data/spec/terraspace/terraform/remote_state/fetcher_spec.rb

@@ -11,41 +11,122 @@ describe Terraspace::Terraform::RemoteState::Fetcher do
     double("c1").as_null_object
   end
 
-  context "pull success" do
-    let(:state_path)   { "spec/fixtures/fetcher/c1.json" }
-    let(:pull_success) { true }
+  context "to_s uses to_json by default" do
+    context "pull success" do
+      let(:state_path)   { "spec/fixtures/fetcher/c1.json" }
+      let(:pull_success) { true }
 
-    context "c1.length" do
-      let(:identifier) { "c1.length" }
-      it "fetched found attribute" do
-        expect(fetcher.output.raw).to eq 1 # matches spec/fixtures/fetcher/c1.json outputs.length.value
+      context "c1.length" do
+        let(:identifier) { "c1.length" }
+        it "fetched found attribute" do
+          expect(fetcher.output.to_s).to eq '1' # Integer as json matches spec/fixtures/fetcher/c1.json outputs.length.value
+        end
+      end
+
+      context "c1.complex" do
+        let(:identifier) { "c1.complex" }
+        it "fetched found attribute" do
+          expect(fetcher.output.to_s).to eq ["a","b"].to_json # matches spec/fixtures/fetcher/c1.json outputs.complex.value
+        end
+      end
+
+      context "c1.does-not-exist" do
+        let(:identifier) { "c1.does-not-exist" }
+        it "fetched missing output" do
+          output_proxy = fetcher.output
+          expect(output_proxy.raw).to be nil
+          value = output_proxy.to_s
+          # (Output length9 was not found for the b1 tfvars file. Either c1 stack has not been deployed yet or it does not have this output: length9)
+          expect(value).to include "not found"
+        end
       end
     end
 
-    context "c1.does-not-exist" do
-      let(:identifier) { "c1.does-not-exist" }
-      it "fetched missing output" do
-        output_proxy = fetcher.output
-        expect(output_proxy.raw).to be nil
-        # (Output length9 was not found for the b1 tfvars file. Either c1 stack has not been deployed yet or it does not have this output: length9)
-        error = output_proxy.options[:error]
-        expect(error).to include "not found"
+    context "pull fail" do
+      let(:state_path)   { nil }
+      let(:pull_success) { false }
+
+      context "stack needs to be deployed" do
+        let(:identifier) { "c1.length" }
+        it "fetched" do
+          output_proxy = fetcher.output
+          expect(output_proxy.raw).to be nil
+          # (Output length could not be looked up for the b1 tfvars file. c1 stack needs to be deployed)
+          value = output_proxy.to_s
+          expect(value).to include "stack needs to be deployed"
+        end
+      end
+
+      context "bucket does not exist" do
+        let(:identifier) { "c1.length" }
+        it "fetched" do
+          fetcher.bucket_not_found_error # fake it!
+          output_proxy = fetcher.output
+          expect(output_proxy.raw).to be nil
+          # (Output length could not be looked up for the b1 tfvars file. c1 stack needs to be deployed)
+          value = output_proxy.to_s
+          expect(value).to include "bucket for the backend could not be found"
+        end
       end
     end
   end
 
-  context "pull fail" do
-    let(:state_path)   { nil }
-    let(:pull_success) { false }
-
-    context "c1.length" do
-      let(:identifier) { "c1.length" }
-      it "fetched" do
-        output_proxy = fetcher.output
-        expect(output_proxy.raw).to be nil
-        # (Output length could not be looked up for the b1 tfvars file. c1 stack needs to be deployed)
-        error = output_proxy.options[:error]
-        expect(error).to include "stack needs to be deployed"
+  context "to_ruby" do
+    context "pull success" do
+      let(:state_path)   { "spec/fixtures/fetcher/c1.json" }
+      let(:pull_success) { true }
+
+      context "c1.length" do
+        let(:identifier) { "c1.length" }
+        it "fetched found attribute" do
+          expect(fetcher.output.to_ruby).to eq 1 # raw Integer
+        end
+      end
+
+      context "c1.complex" do
+        let(:identifier) { "c1.complex" }
+        it "fetched found attribute" do
+          expect(fetcher.output.to_ruby).to eq ["a","b"] # raw Array
+        end
+      end
+
+      context "c1.does-not-exist" do
+        let(:identifier) { "c1.does-not-exist" }
+        it "fetched missing output" do
+          output_proxy = fetcher.output
+          expect(output_proxy.raw).to be nil
+          value = output_proxy.to_ruby
+          # (Output length9 was not found for the b1 tfvars file. Either c1 stack has not been deployed yet or it does not have this output: length9)
+          expect(value).to include "not found"
+        end
+      end
+    end
+
+    context "pull fail" do
+      let(:state_path)   { nil }
+      let(:pull_success) { false }
+
+      context "stack needs to be deployed" do
+        let(:identifier) { "c1.length" }
+        it "fetched" do
+          output_proxy = fetcher.output
+          expect(output_proxy.raw).to be nil
+          # (Output length could not be looked up for the b1 tfvars file. c1 stack needs to be deployed)
+          value = output_proxy.to_s
+          expect(value).to include "stack needs to be deployed"
+        end
+      end
+
+      context "bucket does not exist" do
+        let(:identifier) { "c1.length" }
+        it "fetched" do
+          fetcher.bucket_not_found_error # fake it!
+          output_proxy = fetcher.output
+          expect(output_proxy.raw).to be nil
+          # (Output length could not be looked up for the b1 tfvars file. c1 stack needs to be deployed)
+          value = output_proxy.to_s
+          expect(value).to include "bucket for the backend could not be found"
+        end
       end
     end
   end

data/spec/terraspace/terraform/remote_state/marker/output_spec.rb

@@ -0,0 +1,36 @@
+describe Terraspace::Terraform::RemoteState::Marker::Output do
+  let(:output) do
+    output = described_class.new(mod, identifier, options)
+    allow(output).to receive(:warning) # supress warning messaging about missing child stack
+    output
+  end
+  let (:mod)        { Terraspace::Mod.new("a1") }
+  let (:options)    { {} }
+
+  before(:each) do
+    Terraspace::Dependency::Registry.class_variable_set("@@data", Set.new)
+  end
+
+  # markers are always only called during unresolved stage
+  context "child stack found" do
+    let(:identifier) { "b1.length" }
+    it "registers dependency and always return a OutputProxy" do
+      allow(output).to receive(:valid?).and_return(true) # child stack found
+      result = output.build
+      expect(result).to be_a(Terraspace::Terraform::RemoteState::OutputProxy)
+      set = Terraspace::Dependency::Registry.data
+      expect(set).not_to be_empty
+    end
+  end
+
+  context "child stack not found" do
+    let(:identifier) { "b1.length" }
+    it "does not registers dependency and always return a OutputProxy" do
+      allow(output).to receive(:valid?).and_return(false) # child stack not found
+      result = output.build
+      expect(result).to be_a(Terraspace::Terraform::RemoteState::OutputProxy)
+      set = Terraspace::Dependency::Registry.data
+      expect(set).to be_empty
+    end
+  end
+end

data/spec/terraspace/terraform/remote_state/output_proxy_spec.rb

@@ -0,0 +1,69 @@
+Unresolved = Terraspace::Terraform::RemoteState::Unresolved
+
+describe Terraspace::Terraform::RemoteState::OutputProxy do
+  let(:proxy) do
+    described_class.new(mod, raw, options)
+  end
+  let (:mod) { Terraspace::Mod.new("foo") }
+  let (:raw) { nil }
+  let (:options) { {} }
+
+  context "unresolved" do
+    before(:each) { mod.resolved = false }
+    it "always return Unresolved" do
+      value = proxy.to_s
+      expect(value).to be_a(Unresolved)
+      expect(value.to_str).to eq "(unresolved)"
+    end
+  end
+
+  # Resolved value should always return a String because ERB requires string.
+  # Always use to_json.
+  context "resolved with mock" do
+    before(:each) { mod.resolved = true }
+    let (:raw) { nil }
+    let (:options) { {mock: "mock value"} }
+
+    it "return to_json String with mock value" do
+      value = proxy.to_s
+      expect(value).to be_a(String)
+      expect(value.to_str).to eq '"mock value"' # note double quotes from the to_json
+    end
+  end
+
+  context "resolved with errors" do
+    before(:each) { mod.resolved = true }
+    let (:raw) { nil }
+    let (:options) { {error: "error message"} }
+
+    it "return to_json String with error message" do
+      value = proxy.to_s
+      expect(value).to be_a(String)
+      expect(value.to_str).to eq '"error message"' # note double quotes from the to_json
+    end
+  end
+
+  context "to_ruby resolved with mock" do
+    before(:each) { mod.resolved = true }
+    let (:raw) { nil }
+    let (:options) { {mock: "mock value"} }
+
+    it "return to_json String with mock value" do
+      value = proxy.to_ruby
+      expect(value).to be_a(String)
+      expect(value.to_str).to eq "mock value"
+    end
+  end
+
+  context "to_ruby resolved with errors" do
+    before(:each) { mod.resolved = true }
+    let (:raw) { nil }
+    let (:options) { {error: "error message"} }
+
+    it "return to_json String with error message" do
+      value = proxy.to_ruby
+      expect(value).to be_a(String)
+      expect(value.to_str).to eq "error message"
+    end
+  end
+end

data/terraspace.gemspec

@@ -28,7 +28,7 @@ Gem::Specification.new do |spec|
   spec.add_dependency "memoist"
   spec.add_dependency "rainbow"
   spec.add_dependency "render_me_pretty"
-  spec.add_dependency "terraspace-bundler", "~> 0.2.0"
+  spec.add_dependency "terraspace-bundler", "~> 0.3.0"
   spec.add_dependency "thor"
   spec.add_dependency "tty-tree"
   spec.add_dependency "zeitwerk"

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: terraspace
 version: !ruby/object:Gem::Version
-  version: 0.3.3
+  version: 0.4.1
 platform: ruby
 authors:
 - Tung Nguyen
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2020-09-19 00:00:00.000000000 Z
+date: 2020-10-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -156,14 +156,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.2.0
+        version: 0.3.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.2.0
+        version: 0.3.0
 - !ruby/object:Gem::Dependency
   name: thor
   requirement: !ruby/object:Gem::Requirement
@@ -470,26 +470,71 @@ files:
 - lib/terraspace/cli/bundle.rb
 - lib/terraspace/cli/check_setup.rb
 - lib/terraspace/cli/clean.rb
+- lib/terraspace/cli/clean/all.rb
+- lib/terraspace/cli/clean/base.rb
+- lib/terraspace/cli/clean/cache.rb
+- lib/terraspace/cli/clean/logs.rb
 - lib/terraspace/cli/cloud.rb
 - lib/terraspace/cli/cloud/runs.rb
 - lib/terraspace/cli/commander.rb
 - lib/terraspace/cli/down.rb
 - lib/terraspace/cli/help.rb
+- lib/terraspace/cli/help/all/down.md
+- lib/terraspace/cli/help/all/graph.md
+- lib/terraspace/cli/help/all/init.md
+- lib/terraspace/cli/help/all/output.md
+- lib/terraspace/cli/help/all/plan.md
+- lib/terraspace/cli/help/all/providers.md
+- lib/terraspace/cli/help/all/refresh.md
+- lib/terraspace/cli/help/all/show.md
+- lib/terraspace/cli/help/all/up.md
+- lib/terraspace/cli/help/all/validate.md
+- lib/terraspace/cli/help/build.md
 - lib/terraspace/cli/help/bundle.md
+- lib/terraspace/cli/help/check_setup.md
+- lib/terraspace/cli/help/clean/all.md
+- lib/terraspace/cli/help/clean/cache.md
+- lib/terraspace/cli/help/clean/logs.md
+- lib/terraspace/cli/help/cloud/destroy.md
+- lib/terraspace/cli/help/cloud/list.md
 - lib/terraspace/cli/help/cloud/runs/list.md
 - lib/terraspace/cli/help/cloud/runs/prune.md
 - lib/terraspace/cli/help/cloud/sync.md
 - lib/terraspace/cli/help/completion.md
 - lib/terraspace/cli/help/completion_script.md
-- lib/terraspace/cli/help/log.md
-- lib/terraspace/cli/help/update.md
+- lib/terraspace/cli/help/console.md
+- lib/terraspace/cli/help/down.md
+- lib/terraspace/cli/help/info.md
+- lib/terraspace/cli/help/init.md
+- lib/terraspace/cli/help/list.md
+- lib/terraspace/cli/help/logs.md
+- lib/terraspace/cli/help/logs/remove.md
+- lib/terraspace/cli/help/logs/truncate.md
+- lib/terraspace/cli/help/new/bootstrap_test.md
+- lib/terraspace/cli/help/new/example.md
+- lib/terraspace/cli/help/new/git_hook.md
+- lib/terraspace/cli/help/new/module.md
+- lib/terraspace/cli/help/new/module_test.md
+- lib/terraspace/cli/help/new/plugin.md
+- lib/terraspace/cli/help/new/project.md
+- lib/terraspace/cli/help/new/project_test.md
+- lib/terraspace/cli/help/new/shim.md
+- lib/terraspace/cli/help/new/stack.md
+- lib/terraspace/cli/help/output.md
+- lib/terraspace/cli/help/plan.md
+- lib/terraspace/cli/help/providers.md
+- lib/terraspace/cli/help/refresh.md
+- lib/terraspace/cli/help/seed.md
+- lib/terraspace/cli/help/show.md
+- lib/terraspace/cli/help/summary.md
+- lib/terraspace/cli/help/test.md
+- lib/terraspace/cli/help/up.md
+- lib/terraspace/cli/help/validate.md
 - lib/terraspace/cli/info.rb
 - lib/terraspace/cli/init.rb
 - lib/terraspace/cli/list.rb
-- lib/terraspace/cli/log.rb
-- lib/terraspace/cli/log/concern.rb
 - lib/terraspace/cli/logs.rb
-- lib/terraspace/cli/logs/tasks.rb
+- lib/terraspace/cli/logs/concern.rb
 - lib/terraspace/cli/new.rb
 - lib/terraspace/cli/new/git_hook.rb
 - lib/terraspace/cli/new/helper.rb
@@ -521,6 +566,7 @@ files:
 - lib/terraspace/compiler/cleaner.rb
 - lib/terraspace/compiler/cleaner/backend_change.rb
 - lib/terraspace/compiler/commands_concern.rb
+- lib/terraspace/compiler/dependencies/helpers.rb
 - lib/terraspace/compiler/dirs_concern.rb
 - lib/terraspace/compiler/dsl/base.rb
 - lib/terraspace/compiler/dsl/meta/local.rb
@@ -541,6 +587,7 @@ files:
 - lib/terraspace/compiler/dsl/syntax/tfvar/common.rb
 - lib/terraspace/compiler/dsl/tfvars.rb
 - lib/terraspace/compiler/erb/context.rb
+- lib/terraspace/compiler/erb/helpers.rb
 - lib/terraspace/compiler/erb/render.rb
 - lib/terraspace/compiler/expander.rb
 - lib/terraspace/compiler/strategy/abstract_base.rb
@@ -560,11 +607,18 @@ files:
 - lib/terraspace/completer/script.sh
 - lib/terraspace/core.rb
 - lib/terraspace/dependency/graph.rb
+- lib/terraspace/dependency/helper/base.rb
+- lib/terraspace/dependency/helper/depends_on.rb
+- lib/terraspace/dependency/helper/output.rb
 - lib/terraspace/dependency/node.rb
 - lib/terraspace/dependency/registry.rb
 - lib/terraspace/ext.rb
 - lib/terraspace/ext/bundler.rb
 - lib/terraspace/ext/core/module.rb
+- lib/terraspace/hooks/builder.rb
+- lib/terraspace/hooks/concern.rb
+- lib/terraspace/hooks/dsl.rb
+- lib/terraspace/hooks/runner.rb
 - lib/terraspace/logger.rb
 - lib/terraspace/logger/formatter.rb
 - lib/terraspace/mod.rb
@@ -610,12 +664,11 @@ files:
 - lib/terraspace/terraform/cloud/sync.rb
 - lib/terraspace/terraform/cloud/syncer.rb
 - lib/terraspace/terraform/cloud/workspace.rb
-- lib/terraspace/terraform/hooks/builder.rb
-- lib/terraspace/terraform/hooks/dsl.rb
 - lib/terraspace/terraform/remote_state/fetcher.rb
 - lib/terraspace/terraform/remote_state/marker/output.rb
 - lib/terraspace/terraform/remote_state/marker/pretty_tracer.rb
 - lib/terraspace/terraform/remote_state/output_proxy.rb
+- lib/terraspace/terraform/remote_state/unresolved.rb
 - lib/terraspace/terraform/runner.rb
 - lib/terraspace/tester.rb
 - lib/terraspace/tester/finder.rb
@@ -627,6 +680,7 @@ files:
 - lib/terraspace/version.rb
 - spec/cli_spec.rb
 - spec/fixtures/cache_dir/variables.tf
+- spec/fixtures/dependencies/app/stacks/a1/tfvars/dev.tfvars
 - spec/fixtures/fetcher/c1.json
 - spec/fixtures/initialized/modules.json
 - spec/fixtures/orphans/config/backend.rb
@@ -740,13 +794,18 @@ files:
 - spec/terraspace/compiler/dsl/resource_spec.rb
 - spec/terraspace/compiler/dsl/terraform_spec.rb
 - spec/terraspace/compiler/dsl/variable_spec.rb
+- spec/terraspace/compiler/erb/render_spec.rb
 - spec/terraspace/dependency/graph_spec.rb
+- spec/terraspace/dependency/helper/depends_on_spec.rb
+- spec/terraspace/dependency/helper/output_spec.rb
+- spec/terraspace/hooks/builder_spec.rb
 - spec/terraspace/provider/expander/generic_spec.rb
 - spec/terraspace/seeder/content_spec.rb
 - spec/terraspace/seeder_spec.rb
 - spec/terraspace/terraform/args/custom_spec.rb
-- spec/terraspace/terraform/hooks/builder_spec.rb
 - spec/terraspace/terraform/remote_state/fetcher_spec.rb
+- spec/terraspace/terraform/remote_state/marker/output_spec.rb
+- spec/terraspace/terraform/remote_state/output_proxy_spec.rb
 - terraspace.gemspec
 homepage: https://github.com/boltops-tools/terraspace
 licenses:
@@ -774,6 +833,7 @@ summary: 'Terraspace: The Terraspace Framework'
 test_files:
 - spec/cli_spec.rb
 - spec/fixtures/cache_dir/variables.tf
+- spec/fixtures/dependencies/app/stacks/a1/tfvars/dev.tfvars
 - spec/fixtures/fetcher/c1.json
 - spec/fixtures/initialized/modules.json
 - spec/fixtures/orphans/config/backend.rb
@@ -887,10 +947,15 @@ test_files:
 - spec/terraspace/compiler/dsl/resource_spec.rb
 - spec/terraspace/compiler/dsl/terraform_spec.rb
 - spec/terraspace/compiler/dsl/variable_spec.rb
+- spec/terraspace/compiler/erb/render_spec.rb
 - spec/terraspace/dependency/graph_spec.rb
+- spec/terraspace/dependency/helper/depends_on_spec.rb
+- spec/terraspace/dependency/helper/output_spec.rb
+- spec/terraspace/hooks/builder_spec.rb
 - spec/terraspace/provider/expander/generic_spec.rb
 - spec/terraspace/seeder/content_spec.rb
 - spec/terraspace/seeder_spec.rb
 - spec/terraspace/terraform/args/custom_spec.rb
-- spec/terraspace/terraform/hooks/builder_spec.rb
 - spec/terraspace/terraform/remote_state/fetcher_spec.rb
+- spec/terraspace/terraform/remote_state/marker/output_spec.rb
+- spec/terraspace/terraform/remote_state/output_proxy_spec.rb