aboutsummaryrefslogtreecommitdiffstats
path: root/spec/db
diff options
context:
space:
mode:
authorTeddy Wing2017-11-15 16:22:05 +0100
committerTeddy Wing2017-11-15 16:22:05 +0100
commit35def4911d4eb7f6f3c2aadb6b1983993f9293f6 (patch)
tree78f663c91004a5add71d984a2621fdc472fcb944 /spec/db
parent28b4f328e455a425c6222cdd249515ea958a45d3 (diff)
downloadchouette-core-35def4911d4eb7f6f3c2aadb6b1983993f9293f6.tar.bz2
schema_spec: Remove unnecessary capture
Thanks to Robert for suggesting `(?:...)`. I didn't know that syntax, happy I learned something new. Here, I had used parentheses for the "or" (`|`), but didn't need the captured value. To do the "or" without capturing, you can use the `?:` inside the start of the parentheses. Refs #4951
Diffstat (limited to 'spec/db')
-rw-r--r--spec/db/schema_spec.rb2
1 files changed, 1 insertions, 1 deletions
diff --git a/spec/db/schema_spec.rb b/spec/db/schema_spec.rb
index b2f259abd..74e92bebe 100644
--- a/spec/db/schema_spec.rb
+++ b/spec/db/schema_spec.rb
@@ -16,7 +16,7 @@ RSpec::Matchers.define :use_bigint_keys do
# Primary key
if line =~ /create_table\s/ &&
- !(line =~ /id:\s+(:bigserial|false)/)
+ !(line =~ /id:\s+(?::bigserial|false)/)
expected_line = line.sub(/(create_table\s"\w+",\s)/, '\1id: :bigserial, ')
end