geordi 2.6.0 → 2.7.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8c2c675109e049fa2e448060119880d0e9efbb42275747e4e7ddce0c205872d3
-  data.tar.gz: 55a314c7beee03d00fdd7499585929df02acf2ed2c3dccafba40e46470182627
+  metadata.gz: 17516fcb96c1a27828c5e43eea96d1ca0d8f72df42e467ff842a70a002b86459
+  data.tar.gz: 8985476313a0c6ed93e494878e388edd3ba72a8911c7785dfe53cc9f7e17c284
 SHA512:
-  metadata.gz: 66fc72c66158d5a6c6ed25e9ab41d3c6537fe479fd4d59f3ec61df338963ef139b15ed43d0deb59fe6423b109e0f050a86baa9e5a7b60bf8cfccfecf330450f4
-  data.tar.gz: 8ad6e9b7fd187d4e3668d84c5493b4bb7c6179b6d785ad9217f3e7d8dae3dbd0a700f795cd43d59d816e9379fde6ce972010c626f2bbc9307ecaf2e241cdc8b8
+  metadata.gz: afcfaace06ff3ee48b5165de088849c537d73dea2ac544807793f59902634e07b879ca6968c12e0df3deb375686ea377fe63fd531724dae7a2125028fa9a0189
+  data.tar.gz: 1dbac4ddff4d8c4b505756bd7ea36a2d667141268ebf90f28fd6faec32b6afb092f1f4dc32ff6442f13b07a5bc84398530bd695e1668e23ce42076752a0f7989

data/CHANGELOG.md

@@ -5,12 +5,20 @@ This project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.html
 
 
 ## Unreleased
-
 ### Compatible changes
-
 ### Breaking changes
 
 
