Move basic RSpec rubocop rules to basic cops

These are rules we were already applying.

We've excluded the `factories` folder for some rules because there's a
factory defining a `context` attribute, which rubocop thought was the
`context` RSpec keyword.

.rubocop.yml

@@ -168,15 +168,9 @@ RSpec/AlignRightLetBrace:
 RSpec/AnyInstance:
   Enabled: false
 
-RSpec/AroundBlock:
-  Enabled: true
-
 RSpec/BeEql:
   Enabled: true
 
-RSpec/BeforeAfterAll:
-  Enabled: true
-
 Capybara/FeatureMethods:
   Enabled: false
 
@@ -192,12 +186,6 @@ RSpec/DescribeMethod:
 RSpec/DescribeSymbol:
   Enabled: true
 
-RSpec/EmptyExampleGroup:
-  Enabled: true
-
-RSpec/EmptyLineAfterSubject:
-  Enabled: true
-
 RSpec/ExampleLength:
   Enabled: false
 
@@ -213,9 +201,6 @@ RSpec/ExpectOutput:
 RSpec/FilePath:
   Enabled: true
 
-RSpec/Focus:
-  Enabled: true
-
 RSpec/ImplicitExpect:
   Enabled: true
   EnforcedStyle: should
@@ -238,9 +223,6 @@ RSpec/IteratedExpectation:
 RSpec/LeadingSubject:
   Enabled: true
 
-RSpec/LetBeforeExamples:
-  Enabled: true
-
 RSpec/MessageChain:
   Enabled: true
 
@@ -267,9 +249,6 @@ RSpec/NestedGroups:
   Enabled: true
   Max: 4
 
-RSpec/OverwritingSetup:
-  Enabled: true
-
 RSpec/PredicateMatcher:
   Enabled: true
 
@@ -282,12 +261,6 @@ RSpec/RepeatedDescription:
 RSpec/ReturnFromStub:
   Enabled: true
 
-RSpec/ScatteredLet:
-  Enabled: true
-
-RSpec/ScatteredSetup:
-  Enabled: true
-
 RSpec/SharedContext:
   Enabled: true
 
@@ -300,9 +273,6 @@ RSpec/SubjectStub:
 RSpec/VerifiedDoubles:
   Enabled: true
 
-RSpec/VoidExpect:
-  Enabled: true
-
 Security/Eval:
   Enabled: true
 

.rubocop_basic.yml

@@ -169,21 +169,55 @@ Rails/UnknownEnv:
 Rails/Validation:
   Enabled: true
 
+RSpec/AroundBlock:
+  Enabled: true
+
+RSpec/BeforeAfterAll:
+  Enabled: true
+
+RSpec/EmptyExampleGroup:
+  Enabled: true
+  Exclude:
+    - spec/factories/**/*
+
+RSpec/EmptyLineAfterExampleGroup:
+  Enabled: true
+  Exclude:
+    - spec/factories/**/*
+
 RSpec/ExampleWording:
   Enabled: true
 
+RSpec/Focus:
+  Enabled: true
+
 RSpec/HookArgument:
   Enabled: true
 
+RSpec/LetBeforeExamples:
+  Enabled: true
+
 RSpec/LetSetup:
   Enabled: true
 
 RSpec/NotToNot:
   Enabled: true
 
+RSpec/OverwritingSetup:
+  Enabled: true
+
 RSpec/RepeatedExample:
   Enabled: true
 
+RSpec/ScatteredLet:
+  Enabled: true
+
+RSpec/ScatteredSetup:
+  Enabled: true
+
+RSpec/VoidExpect:
+  Enabled: true
+
 Style/PercentLiteralDelimiters:
   Enabled: true