Skip to content
This repository was archived by the owner on Jul 14, 2021. It is now read-only.

Disable @spawn tests on berks to unblock chefdk releases. #9

Merged
merged 1 commit into from
Apr 11, 2014

Conversation

sersut
Copy link

@sersut sersut commented Apr 11, 2014

This fixes the spec errors similar to:

(::) failed steps (::)

expected "Successfully uninstalled ekaf (2.3.4)\n" to include "[fake (1.0.0)] depend on ekaf.\n\nAre you sure you want to continue? (y/N)"
Diff:
@@ -1,2 +1,2 @@
-[fake (1.0.0)] depend on ekaf.\n\nAre you sure you want to continue? (y/N)
+Successfully uninstalled ekaf (2.3.4)
 (RSpec::Expectations::ExpectationNotMetError)
features/commands/shelf/uninstall.feature:70:in `Then the output should contain:'

Failing Scenarios:
cucumber features/commands/shelf/uninstall.feature:63 # Scenario: With contingencies

We should enable this back when we can run tests with a tty.

…the spec errors similar to:

(::) failed steps (::)

expected "Successfully uninstalled ekaf (2.3.4)\n" to include "[fake (1.0.0)] depend on ekaf.\n\nAre you sure you want to continue? (y/N)"
Diff:
@@ -1,2 +1,2 @@
-[fake (1.0.0)] depend on ekaf.\n\nAre you sure you want to continue? (y/N)
+Successfully uninstalled ekaf (2.3.4)
 (RSpec::Expectations::ExpectationNotMetError)
features/commands/shelf/uninstall.feature:70:in `Then the output should contain:'

Failing Scenarios:
cucumber features/commands/shelf/uninstall.feature:63 # Scenario: With contingencies

We should enable this back when we can run tests with a tty.
@danielsdeleo
Copy link
Contributor

Makes sense. 👍

sersut pushed a commit that referenced this pull request Apr 11, 2014
Disable @Spawn tests on berks to unblock chefdk releases.
@sersut sersut merged commit 480251b into master Apr 11, 2014
@sersut sersut deleted the disable-berks-spawn-tests branch April 11, 2014 22:16
@chef-boneyard chef-boneyard locked and limited conversation to collaborators Feb 14, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants