chef-cli 5.0.1 → 5.3.1

data/spec/unit/command/generator_commands/cookbook_spec.rb

@@ -189,201 +189,6 @@ describe ChefCLI::Command::GeneratorCommands::Cookbook do
       end
     end
 
-    context "when passed workflow option" do
-
-      context "generates a workflow (delivery) cookbook" do
-
-        let(:argv) { %w{new_cookbook --workflow} }
-
-        let(:dot_delivery) { File.join(tempdir, "new_cookbook", ".delivery") }
-
-        before do
-          Dir.chdir(tempdir) do
-            allow(cookbook_generator.chef_runner).to receive(:stdout).and_return(stdout_io)
-            expect(cookbook_generator.run).to eq(0)
-          end
-        end
-
-        describe "when passed without --specs subcommand .delivery/project.toml" do
-
-          let(:file) { File.join(tempdir, "new_cookbook", ".delivery", "project.toml") }
-
-          let(:expected_content) do
-            <<~PROJECT_DOT_TOML
-              # Delivery for Local Phases Execution
-              #
-              # This file allows you to execute test phases locally on a workstation or
-              # in a CI pipeline. The delivery-cli will read this file and execute the
-              # command(s) that are configured for each phase. You can customize them
-              # by just modifying the phase key on this file.
-              #
-              # By default these phases are configured for Cookbook Workflow only
-              #
-
-              [local_phases]
-              unit = "echo skipping unit phase."
-              lint = "chef exec cookstyle"
-              # foodcritic has been deprecated in favor of cookstyle so we skip the syntax
-              # phase now.
-              syntax = "echo skipping syntax phase. Use lint phase instead."
-              provision = "chef exec kitchen create"
-              deploy = "chef exec kitchen converge"
-              smoke = "chef exec kitchen verify"
-              # The functional phase is optional, you can define it by uncommenting
-              # the line below and running the command: `delivery local functional`
-              # functional = ""
-              cleanup = "chef exec kitchen destroy"
-
-              # Remote project.toml file
-              #
-              # Instead of the local phases above, you may specify a remote URI location for
-              # the `project.toml` file. This is useful for teams that wish to centrally
-              # manage the behavior of the `delivery local` command across many different
-              # projects.
-              #
-              # remote_file = "https://url/project.toml"
-            PROJECT_DOT_TOML
-          end
-
-          it "exists with default config for Cookbook Workflow" do
-            expect(IO.read(file)).to eq(expected_content)
-          end
-
-        end
-
-        describe "when passed with --specs subcommand .delivery/project.toml" do
-
-          let(:argv) { %w{new_cookbook --workflow --specs} }
-
-          let(:file) { File.join(tempdir, "new_cookbook", ".delivery", "project.toml") }
-
-          let(:expected_content) do
-            <<~PROJECT_DOT_TOML
-              # Delivery for Local Phases Execution
-              #
-              # This file allows you to execute test phases locally on a workstation or
-              # in a CI pipeline. The delivery-cli will read this file and execute the
-              # command(s) that are configured for each phase. You can customize them
-              # by just modifying the phase key on this file.
-              #
-              # By default these phases are configured for Cookbook Workflow only
-              #
-
-              [local_phases]
-              unit = "chef exec rspec spec/"
-              lint = "chef exec cookstyle"
-              # foodcritic has been deprecated in favor of cookstyle so we skip the syntax
-              # phase now.
-              syntax = "echo skipping syntax phase. Use lint phase instead."
-              provision = "chef exec kitchen create"
-              deploy = "chef exec kitchen converge"
-              smoke = "chef exec kitchen verify"
-              # The functional phase is optional, you can define it by uncommenting
-              # the line below and running the command: `delivery local functional`
-              # functional = ""
-              cleanup = "chef exec kitchen destroy"
-
-              # Remote project.toml file
-              #
-              # Instead of the local phases above, you may specify a remote URI location for
-              # the `project.toml` file. This is useful for teams that wish to centrally
-              # manage the behavior of the `delivery local` command across many different
-              # projects.
-              #
-              # remote_file = "https://url/project.toml"
-            PROJECT_DOT_TOML
-          end
-
-          it "exists with default config for Cookbook Workflow" do
-            expect(IO.read(file)).to eq(expected_content)
-          end
-
-        end
-
-      end
-
-      context "when no delivery CLI configuration is present" do
-
-        let(:argv) { %w{new_cookbook --workflow} }
-
-        it "detects no delivery config" do
-          Dir.chdir(tempdir) do
-            expect(cookbook_generator.have_delivery_config?).to be(false)
-          end
-        end
-
-        it "emits concise output" do
-          Dir.chdir(tempdir) do
-            allow(cookbook_generator.chef_runner).to receive(:stdout).and_return(stdout_io)
-            expect(cookbook_generator.run).to eq(0)
-          end
-
-          expected = <<~OUTPUT
-            Generating cookbook new_cookbook
-            - Ensuring correct cookbook content
-            - Committing cookbook files to git
-
-            #{non_delivery_breadcrumb}
-          OUTPUT
-
-          actual = stdout_io.string
-
-          # the formatter will add escape sequences to turn off any colors
-          actual.gsub!("\e[0m", "")
-          expect(actual).to eq(expected)
-        end
-      end
-
-      context "when a delivery CLI config is present" do
-
-        # Setup a situation like this:
-        # there is a dir for the delivery organization with the
-        # `.delivery/cli.toml` in it. Inside that is another dir (maybe IRL this
-        # would be "cookbooks"), then we create the cookbook inside that.
-
-        let(:argv) { %w{new_cookbook --workflow} }
-
-        let(:tempdir_subdir) { File.join(tempdir, "subdirectory") }
-
-        let(:dot_delivery_dir) { File.join(tempdir, ".delivery") }
-
-        let(:dot_delivery_cli_toml) { File.join(dot_delivery_dir, "cli.toml") }
-
-        before do
-          Dir.mkdir(tempdir_subdir)
-          Dir.mkdir(dot_delivery_dir)
-          FileUtils.touch(dot_delivery_cli_toml)
-        end
-
-        it "detects the delivery config" do
-          Dir.chdir(tempdir_subdir) do
-            expect(cookbook_generator.have_delivery_config?).to be(true)
-          end
-        end
-
-        it "emits concise output" do
-          Dir.chdir(tempdir) do
-            allow(cookbook_generator.chef_runner).to receive(:stdout).and_return(stdout_io)
-            expect(cookbook_generator.run).to eq(0)
-          end
-
-          expected = <<~OUTPUT
-            Generating cookbook new_cookbook
-            - Ensuring correct cookbook content
-            - Committing cookbook files to git
-
-            Your cookbook is ready. To setup the pipeline, type `cd new_cookbook`, then run `delivery init`
-          OUTPUT
-
-          actual = stdout_io.string
-
-          # the formatter will add escape sequences to turn off any colors
-          actual.gsub!("\e[0m", "")
-          expect(actual).to eq(expected)
-        end
-      end
-    end
-
     context "when given the specs flag" do
 
       let(:argv) { %w{ new_cookbook --specs } }
