Skip to content

Commit

Permalink
Merge pull request exercism#951 from exercism/revert-950-update-acronym
Browse files Browse the repository at this point in the history
Revert "acronym: Update and exclude new tests"
  • Loading branch information
kotp authored Mar 26, 2019
2 parents adf1909 + fb86d74 commit 2d426ab
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 17 deletions.
15 changes: 0 additions & 15 deletions exercises/acronym/.meta/generator/acronym_case.rb
Original file line number Diff line number Diff line change
Expand Up @@ -4,19 +4,4 @@ class AcronymCase < Generator::ExerciseCase
def workload
assert_equal(expected, "Acronym.abbreviate('#{phrase}')")
end

def to_s(*args)
super unless excluded_tests.include?(test_name)
end

private

# We exclude these tests because they currently don't fit the purpose
# we have for Acronym on the Ruby track.
def excluded_tests
%w(
test_apostrophes
test_underscore_emphasis
)
end
end
2 changes: 1 addition & 1 deletion exercises/acronym/acronym_test.rb
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
require 'minitest/autorun'
require_relative 'acronym'

# Common test data version: 1.7.0 cacf1f1
# Common test data version: 1.5.0 787d24e
class AcronymTest < Minitest::Test
def test_basic
# skip
Expand Down
2 changes: 1 addition & 1 deletion lib/generator/test_template.erb
Original file line number Diff line number Diff line change
Expand Up @@ -6,5 +6,5 @@ class <%= exercise_test_classname %> < Minitest::Test
<%=
test_cases.map.with_index do |test_case, index|
test_case.to_s(index.zero?)
end.compact.join("\n")
end.join("\n")
%>end

0 comments on commit 2d426ab

Please sign in to comment.