Consider the multibyte value in the method name of system test

`String#[]` returns a value that considers multibyte value. But some
file systems use byte for maximum filename length. So if applications
use that file system and multibyte value to a method name, currently
check doesn't work expected.

This PR fixes to use `String#byteslice` instead of `String#[]`. Also,
added `String#scrub` to avoid generating an invalid byte sequence.

Author: y-yagi

lib/rspec/rails/example/system_example_group.rb

@@ -38,10 +38,12 @@ def passed?
 
       # @private
       def method_name
-        @method_name ||= [
-          self.class.name.underscore,
-          RSpec.current_example.description.underscore
-        ].join("_").tr(CHARS_TO_TRANSLATE.join, "_")[0...200] + "_#{rand(1000)}"
+        @method_name ||= begin
+          [
+            self.class.name.underscore,
+            RSpec.current_example.description.underscore
+          ].join("_").tr(CHARS_TO_TRANSLATE.join, "_").byteslice(0...200).scrub("") + "_#{rand(1000)}"
+        end
       end
 
       # Delegates to `Rails.application`.

spec/rspec/rails/example/system_example_group_spec.rb

@@ -16,6 +16,15 @@ module RSpec::Rails
             expect(example.send(:method_name)).to start_with('method_name')
           end
         end
+
+        it "slices long method name - #{'あ'*100}" do
+          group = RSpec::Core::ExampleGroup.describe do
+            include SystemExampleGroup
+          end
+          example = group.new
+          group.hooks.run(:before, :example, example)
+          expect(example.send(:method_name).bytesize).to be <= 210
+        end
       end
 
       describe '#driver' do