@@ -615,6 +420,55 @@ describe ChefCLI::Command::GeneratorCommands::Cookbook do
 
     end
 
+    context "when YAML recipe flag is passed" do
+
+      let(:argv) { %w{new_cookbook --yaml} }
+
+      describe "recipes/default.yml" do
+        let(:file) { File.join(tempdir, "new_cookbook", "recipes", "default.yml") }
+
+        let(:expected_content_header) do
+          <<~DEFAULT_YML_HEADER
+          #
+          # Cookbook:: new_cookbook
+          # Recipe:: default
+          #
+          DEFAULT_YML_HEADER
+        end
+
+        let(:expected_content) do
+          <<~DEFAULT_YML_CONTENT
+          ---
+          resources:
+          # Example Syntax
+          # Additional snippets are available using the Chef Infra Extension for Visual Studio Code
+          # - type: file
+          #   name: '/path/to/file'
+          #   content: 'content'
+          #   owner: 'root'
+          #   group: 'root'
+          #   mode: '0755'
+          #   action:
+          #     - create
+          DEFAULT_YML_CONTENT
+        end
+
+        before do
+          Dir.chdir(tempdir) do
+            allow(cookbook_generator.chef_runner).to receive(:stdout).and_return(stdout_io)
+            expect(cookbook_generator.run).to eq(0)
+          end
+        end
+
+        it "has a default.yml file with template contents" do
+          expect(IO.read(file)).to match(expected_content_header)
+          expect(IO.read(file)).to match(expected_content)
+        end
+
+      end
+
+    end
+
     context "when configured for Berkshelf" do
 
       let(:argv) { %w{new_cookbook --berks} }

data/spec/unit/command/generator_commands/recipe_spec.rb

@@ -35,4 +35,38 @@ describe ChefCLI::Command::GeneratorCommands::Recipe do
 
   end
 