+## Unreleased
+
+### Compatible changes
+- Fixed [#68](https://github.com/makandra/geordi/issues/68): The "cucumber" command now fails early when @solo features fail.
+- Added: The "setup" command now prints the db adapter when prompting db credentials.
+- Fixed [#71](https://github.com/makandra/geordi/issues/71): When used without staged changes, the "commit" command will print a warning and create an empty commit. Any arguments to the command are forwarded to Git.
+- Fixed: The "commit" command will not print the extra message any more.
+- Added: The "commit" command prints a (progress) loading message. The message is removed once loading is done.
+
+
 ## 2.6.0 2019-11-04
 
 ### Compatible changes

data/Gemfile

@@ -8,4 +8,5 @@ gem 'rspec-mocks'
 gem 'highline'
 gem 'parallel_tests'
 gem 'launchy'
+gem 'tracker_api'
 # gem 'pry' # Not supported for 1.8.7

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    geordi (2.6.0)
+    geordi (2.7.0)
       thor (>= 0.18.0)
 
 GEM
@@ -12,27 +12,50 @@ GEM
       childprocess (>= 0.3.6)
       cucumber (>= 1.1.1)
       rspec-expectations (>= 2.7.0)
+    axiom-types (0.1.1)
+      descendants_tracker (~> 0.0.4)
+      ice_nine (~> 0.11.0)
+      thread_safe (~> 0.3, >= 0.3.1)
     builder (3.2.3)
     childprocess (1.0.1)
       rake (< 13.0)
+    coercible (1.0.0)
+      descendants_tracker (~> 0.0.1)
     cucumber (1.3.20)
       builder (>= 2.1.2)
       diff-lcs (>= 1.1.3)
       gherkin (~> 2.12)
       multi_json (>= 1.7.5, < 2.0)
       multi_test (>= 0.1.2)
+    declarative (0.0.10)
+    declarative-option (0.1.0)
+    descendants_tracker (0.0.4)
+      thread_safe (~> 0.3, >= 0.3.1)
     diff-lcs (1.2.5)
+    equalizer (0.0.11)
+    excon (0.67.0)
+    faraday (0.17.0)
+      multipart-post (>= 1.2, < 3)
+    faraday_middleware (0.13.1)
+      faraday (>= 0.7.4, < 1.0)
     gherkin (2.12.2)
       multi_json (~> 1.3)
     highline (1.6.21)
+    ice_nine (0.11.2)
     launchy (2.4.3)
       addressable (~> 2.3)
+    mimemagic (0.3.3)
     multi_json (1.13.1)
     multi_test (0.1.2)
+    multipart-post (2.1.1)
     parallel (0.5.16)
     parallel_tests (0.6.18)
       parallel
     rake (10.5.0)
+    representable (3.0.4)
+      declarative (< 0.1.0)
+      declarative-option (< 0.2.0)
+      uber (< 0.2.0)
     rspec-expectations (3.4.0)
       diff-lcs (>= 1.2.0, < 2.0)
       rspec-support (~> 3.4.0)
@@ -41,6 +64,23 @@ GEM
       rspec-support (~> 3.4.0)
     rspec-support (3.4.1)
     thor (0.20.3)
+    thread_safe (0.3.6)
+    tracker_api (1.10.0)
+      addressable
+      equalizer
+      excon
+      faraday
+      faraday_middleware
+      mimemagic
+      multi_json
+      representable
+      virtus
+    uber (0.1.0)
+    virtus (1.0.5)
+      axiom-types (~> 0.1)
+      coercible (~> 1.0)
+      descendants_tracker (~> 0.0, >= 0.0.3)
+      equalizer (~> 0.0, >= 0.0.9)
 
 PLATFORMS
   ruby
@@ -52,6 +92,7 @@ DEPENDENCIES
   launchy
   parallel_tests
   rspec-mocks
+  tracker_api
 
 BUNDLED WITH
    1.17.3

data/features/commit.feature

@@ -0,0 +1,17 @@
+@announce-output
+Feature: Creating a git commit from a Pivotal Tracker story
+
+  Scenario: Extra arguments are forwarded to "git commit"
+    Given I have staged changes
+
+    When I run `geordi commit --extra-option` interactively
+      # No optional message
+      And I type ""
+    Then the output should contain "Util.system! git, commit, --allow-empty, -m, [#12] Test Story, --extra-option"
+
+
+  Scenario: With no staged changes, a warning is printed
+    When I run `geordi commit --allow-empty` interactively
+      # No optional message
+      And I type ""
+    Then the output should contain "> No staged changes. Will create an empty commit."

data/features/cucumber.feature

@@ -18,6 +18,7 @@ Feature: The cucumber command
     When I run `geordi cucumber features/single.feature`
     Then the output should contain "# Running features"
       And the output should contain "> Only: features/single.feature"
+      And the output should contain "Features green."
     But the output should not contain "parallel"
 
 
@@ -60,6 +61,7 @@ Feature: The cucumber command
       And the output should contain "Rerunning failed scenarios"
       And the output should contain "Using the rerun profile"
       And the exit status should be 1
+      And the output should contain "Features failed."
 
 
   Scenario: Running all features in a given subfolder
@@ -96,10 +98,10 @@ Feature: The cucumber command
     """
 
     When I run `geordi cucumber --verbose features`
-    Then the output should contain "# Running features"
-      And the output should match /^> .*cucumber .*--tags \"~@solo\"/
-      And the output should contain "# Running @solo features"
+    Then the output should contain "# Running @solo features"
       And the output should match /^> .*cucumber .*--tags @solo/
+      And the output should contain "# Running features"
+      And the output should match /^> .*cucumber .*--tags \"~@solo\"/
 
 
   Scenario: When there are no scenarios tagged @solo, the extra run is skipped
@@ -146,6 +148,28 @@ Feature: The cucumber command
       And the output should not contain "No such file or directory"
 
 
+  Scenario: It does not start the full test run when the @solo run fails
+    Given a file named "features/step_definitions/test_steps.rb" with:
+    """
+    Given 'this test fails' do
+      raise
+    end
+    """
+    And a file named "features/failing.feature" with:
+    """
+    Feature: Failing feature
+      @solo
+      Scenario: Failing scenario
+        And this test fails
+      Scenario: Other scenario
+    """
+
+    When I run `geordi cucumber`
+    Then the output should contain "# Running @solo features"
+      And the output should contain "Features failed."
+    But the output should not contain "# Running features"
+
+
   Scenario: Specifying a firefox version to use
     Given a file named "features/sub/one.feature" with:
     """

data/features/firefox.feature

@@ -6,7 +6,7 @@ Feature: The firefox/chrome command
     Given a file named "testfile" with "testcontent"
 
     When I run `geordi firefox cat testfile`
-    Then the output should contain "> VNC is ready to hold Selenium test browsers. Use `geordi vnc` to view them."
+    Then the output should contain "> Run `geordi vnc` to view the Selenium test browsers"
       And the output should contain "testcontent"
     But the output should not contain "Firefox for Selenium"
 
@@ -15,7 +15,7 @@ Feature: The firefox/chrome command
     Given a file named "testfile" with "testcontent"
 
     When I run `geordi chrome cat testfile`
-    Then the output should contain "> VNC is ready to hold Selenium test browsers. Use `geordi vnc` to view them."
+    Then the output should contain "> Run `geordi vnc` to view the Selenium test browsers"
       And the output should contain "testcontent"
 
 
@@ -48,7 +48,7 @@ Feature: The firefox/chrome command
       And a file named "testfile" with "testcontent"
 
     When I run `geordi firefox cat testfile`
-    Then the output should contain "> VNC is ready"
+    Then the output should contain "> Run `geordi vnc`"
       And the output should contain "testcontent"
     But the output should not contain "Firefox for Selenium"
 

data/features/support/step_definitions/miscellaneous_steps.rb

@@ -1,3 +1,11 @@
 When /^I wait for (\d+) seconds?$/ do |seconds|
   sleep seconds.to_i
 end
+
+Given 'I have staged changes' do
+  ENV['GEORDI_TESTING_STAGED_CHANGES'] = 'true'
+end
+
+After do
+  ENV['GEORDI_TESTING_STAGED_CHANGES'] = 'false'
+end

data/lib/geordi/commands/commit.rb

@@ -3,17 +3,22 @@ desc 'commit', 'Commit using a story title from Pivotal Tracker'
 long_desc <<-LONGDESC
 Example: `geordi commit`
 
+Any extra arguments are forwarded to `git commit -m <message>`.
+
+If there are no staged changes, prints a warning but will continue to create
+an empty commit.
+
 On the first execution we ask for your Pivotal Tracker API token. It will be
 stored in `~/.gitpt`.
 LONGDESC
 
-def commit
+def commit(*git_args)
   raise <<-TEXT if Gem::Version.new(RUBY_VERSION) < Gem::Version.new('2.1')
-Unsupported Ruby Version #{RUBY_VERSION}. `geordi commit` does not work with a Ruby version < 2.1.
+Unsupported Ruby Version #{RUBY_VERSION}. `geordi commit` requires Ruby 2.1+.
   TEXT
 
   require 'geordi/gitpt'
 
-  Gitpt.new.run
+  Gitpt.new.run(git_args)
 end
 

data/lib/geordi/commands/create_database_yml.rb

@@ -6,10 +6,12 @@ def create_database_yml
   if File.exists?(sample_yml) and not File.exists?(real_yml)
     announce 'Creating ' + real_yml
 
-    print 'Please enter your DB password: '
+    sample = File.read(sample_yml)
+    adapter = sample.match(/adapter: (\w+)\n/).captures.first
+
+    print "Please enter your #{adapter} password: "
     db_password = STDIN.gets.strip
 
-    sample = File.read(sample_yml)
     real = sample.gsub(/password:.*$/, "password: #{db_password}")
     File.open(real_yml, 'w') { |f| f.write(real) }
 

data/lib/geordi/commands/cucumber.rb

@@ -54,9 +54,7 @@ def cucumber(*args)
     cmd_opts, files = args.partition { |f| f.start_with? '-' }
     cmd_opts << '--format' << 'pretty' << '--backtrace' if options.debug
 
-    announce 'Running features'
-
-    # Serial run of @solo scenarios
+    # Serial run of @solo scenarios ############################################
     if files.any? { |f| f.include? ':' }
       note '@solo run skipped when called with line numbers' if options.verbose
     else
@@ -73,16 +71,20 @@ def cucumber(*args)
         solo_cmd_opts << '--tags' << '@solo'
 
         announce 'Running @solo features'
-        Geordi::Cucumber.new.run files, solo_cmd_opts, :verbose => options.verbose, :parallel => false
+        solo_success = Geordi::Cucumber.new.run files, solo_cmd_opts, :verbose => options.verbose, :parallel => false
+        solo_success or fail 'Features failed.'
       end
     end
 
-    # Normal run
-    unless Geordi::Cucumber.new.run(files, cmd_opts, :verbose => options.verbose)
+    # Parallel run of all given features + reruns ##############################
+    announce 'Running features'
+    normal_run_successful = Geordi::Cucumber.new.run(files, cmd_opts, :verbose => options.verbose)
+
+    unless normal_run_successful
       cmd_opts << '--profile' << 'rerun'
 
       # Reruns
-      (1 + options.rerun).times do |i|
+      (options.rerun + 1).times do |i|
         fail 'Features failed.' if (i == options.rerun) # All reruns done?
 
         announce "Rerun ##{ i + 1 } of #{ options.rerun }"
@@ -90,6 +92,8 @@ def cucumber(*args)
       end
     end
 
+    success 'Features green.'
+
   else
     note 'Cucumber not employed.'
   end

data/lib/geordi/commands/tests.rb

@@ -2,9 +2,9 @@ desc 'tests', 'Run all employed tests'
 def tests
   rake_result = invoke_cmd 'with_rake'
 
+  # Since `rake` usually is configured to run all tests, only run them if `rake`
+  # did not perform
   if rake_result == :did_not_perform
-    # Since `rake` usually runs all tests, only run them here if `rake` did not
-    # perform
     invoke_cmd 'unit'
     invoke_cmd 'rspec'
     invoke_cmd 'cucumber'

data/lib/geordi/cucumber.rb

@@ -60,7 +60,7 @@ module Geordi
         ENV["BROWSER"] = ENV["LAUNCHY_BROWSER"] = File.expand_path('../../../bin/launchy_browser', __FILE__)
         ENV["DISPLAY"] = VNC_DISPLAY
 
-        note 'VNC is ready to hold Selenium test browsers. Use `geordi vnc` to view them.'
+        note 'Run `geordi vnc` to view the Selenium test browsers'
       end
     end
 

data/lib/geordi/gitpt.rb

@@ -9,22 +9,23 @@ class Gitpt
 
   def initialize
     self.highline = HighLine.new
+    self.client = build_client(read_settings)
   end
 
-  def run
-    settings = read_settings
-    client = build_client(settings)
+  def run(git_args)
+    warn <<-WARNING if !Geordi::Util.staged_changes?
+No staged changes. Will create an empty commit.
+    WARNING
 
-    puts 'Connecting to Pivotal Tracker...'
-
-    projects = load_projects(client)
-    applicable_stories = load_applicable_stories(projects)
-    choose_story(client.me, applicable_stories)
+    story = choose_story
+    if story
+      create_commit "[##{story.id}] #{story.name}", *git_args
+    end
   end
 
   private
 
-  attr_accessor :highline
+  attr_accessor :highline, :client
 
   def read_settings
     file_path = File.join(ENV['HOME'], SETTINGS_FILE_NAME)
@@ -51,9 +52,8 @@ class Gitpt
     TrackerApi::Client.new(:token => settings.fetch(:token))
   end
 
-  def load_projects(client)
+  def load_projects
     project_ids = read_project_ids
-
     project_ids.collect { |project_id| client.project(project_id) }
   end
 
@@ -76,19 +76,32 @@ class Gitpt
     end
   end
 
-  def load_applicable_stories(projects)
-    projects.collect { |project| project.stories(:filter => 'state:started,finished,rejected') }.flatten
+  def applicable_stories
+    projects = load_projects
+    projects.collect do |project|
+      project.stories(:filter => 'state:started,finished,rejected')
+    end.flatten
   end
 
-  def choose_story(me, applicable_stories)
-    selected_story = nil
+  def choose_story
+    if Geordi::Util.testing?
+      return OpenStruct.new(:id => 12, :name => 'Test Story')
+    end
+
+    loading_message = 'Connecting to Pivotal Tracker ...'
+    print(loading_message)
+    stories = applicable_stories
+    reset_loading_message = "\r#{ ' ' * (loading_message.length + stories.length)}\r"
 
     highline.choose do |menu|
-      menu.header = "Choose a story"
-      applicable_stories.each do |story|
+      menu.header = 'Choose a story'
+
+      stories.each do |story|
+        print '.' # Progress
+
         state = story.current_state
         owners = story.owners
-        owner_is_me = owners.collect(&:id).include?(me.id)
+        owner_is_me = owners.collect(&:id).include?(client.me.id)
 
         if state == 'started'
           state = HighLine::GREEN + state + HighLine::RESET
@@ -100,22 +113,22 @@ class Gitpt
 
         label = "(#{owners.collect(&:name).join(', ')}, #{state}) #{story.name}"
         label = bold(label) if owner_is_me
-        menu.choice(label) { selected_story = story }
+
+        menu.choice(label) { return story }
       end
+
       menu.hidden ''
+      print reset_loading_message # Once menu is build
     end
 
-    if selected_story
-      message = highline.ask("\nAdd an optional message")
-      highline.say message
+    nil # Return nothing
+  end
 
-      commit_message = "[##{selected_story.id}] #{selected_story.name}"
-      if message.strip != ''
-        commit_message << ' - '<< message.strip
-      end
+  def create_commit(message, *git_args)
+    extra = highline.ask("\nAdd an optional message").strip
+    message << ' - ' << extra if (extra != "")
 
-      exec('git', 'commit', '-m', commit_message)
-    end
+    Geordi::Util.system! 'git', 'commit', '--allow-empty', '-m', message, *git_args
   end
 
   def bold(string)

data/lib/geordi/util.rb

@@ -80,6 +80,15 @@ module Geordi
         end
       end
 
+      def staged_changes?
+        if testing?
+          ENV['GEORDI_TESTING_STAGED_CHANGES'] == 'true'
+        else
+          statuses = `git status --porcelain`.split("\n")
+          statuses.any? { |l| l.start_with? 'A' }
+        end
+      end
+
       def deploy_targets
         Dir['config/deploy/*'].map do |f|
           File.basename f, '.rb' # Filename without .rb extension

data/lib/geordi/version.rb

@@ -1,3 +1,3 @@
 module Geordi
-  VERSION = '2.6.0'
+  VERSION = '2.7.0'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: geordi
 version: !ruby/object:Gem::Version
-  version: 2.6.0
+  version: 2.7.0
 platform: ruby
 authors:
 - Henning Koch
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-11-04 00:00:00.000000000 Z
+date: 2019-11-25 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: thor
@@ -77,6 +77,7 @@ files:
 - bin/run_tests
 - bin/shell-for
 - bin/tests
+- features/commit.feature
 - features/console.feature
 - features/cucumber.feature
 - features/deploy.feature