Merge pull request soutaro#1364 from soutaro/symbolize-names

Symbolize target/group names

lib/steep/project/dsl.rb

@@ -164,6 +164,7 @@ def code_diagnostics_config
         end
 
         def group(name, &block)
+          name = name.to_str.to_sym unless Symbol === name
           group = GroupDSL.new(name, self)
 
           Steep.logger.tagged "group=#{name}" do
@@ -230,6 +231,7 @@ def self.eval(project, &block)
       end
 
       def target(name, &block)
+        name = name.to_str.to_sym unless Symbol === name
         dsl = TargetDSL.new(name, project: project)
 
         Steep.logger.tagged "target=#{name}" do

sig/steep/project/dsl.rbs

@@ -133,7 +133,7 @@ module Steep
         def configure_code_diagnostics: (?diagnostics_config hash) { (diagnostics_config) -> void } -> void
                                       | (diagnostics_config config) -> void
 
-        def group: (Symbol) ?{ (GroupDSL) [self: GroupDSL] -> void } -> void
+        def group: (interned) ?{ (GroupDSL) [self: GroupDSL] -> void } -> void
       end
 
       class GroupDSL
@@ -174,7 +174,7 @@ module Steep
         # ```rb
         # configure_code_diagnostics(nil)
         # ```
-        # 
+        #
         def configure_code_diagnostics: (?diagnostics_config hash) { (diagnostics_config) -> void } -> void
                                       | (diagnostics_config? config) -> void
       end
@@ -189,7 +189,7 @@ module Steep
 
       include LibraryOptions
 
-      def target: (Symbol name) ?{ (TargetDSL) [self: TargetDSL] -> void } -> void
+      def target: (interned name) ?{ (TargetDSL) [self: TargetDSL] -> void } -> void
     end
   end
 end

test/steepfile_test.rb

@@ -319,4 +319,21 @@ def test_group
       end
     end
   end
+
+  def test_string
+    in_tmpdir do
+      project = Project.new(steepfile_path: current_dir + "Steepfile")
+
+      Project::DSL.eval(project) do
+        target "app" do
+          group "core" do
+          end
+        end
+      end
+
+      project.targets.find {|target| target.name == :app }.tap do |target|
+        assert_equal [:core], target.groups.map(&:name)
+      end
+    end
+  end
 end