{"push_id":695044158,"size":1,"distinct_size":1,"ref":"refs/heads/fix-inflector-test","head":"e295f090e4cd52d8617b7a3d2b3d40909a90d75a","before":"d34839a43500e2cf6624b3295790495a39f8f337","commits":[{"sha":"e295f090e4cd52d8617b7a3d2b3d40909a90d75a","author":{"email":"0f1266fdb7aab22577884a834ca464c81c7dda2c@gmail.com","name":"Roque Pinel"},"message":"Fix inflector test by using dup inflections when it needs to be changed\n\nThis is not something that is failing at the moment, but can do it\neventually. I had the issue with db62081 as the HEAD and with the\nfollowing change:\n\n```\n--- a/activesupport/test/inflector_test.rb\n+++ b/activesupport/test/inflector_test.rb\n@@ -101,6 +101,7 @@ class InflectorTest < ActiveSupport::TestCase\n   def test_acronyms\n     ActiveSupport::Inflector.inflections do |inflect|\n       inflect.acronym(\"API\")\n+      inflect.acronym(\"HTM\")\n       inflect.acronym(\"HTML\")\n       inflect.acronym(\"HTTP\")\n       inflect.acronym(\"RESTful\")\n```\n\nI was expecting only `test_acronyms` to fail, but with a specific `seed`\nothers were also failing: `ruby -w -I\"lib:test\" test/inflector_test.rb\n--seed 4313`.\n\nNow, the tests that need to change the `inflections` are using a duplicate\nof the original `inflections`, just like it was already done for some\ntests.\n\nI decided to benchmark and check the impact of the patch and it seems\nto me to be fine.\n\n```\nCalculating -------------------------------------\n     without changes     1.000  i/100ms\n with additional dup     1.000  i/100ms\n-------------------------------------------------\n     without changes      0.800  (± 0.0%) i/s -      4.000\n with additional dup      0.792  (± 0.0%) i/s -      4.000\n```","distinct":true,"url":"https://api.github.com/repos/repinel/rails/commits/e295f090e4cd52d8617b7a3d2b3d40909a90d75a"}]}