+  context "when YAML recipe flag is passed" do
+
+    let(:argv) { %w{some_recipe --yaml} }
+    let(:expected_cookbook_root) { tempdir }
+    let(:cookbook_name) { "example_cookbook" }
+    let(:cookbook_path) { File.join(tempdir, cookbook_name) }
+
+    let(:generator_name) { "recipe" }
+    let(:generated_files) do
+      [ "recipes/some_recipe.yml",
+                              "spec/spec_helper.rb",
+                              "spec/unit/recipes/some_recipe_spec.rb",
+                              "test/integration/default/some_recipe_test.rb",
+                            ]
+    end
+    let(:new_file_name) { "some_recipe" }
+
+    before do
+      FileUtils.cp_r(File.join(fixtures_path, "example_cookbook"), tempdir)
+    end
+
+    it "creates a new recipe" do
+      Dir.chdir(cookbook_path) do
+        allow(recipe_generator.chef_runner).to receive(:stdout).and_return(stdout_io)
+        recipe_generator.run
+      end
+
+      generated_files.each do |expected_file|
+        expect(File).to exist(File.join(cookbook_path, expected_file))
+      end
+    end
+
+  end
+
 end

data/spec/unit/cookbook_metadata_spec.rb

@@ -1,5 +1,5 @@
 #
-# Copyright:: Copyright (c) 2014-2018 Chef Software Inc.
+# Copyright:: Copyright (c) Chef Software Inc.
 # License:: Apache License, Version 2.0
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
@@ -80,6 +80,25 @@ describe ChefCLI::CookbookMetadata do
 
     end
 
+    context "and the cookbook has both a metadata.json and an (invalid) metadata.rb" do
+
+      let(:cookbook_root) { File.join(fixtures_path, "example_cookbook_both_metadata") }
+
+      it "has a name" do
+        expect(cookbook.name).to eq("example_cookbook")
+        expect(cookbook.cookbook_name).to eq("example_cookbook")
+      end
+
+      it "has a version" do
+        expect(cookbook.version).to eq("0.1.0")
+      end
+
+      it "has a map of dependencies" do
+        expect(cookbook.dependencies).to eq({})
+      end
+
+    end
+
     context "and the cookbook has no metadata" do
 
       let(:cookbook_root) { File.join(fixtures_path, "example_cookbook_no_metadata") }

data/spec/unit/fixtures/example_cookbook_both_metadata/.gitignore

@@ -0,0 +1,17 @@
+.vagrant
+Berksfile.lock
+*~
+*#
+.#*
+\#*#
+.*.sw[a-z]
+*.un~
+/cookbooks
+
+# Bundler
+Gemfile.lock
+bin/*
+.bundle/*
+
+.kitchen/
+.kitchen.local.yml

data/spec/unit/fixtures/example_cookbook_both_metadata/.kitchen.yml

@@ -0,0 +1,16 @@
+---
+driver:
+  name: vagrant
+
+provisioner:
+  name: chef_solo
+
+platforms:
+  - name: ubuntu-12.04
+  - name: centos-6.4
+
+suites:
+  - name: default
+    run_list:
+      - recipe[bar::default]
+    attributes:

data/spec/unit/fixtures/example_cookbook_both_metadata/Berksfile

@@ -0,0 +1,3 @@
+source "https://supermarket.getchef.com"
+
+metadata

data/spec/unit/fixtures/example_cookbook_both_metadata/README.md

@@ -0,0 +1,4 @@
+# bar-cookbook
+
+TODO: Enter the cookbook description here.
+

data/spec/unit/fixtures/example_cookbook_both_metadata/chefignore

@@ -0,0 +1,96 @@
+# Put files/directories that should be ignored in this file when uploading
+# or sharing to the community site.
+# Lines that start with '# ' are comments.
+
+# OS generated files #
+######################
+.DS_Store
+Icon?
+nohup.out
+ehthumbs.db
+Thumbs.db
+
+# SASS #
+########
+.sass-cache
+
+# EDITORS #
+###########
+\#*
+.#*
+*~
+*.sw[a-z]
+*.bak
+REVISION
+TAGS*
+tmtags
+*_flymake.*
+*_flymake
+*.tmproj
+.project
+.settings
+mkmf.log
+
+## COMPILED ##
+##############
+a.out
+*.o
+*.pyc
+*.so
+*.com
+*.class
+*.dll
+*.exe
+*/rdoc/
+
+# Testing #
+###########
+.watchr
+.rspec
+spec/*
+spec/fixtures/*
+test/*
+features/*
+Guardfile
+Procfile
+
+# SCM #
+#######
+.git
+*/.git
+.gitignore
+.gitmodules
+.gitconfig
+.gitattributes
+.svn
+*/.bzr/*
+*/.hg/*
+*/.svn/*
+
+# Berkshelf #
+#############
+Berksfile
+Berksfile.lock
+cookbooks/*
+tmp
+
+# Cookbooks #
+#############
+CONTRIBUTING
+CHANGELOG*
+
+# Strainer #
+############
+Colanderfile
+Strainerfile
+.colander
+.strainer
+
+# Vagrant #
+###########
+.vagrant
+Vagrantfile
+
+# Travis #
+##########
+.travis.yml