From 3614ccc7635ad1fca313422a3a2ee58b273a5f7f Mon Sep 17 00:00:00 2001
From: sue445 <sue445@sue445.net>
Date: Wed, 20 Oct 2021 16:29:38 +0900
Subject: [PATCH] Fixed. NPlusOneQuery duplicate offense

---
 .../cop/isucon/mysql2/n_plus_one_query_spec.rb     | 14 --------------
 1 file changed, 14 deletions(-)

diff --git a/spec/rubocop/cop/isucon/mysql2/n_plus_one_query_spec.rb b/spec/rubocop/cop/isucon/mysql2/n_plus_one_query_spec.rb
index 64c45ff6..615f7af9 100644
--- a/spec/rubocop/cop/isucon/mysql2/n_plus_one_query_spec.rb
+++ b/spec/rubocop/cop/isucon/mysql2/n_plus_one_query_spec.rb
@@ -122,7 +122,6 @@
   describe "#perform_autocorrect" do
     context "Hash#[] with symbol key" do
       it "registers an offense and correct" do
-        # FIXME: duplicate offense messages
         # c.f. https://github.com/isucon/isucon11-final/blob/667be3ec70c025eadde541e21d5ab1167efa1dd3/webapp/ruby/app.rb#L171-L190
         expect_offense(<<~RUBY)
           courses = db.xquery(
@@ -135,7 +134,6 @@
 
           courses.map do |course|
             teacher = db.xquery('SELECT * FROM `users` WHERE `id` = ?', course[:teacher_id]).first
-                      ^^ This looks like N+1 query.
                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ This looks like N+1 query.
             raise unless teacher
 
@@ -177,7 +175,6 @@
 
     context "Hash#[] with string key" do
       it "registers an offense and correct" do
-        # FIXME: duplicate offense messages
         # c.f. https://github.com/isucon/isucon11-final/blob/667be3ec70c025eadde541e21d5ab1167efa1dd3/webapp/ruby/app.rb#L171-L190
         expect_offense(<<~RUBY)
           courses = db.xquery(
@@ -190,7 +187,6 @@
 
           courses.map do |course|
             teacher = db.xquery('SELECT * FROM `users` WHERE `id` = ?', course["teacher_id"]).first
-                      ^^ This looks like N+1 query.
                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ This looks like N+1 query.
             raise unless teacher
 
@@ -232,7 +228,6 @@
 
     context "Hash#fetch with symbol key" do
       it "registers an offense and correct" do
-        # FIXME: duplicate offense messages
         expect_offense(<<~RUBY)
           courses = db.xquery(
             "SELECT `courses`.*" \\
@@ -244,7 +239,6 @@
 
           courses.map do |course|
             teacher = db.xquery('SELECT * FROM `users` WHERE `id` = ?', course.fetch(:teacher_id)).first
-                      ^^ This looks like N+1 query.
                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ This looks like N+1 query.
             raise unless teacher
 
@@ -286,7 +280,6 @@
 
     context "Hash#fetch with string key" do
       it "registers an offense and correct" do
-        # FIXME: duplicate offense messages
         # c.f. https://github.com/isucon/isucon11-final/blob/667be3ec70c025eadde541e21d5ab1167efa1dd3/webapp/ruby/app.rb#L171-L190
         expect_offense(<<~RUBY)
           courses = db.xquery(
@@ -299,7 +292,6 @@
 
           courses.map do |course|
             teacher = db.xquery('SELECT * FROM `users` WHERE `id` = ?', course.fetch("teacher_id")).first
-                      ^^ This looks like N+1 query.
                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ This looks like N+1 query.
             raise unless teacher
 
@@ -352,7 +344,6 @@
 
           courses.map do |course|
             teacher = db.xquery('SELECT * FROM `users` WHERE `name` = ?', course[:teacher_id]).first
-                      ^^ This looks like N+1 query.
                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ This looks like N+1 query.
             raise unless teacher
 
@@ -375,7 +366,6 @@
         expect_offense(<<~RUBY)
           courses.map do |course|
             teacher = db.xquery('SELECT * FROM `users` WHERE `id` = ?', course[:teacher_id]).first
-                      ^^ This looks like N+1 query.
                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ This looks like N+1 query.
             raise unless teacher
 
@@ -409,7 +399,6 @@
 
     context "has LIMIT" do
       it "registers an offense" do
-        # FIXME: duplicate offense messages
         expect_offense(<<~RUBY)
           courses = db.xquery(
             "SELECT `courses`.*" \\
@@ -421,7 +410,6 @@
 
           courses.map do |course|
             teacher = db.xquery('SELECT * FROM `users` WHERE `id` = ? LIMIT 1', course[:teacher_id]).first
-                      ^^ This looks like N+1 query.
                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ This looks like N+1 query.
             raise unless teacher
 
@@ -441,7 +429,6 @@
 
     context "has GROUP BY" do
       it "registers an offense" do
-        # FIXME: duplicate offense messages
         expect_offense(<<~RUBY)
           courses = db.xquery(
             "SELECT `courses`.*" \\
@@ -453,7 +440,6 @@
 
           courses.map do |course|
             teacher = db.xquery('SELECT * FROM `users` WHERE `id` = ? GROUP BY name', course[:teacher_id]).first
-                      ^^ This looks like N+1 query.
                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ This looks like N+1 query.
             raise unless teacher